diff mbox series

[net,6/6] net: dsa: mv88e6xxx: enable SerDes PCS register dump via ethtool -d on Topaz

Message ID 20210630174308.31831-7-kabel@kernel.org
State Superseded
Headers show
Series dsa: mv88e6xxx: Topaz fixes | expand

Commit Message

Marek Behún June 30, 2021, 5:43 p.m. UTC
Commit bf3504cea7d7e ("net: dsa: mv88e6xxx: Add 6390 family PCS
registers to ethtool -d") added support for dumping SerDes PCS registers
via ethtool -d for Peridot.

The same implementation is also valid for Topaz, but was not
enabled at the time.

Signed-off-by: Marek Behún <kabel@kernel.org>
Fixes: bf3504cea7d7e ("net: dsa: mv88e6xxx: Add 6390 family PCS registers to ethtool -d")
---
 drivers/net/dsa/mv88e6xxx/chip.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Andrew Lunn June 30, 2021, 8:10 p.m. UTC | #1
On Wed, Jun 30, 2021 at 07:43:08PM +0200, Marek Behún wrote:
> Commit bf3504cea7d7e ("net: dsa: mv88e6xxx: Add 6390 family PCS
> registers to ethtool -d") added support for dumping SerDes PCS registers
> via ethtool -d for Peridot.
> 
> The same implementation is also valid for Topaz, but was not
> enabled at the time.
> 
> Signed-off-by: Marek Behún <kabel@kernel.org>
> Fixes: bf3504cea7d7e ("net: dsa: mv88e6xxx: Add 6390 family PCS registers to ethtool -d")

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

I actually think devlink regions will be better for this. But the
ethtool code exists.

    Andrew
diff mbox series

Patch

diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net/dsa/mv88e6xxx/chip.c
index 1e95a0facbd4..beb41572d04e 100644
--- a/drivers/net/dsa/mv88e6xxx/chip.c
+++ b/drivers/net/dsa/mv88e6xxx/chip.c
@@ -3626,6 +3626,8 @@  static const struct mv88e6xxx_ops mv88e6141_ops = {
 	.serdes_get_sset_count = mv88e6390_serdes_get_sset_count,
 	.serdes_get_strings = mv88e6390_serdes_get_strings,
 	.serdes_get_stats = mv88e6390_serdes_get_stats,
+	.serdes_get_regs_len = mv88e6390_serdes_get_regs_len,
+	.serdes_get_regs = mv88e6390_serdes_get_regs,
 	.phylink_validate = mv88e6341_phylink_validate,
 };
 
@@ -4435,6 +4437,8 @@  static const struct mv88e6xxx_ops mv88e6341_ops = {
 	.serdes_get_sset_count = mv88e6390_serdes_get_sset_count,
 	.serdes_get_strings = mv88e6390_serdes_get_strings,
 	.serdes_get_stats = mv88e6390_serdes_get_stats,
+	.serdes_get_regs_len = mv88e6390_serdes_get_regs_len,
+	.serdes_get_regs = mv88e6390_serdes_get_regs,
 	.phylink_validate = mv88e6341_phylink_validate,
 };