| Message ID | 20241004161601.2932901-6-maxime.chevallier@bootlin.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | Allow isolating PHY devices | expand |
diff --git a/drivers/net/phy/marvell10g.c b/drivers/net/phy/marvell10g.c index 6642eb642d4b..c1a471f55cd4 100644 --- a/drivers/net/phy/marvell10g.c +++ b/drivers/net/phy/marvell10g.c @@ -1422,6 +1422,7 @@ static struct phy_driver mv3310_drivers[] = { .set_loopback = genphy_c45_loopback, .get_wol = mv3110_get_wol, .set_wol = mv3110_set_wol, + .can_isolate = genphy_no_isolate, }, { .phy_id = MARVELL_PHY_ID_88X3310, @@ -1441,6 +1442,7 @@ static struct phy_driver mv3310_drivers[] = { .set_tunable = mv3310_set_tunable, .remove = mv3310_remove, .set_loopback = genphy_c45_loopback, + .can_isolate = genphy_no_isolate, }, { .phy_id = MARVELL_PHY_ID_88E2110,
The 88x3310 and 88x3340 PHYs don't support the isolation mode, regardless of the MII mode being used (SGMII/1000BaseX/2500BaseX/5GBaseR/10GBaseR). Report this through the .can_isolate() callback. This was confirmed on the 88x3310 on a MacchiatoBin. Signed-off-by: Maxime Chevallier <maxime.chevallier@bootlin.com> --- V2 : Use callback instead of flag drivers/net/phy/marvell10g.c | 2 ++ 1 file changed, 2 insertions(+)