When the link is up on the ADIN1300/ADIN1200, the signal quality on each
pair is indicated in the mean square error register for each pair (MSE_A,
MSE_B, MSE_C, and MSE_D registers, Address 0x8402 to Address 0x8405,
Bits[7:0]).

These values can be useful for some industrial applications.

This change implements support for these registers using the PHY
statistics mechanism.

Signed-off-by: Alexandru Ardelean <alexandru.ardel...@analog.com>
---
 drivers/net/phy/adin.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/drivers/net/phy/adin.c b/drivers/net/phy/adin.c
index 55a0b91816e2..e4441bba98c3 100644
--- a/drivers/net/phy/adin.c
+++ b/drivers/net/phy/adin.c
@@ -184,6 +184,7 @@ struct adin_hw_stat {
        const char *string;
        u16 reg1;
        u16 reg2;
+       bool do_not_accumulate;
 };
 
 static const struct adin_hw_stat adin_hw_stats[] = {
@@ -197,6 +198,10 @@ static const struct adin_hw_stat adin_hw_stats[] = {
        { "odd_preamble_packet_count",          0x9412 },
        { "dribble_bits_frames_count",          0x9413 },
        { "false_carrier_events_count",         0x9414 },
+       { "signal_mean_square_error_a",         0x8402, 0,      true },
+       { "signal_mean_square_error_b",         0x8403, 0,      true },
+       { "signal_mean_square_error_c",         0x8404, 0,      true },
+       { "signal_mean_square_error_d",         0x8405, 0,      true },
 };
 
 /**
@@ -757,7 +762,10 @@ static u64 adin_get_stat(struct phy_device *phydev, int i)
                val = (ret & 0xffff);
        }
 
-       priv->stats[i] += val;
+       if (stat->do_not_accumulate)
+               priv->stats[i] = val;
+       else
+               priv->stats[i] += val;
 
        return priv->stats[i];
 }
-- 
2.27.0

Reply via email to