net: phy: don't duplicate driver name in phy_attached_print
authorHeiner Kallweit <hkallweit1@gmail.com>
Sun, 15 Nov 2020 15:03:10 +0000 (16:03 +0100)
committerJakub Kicinski <kuba@kernel.org>
Tue, 17 Nov 2020 18:28:37 +0000 (10:28 -0800)
commita98cabdb8cb8941965721ee8d4edfaa3c7d427d4
tree3be582ec236abb48f0440afd6731d32c50f64e66
parent83c317d7b36bb3858cf1cb86d2635ec3f3bd6ea3
net: phy: don't duplicate driver name in phy_attached_print

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 <hkallweit1@gmail.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Link: https://lore.kernel.org/r/8ab72586-f079-41d8-84ee-9f6a5bd97b2a@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/phy/phy_device.c