Message ID | 20171128132630.26619-1-antoine.tenart@free-electrons.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Series | [net] net: phy: marvell10g: fix the PHY id mask | expand |
On Tue, Nov 28, 2017 at 02:26:30PM +0100, Antoine Tenart wrote: > The Marvell 10G PHY driver supports different hardware revisions, which > have their bits 3..0 differing. To get the correct revision number these > bits should be ignored. This patch fixes this by using the already > defined MARVELL_PHY_ID_MASK (0xfffffff0) instead of the custom > 0xffffffff mask. > > Fixes: 20b2af32ff3f ("net: phy: add Marvell Alaska X 88X3310 10Gigabit PHY support") > Suggested-by: Yan Markman <ymarkman@marvell.com> > Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Andrew
From: Antoine Tenart <antoine.tenart@free-electrons.com> Date: Tue, 28 Nov 2017 14:26:30 +0100 > The Marvell 10G PHY driver supports different hardware revisions, which > have their bits 3..0 differing. To get the correct revision number these > bits should be ignored. This patch fixes this by using the already > defined MARVELL_PHY_ID_MASK (0xfffffff0) instead of the custom > 0xffffffff mask. > > Fixes: 20b2af32ff3f ("net: phy: add Marvell Alaska X 88X3310 10Gigabit PHY support") > Suggested-by: Yan Markman <ymarkman@marvell.com> > Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com> Applied.
diff --git a/drivers/net/phy/marvell10g.c b/drivers/net/phy/marvell10g.c index aebc08beceba..21b3f36e023a 100644 --- a/drivers/net/phy/marvell10g.c +++ b/drivers/net/phy/marvell10g.c @@ -16,6 +16,7 @@ * link takes priority and the other port is completely locked out. */ #include <linux/phy.h> +#include <linux/marvell_phy.h> enum { MV_PCS_BASE_T = 0x0000, @@ -338,7 +339,7 @@ static int mv3310_read_status(struct phy_device *phydev) static struct phy_driver mv3310_drivers[] = { { .phy_id = 0x002b09aa, - .phy_id_mask = 0xffffffff, + .phy_id_mask = MARVELL_PHY_ID_MASK, .name = "mv88x3310", .features = SUPPORTED_10baseT_Full | SUPPORTED_100baseT_Full | @@ -360,7 +361,7 @@ static struct phy_driver mv3310_drivers[] = { module_phy_driver(mv3310_drivers); static struct mdio_device_id __maybe_unused mv3310_tbl[] = { - { 0x002b09aa, 0xffffffff }, + { 0x002b09aa, MARVELL_PHY_ID_MASK }, { }, }; MODULE_DEVICE_TABLE(mdio, mv3310_tbl);
The Marvell 10G PHY driver supports different hardware revisions, which have their bits 3..0 differing. To get the correct revision number these bits should be ignored. This patch fixes this by using the already defined MARVELL_PHY_ID_MASK (0xfffffff0) instead of the custom 0xffffffff mask. Fixes: 20b2af32ff3f ("net: phy: add Marvell Alaska X 88X3310 10Gigabit PHY support") Suggested-by: Yan Markman <ymarkman@marvell.com> Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com> --- drivers/net/phy/marvell10g.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)