[PATCH 2/2] drivers: net: phy: aquantia: make it less verbose

[Priyanka Jain]

>-----Original Message-----
>From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Alex Marginean
>Sent: Wednesday, December 4, 2019 7:02 PM
>To: u-boot at lists.denx.de
>Cc: Joe Hershberger <joe.hershberger at ni.com>
>Subject: [PATCH 2/2] drivers: net: phy: aquantia: make it less verbose
>
>The driver now unconditionally prints some information that's not universally
>useful.  Replace printf with debug.
>
>Signed-off-by: Alex Marginean <alexandru.marginean at nxp.com>
>---
> drivers/net/phy/aquantia.c | 24 ++++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
>diff --git a/drivers/net/phy/aquantia.c b/drivers/net/phy/aquantia.c index
>2af09907a1..aa0deb1f78 100644
>--- a/drivers/net/phy/aquantia.c
>+++ b/drivers/net/phy/aquantia.c
>@@ -409,9 +409,9 @@ int aquantia_config(struct phy_device *phydev)
> 	fault = phy_read(phydev, MDIO_MMD_VEND1, GLOBAL_FAULT);
>
> 	if (id != 0)
>-		printf("%s running firmware version %X.%X.%X\n",
>-		       phydev->dev->name, (id >> 8), id & 0xff,
>-		       (rstatus >> 4) & 0xf);
>+		debug("%s running firmware version %X.%X.%X\n",
>+		      phydev->dev->name, (id >> 8), id & 0xff,
>+		      (rstatus >> 4) & 0xf);
>
> 	if (fault != 0)
> 		printf("%s fault 0x%04x detected\n", phydev->dev->name,
>fault); @@ -492,12 +492,12 @@ int aquantia_config(struct phy_device
>*phydev)
>
> 		if (usx_an) {
> 			reg_val1 |=
>AQUANTIA_USX_AUTONEG_CONTROL_ENA;
>-			printf("%s: system interface USXGMII\n",
>-			       phydev->dev->name);
>+			debug("%s: system interface USXGMII\n",
>+			      phydev->dev->name);
> 		} else {
> 			reg_val1 &=
>~AQUANTIA_USX_AUTONEG_CONTROL_ENA;
>-			printf("%s: system interface XFI\n",
>-			       phydev->dev->name);
>+			debug("%s: system interface XFI\n",
>+			      phydev->dev->name);
> 		}
>
> 		phy_write(phydev, MDIO_MMD_PHYXS,
>@@ -524,11 +524,11 @@ int aquantia_config(struct phy_device *phydev)
> 	val = phy_read(phydev, MDIO_MMD_VEND1,
>AQUANTIA_RESERVED_STATUS);
> 	reg_val1 = phy_read(phydev, MDIO_MMD_VEND1,
>AQUANTIA_FIRMWARE_ID);
>
>-	printf("%s: %s Firmware Version %x.%x.%x\n", phydev->dev->name,
>-	       phydev->drv->name,
>-	       (reg_val1 & AQUANTIA_FIRMWARE_MAJOR_MASK) >> 8,
>-	       reg_val1 & AQUANTIA_FIRMWARE_MINOR_MASK,
>-	       (val & AQUANTIA_FIRMWARE_BUILD_MASK) >> 4);
>+	debug("%s: %s Firmware Version %x.%x.%x\n", phydev->dev->name,
>+	      phydev->drv->name,
>+	      (reg_val1 & AQUANTIA_FIRMWARE_MAJOR_MASK) >> 8,
>+	      reg_val1 & AQUANTIA_FIRMWARE_MINOR_MASK,
>+	      (val & AQUANTIA_FIRMWARE_BUILD_MASK) >> 4);
>
> 	return 0;
> }
>--
>2.17.1
Reviewed-by: Priyanka Jain <priyanka.jain at nxp.com>