On 01/13/2017 08:31 AM, Andrew Lunn wrote:
> On Fri, Jan 13, 2017 at 04:19:12PM +0100, Volodymyr Bendiuga wrote:
>> From: Volodymyr Bendiuga <volodymyr.bendi...@westermo.se>
>>
>> Some bus names are pretty long and do not fit into 20 chars.
>>
>> Signed-off-by: Volodymyr Bendiuga <volodymyr.bendi...@westermo.se>
>> Signed-off-by: Magnus Öberg <magnus.ob...@westermo.se>
>> ---
>> include/linux/phy.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/linux/phy.h b/include/linux/phy.h
>> index feb8a98..b67f94d 100644
>> --- a/include/linux/phy.h
>> +++ b/include/linux/phy.h
>> @@ -162,7 +162,7 @@ static inline const char *phy_modes(phy_interface_t
>> interface)
>> * Need to be a little smaller than phydev->dev.bus_id to leave room
>> * for the ":%02x"
>> */
>> -#define MII_BUS_ID_SIZE (20 - 3)
>> +#define MII_BUS_ID_SIZE (64 - 3)
>
> Hi Volodymyr
>
> Humm, i assume you looked at the comment? What is the size of
> phydev->dev.bus_id?
> Is 61 still a little smaller?
Also it seems like you missed this one:
/* A Structure for boards to register fixups with the PHY Lib */
struct phy_fixup {
struct list_head list;
char bus_id[20];
^================
u32 phy_uid;
u32 phy_uid_mask;
int (*run)(struct phy_device *phydev);
};
Did you really run into naming conflicts where increasing the bus ID
size became the only solution here?
--
Florian
