In ft_fixup_enet_phy_connect_type(), use strlen() instead of sizeof() on
the pointer result of phy_string_for_interface().
sizeof() was returning the size of the pointer (4 bytes), resulting in
the phy-connection-type being set to "rgmi" rather than "rgmii-id".

Signed-off-by: Brendan Shanks <brendan.sha...@teradek.com>
Cc: York Sun <york....@nxp.com>
---
 arch/arm/cpu/armv7/ls102xa/fdt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/cpu/armv7/ls102xa/fdt.c b/arch/arm/cpu/armv7/ls102xa/fdt.c
index 6c3caf8..8bf9c42 100644
--- a/arch/arm/cpu/armv7/ls102xa/fdt.c
+++ b/arch/arm/cpu/armv7/ls102xa/fdt.c
@@ -64,8 +64,8 @@ void ft_fixup_enet_phy_connect_type(void *fdt)
                do_fixup_by_path(fdt, enet_path, "phy-connection-type",
                                 phy_string_for_interface(
                                 PHY_INTERFACE_MODE_RGMII_ID),
-                                sizeof(phy_string_for_interface(
-                                PHY_INTERFACE_MODE_RGMII_ID)),
+                                strlen(phy_string_for_interface(
+                                PHY_INTERFACE_MODE_RGMII_ID)) + 1,
                                 1);
        }
 }
-- 
2.7.4

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to