diff mbox

[U-Boot,v2,1/5] net: phy: Fix mask so that we can identify Marvell 88E1518

Message ID 1481290705-10401-2-git-send-email-phil.edworthy@renesas.com
State Superseded
Delegated to: Joe Hershberger
Headers show

Commit Message

Phil Edworthy Dec. 9, 2016, 1:38 p.m. UTC 
The mask for the 88E1510 meant that the 88E1518 code would never be
used.

Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
Reviewed-by: Stefan Roese <sr@denx.de>
---
Note: This has only been tested on a board that uses a Marvell 88E1512
PHY, see subsequent patches.

v2:
 No changes
---
 drivers/net/phy/marvell.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Joe Hershberger Dec. 9, 2016, 7:04 p.m. UTC | #1 
On Fri, Dec 9, 2016 at 7:38 AM, Phil Edworthy <phil.edworthy@renesas.com> wrote:
> The mask for the 88E1510 meant that the 88E1518 code would never be
> used.
>
> Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> Reviewed-by: Stefan Roese <sr@denx.de>

Acked-by: Joe Hershberger <joe.hershberger@ni.com>
diff mbox

Patch

diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
index ac13a67..95d1492 100644
--- a/drivers/net/phy/marvell.c
+++ b/drivers/net/phy/marvell.c
@@ -578,7 +578,7 @@  static struct phy_driver M88E1149S_driver = {
 static struct phy_driver M88E1510_driver = {
 	.name = "Marvell 88E1510",
 	.uid = 0x1410dd0,
-	.mask = 0xffffff0,
+	.mask = 0xfffffff,
 	.features = PHY_GBIT_FEATURES,
 	.config = &m88e1510_config,
 	.startup = &m88e1011s_startup,
@@ -588,7 +588,7 @@  static struct phy_driver M88E1510_driver = {
 static struct phy_driver M88E1518_driver = {
 	.name = "Marvell 88E1518",
 	.uid = 0x1410dd1,
-	.mask = 0xffffff0,
+	.mask = 0xfffffff,
 	.features = PHY_GBIT_FEATURES,
 	.config = &m88e1518_config,
 	.startup = &m88e1011s_startup,