On 02/06/2017 05:48 AM, Andrew Lunn wrote: >> diff --git a/include/linux/mod_devicetable.h >> b/include/linux/mod_devicetable.h >> index 8a57f0b1242d..8850fcaf50db 100644 >> --- a/include/linux/mod_devicetable.h >> +++ b/include/linux/mod_devicetable.h >> @@ -501,6 +501,7 @@ struct platform_device_id { >> kernel_ulong_t driver_data; >> }; >> >> +#define MDIO_NAME_SIZE 32 > > ... > >> +struct mdio_board_info { >> + const char *bus_id; >> + char modalias[MDIO_NAME_SIZE]; >> + > > I think it might be better to use MII_BUS_ID_SIZE. These are all > related, so using one define should avoid surprise buffer overruns, > truncation's.
I considered doing that, but left it that way because I valued the "const" qualifier more than running out of buffer space. What I could do though, is change the strcmp() in drivers/net/phy/mdio-boardinfo.c to be bound by MII_BUS_ID_SIZE. -- Florian