[U-Boot] [PATCH] drivers: net: cpsw: Fix reading of mac address for am43 SoCs

cpsw driver tries to get macid for am43xx SoCs using the compatible ti,am4372. But not all variants of am43x uses this complatible like epos evm uses ti,am438x. So use a generic compatible ti,am43 to get macid for all am43 based platforms.
Tested-by: Aparna Balasubramanian aparnab@ti.com Signed-off-by: Lokesh Vutla lokeshvutla@ti.com --- drivers/net/cpsw-common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/cpsw-common.c b/drivers/net/cpsw-common.c index e828e85d8b..607eb4563c 100644 --- a/drivers/net/cpsw-common.c +++ b/drivers/net/cpsw-common.c @@ -110,7 +110,7 @@ int ti_cm_get_macid(struct udevice *dev, int slave, u8 *mac_addr) if (of_device_is_compatible(dev, "ti,dm816-emac")) return cpsw_am33xx_cm_get_macid(dev, 0x30, slave, mac_addr);
- if (of_machine_is_compatible("ti,am4372")) + if (of_machine_is_compatible("ti,am43")) return cpsw_am33xx_cm_get_macid(dev, 0x630, slave, mac_addr);
if (of_machine_is_compatible("ti,dra7"))

+ Joe
On Wednesday 01 February 2017 09:20 AM, Lokesh Vutla wrote:
cpsw driver tries to get macid for am43xx SoCs using the compatible ti,am4372. But not all variants of am43x uses this complatible like epos evm uses ti,am438x. So use a generic compatible ti,am43 to get macid for all am43 based platforms.
Tested-by: Aparna Balasubramanian aparnab@ti.com Signed-off-by: Lokesh Vutla lokeshvutla@ti.com
drivers/net/cpsw-common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/cpsw-common.c b/drivers/net/cpsw-common.c index e828e85d8b..607eb4563c 100644 --- a/drivers/net/cpsw-common.c +++ b/drivers/net/cpsw-common.c @@ -110,7 +110,7 @@ int ti_cm_get_macid(struct udevice *dev, int slave, u8 *mac_addr) if (of_device_is_compatible(dev, "ti,dm816-emac")) return cpsw_am33xx_cm_get_macid(dev, 0x30, slave, mac_addr);
- if (of_machine_is_compatible("ti,am4372"))
if (of_machine_is_compatible("ti,am43")) return cpsw_am33xx_cm_get_macid(dev, 0x630, slave, mac_addr);
if (of_machine_is_compatible("ti,dra7"))

On Tue, Jan 31, 2017 at 9:50 PM, Lokesh Vutla lokeshvutla@ti.com wrote:
cpsw driver tries to get macid for am43xx SoCs using the compatible ti,am4372. But not all variants of am43x uses this complatible like epos evm uses ti,am438x. So use a generic compatible ti,am43 to get macid for all am43 based platforms.
Tested-by: Aparna Balasubramanian aparnab@ti.com Signed-off-by: Lokesh Vutla lokeshvutla@ti.com
Reviewed-by: Joe Hershberger joe.hershberger@ni.com

On Wed, Feb 01, 2017 at 09:20:49AM +0530, Lokesh Vutla wrote:
cpsw driver tries to get macid for am43xx SoCs using the compatible ti,am4372. But not all variants of am43x uses this complatible like epos evm uses ti,am438x. So use a generic compatible ti,am43 to get macid for all am43 based platforms.
Tested-by: Aparna Balasubramanian aparnab@ti.com Signed-off-by: Lokesh Vutla lokeshvutla@ti.com
Reviewed-by: Tom Rini trini@konsulko.com

On Wed, Feb 01, 2017 at 09:20:49AM +0530, Lokesh Vutla wrote:
cpsw driver tries to get macid for am43xx SoCs using the compatible ti,am4372. But not all variants of am43x uses this complatible like epos evm uses ti,am438x. So use a generic compatible ti,am43 to get macid for all am43 based platforms.
Tested-by: Aparna Balasubramanian aparnab@ti.com Signed-off-by: Lokesh Vutla lokeshvutla@ti.com Reviewed-by: Joe Hershberger joe.hershberger@ni.com Reviewed-by: Tom Rini trini@konsulko.com
Applied to u-boot/master, thanks!
participants (3)
-
Joe Hershberger
-
Lokesh Vutla
-
Tom Rini