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 > > > > 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 driver name (if set) in the info string. > > > > Example from r8169 that uses phylib: > > > > old: Generic FE-GE Realtek PHY r8169-300:00: attached PHY driver \ > > [Generic FE-GE Realtek PHY] (mii_bus:phy_addr=r8169-300:00, irq=IGNORE) > > new: Generic FE-GE Realtek PHY r8169-300:00: attached PHY driver \ > > (mii_bus:phy_addr=r8169-300:00, irq=IGNORE) > > > > Signed-off-by: Heiner Kallweit <hkallwe...@gmail.com> > > Reviewed-by: Andrew Lunn <and...@lunn.ch>
Applied, thanks!
