Multiple drivers copy the MDIO instance name from struct mii_phy to struct bb_miiphy_bus, but the name field in these two structs have different lengths, the bb_miiphy_bus one is 16 bytes, the source one is 32 bytes. Make sure these two lengths are always synchronized, use MDIO_NAME_LEN for both.
Reviewed-by: Paul Barker paul.barker.ct@bp.renesas.com Signed-off-by: Marek Vasut marek.vasut+renesas@mailbox.org --- Cc: Adam Ford aford173@gmail.com Cc: Hai Pham hai.pham.ud@renesas.com Cc: Ilias Apalodimas ilias.apalodimas@linaro.org Cc: Jerome Forissier jerome.forissier@linaro.org Cc: Joe Hershberger joe.hershberger@ni.com Cc: Mario Six mario.six@gdsys.cc Cc: Nobuhiro Iwamatsu nobuhiro.iwamatsu.yj@renesas.com Cc: Paul Barker paul.barker.ct@bp.renesas.com Cc: Peter Robinson pbrobinson@gmail.com Cc: Ramon Fried rfried.dev@gmail.com Cc: Sumit Garg sumit.garg@linaro.org Cc: Tom Rini trini@konsulko.com Cc: u-boot@lists.denx.de --- V2: - Reword commit message - Add RB from Paul --- include/miiphy.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/miiphy.h b/include/miiphy.h index 2bb4f62ea76..1e6c7041fdc 100644 --- a/include/miiphy.h +++ b/include/miiphy.h @@ -64,7 +64,7 @@ void mdio_list_devices(void); #define BB_MII_DEVNAME "bb_miiphy"
struct bb_miiphy_bus { - char name[16]; + char name[MDIO_NAME_LEN]; int (*init)(struct bb_miiphy_bus *bus); int (*mdio_active)(struct bb_miiphy_bus *bus); int (*mdio_tristate)(struct bb_miiphy_bus *bus);