Re: [PATCH net-next] net: phy: don't duplicate driver name in phy_attached_print

2020-11-17 Thread Jakub Kicinski
On Tue, 17 Nov 2020 03:53:54 +0100 Andrew Lunn wrote: > On Sun, Nov 15, 2020 at 04:03:10PM +0100, Heiner Kallweit wrote: > > Currently we print the driver name twice in phy_attached_print(): > > - phy_dev_info() prints it as part of the device info > > - and we print it as part of the info string >

Re: [PATCH net-next] net: phy: don't duplicate driver name in phy_attached_print

2020-11-16 Thread Andrew Lunn
On Sun, Nov 15, 2020 at 04:03:10PM +0100, Heiner Kallweit wrote: > Currently we print the driver name twice in phy_attached_print(): > - phy_dev_info() prints it as part of the device info > - and we print it as part of the info string > > This is a little bit ugly, it makes the info harder to rea

[PATCH net-next] net: phy: don't duplicate driver name in phy_attached_print

2020-11-15 Thread Heiner Kallweit
Currently we print the driver name twice in phy_attached_print(): - phy_dev_info() prints it as part of the device info - and we print it as part of the info string This is a little bit ugly, it makes the info harder to read, especially if the driver name is a little bit longer. Therefore omit the