[PATCH 17/20] dpaa_eth: remove netdev_err() for user errors

From: Madalin Bucur
Date: Tue Oct 08 2019 - 08:11:42 EST


User reports that an application making an (incorrect) call to
restart AN on a fixed link DPAA interface triggers an error in
the kernel log while the returned EINVAL should be enough.

Reported-by: Joakim Tjernlund <Joakim.Tjernlund@xxxxxxxxxxxx>
Signed-off-by: Madalin Bucur <madalin.bucur@xxxxxxx>
---
drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c | 4 ----
1 file changed, 4 deletions(-)

diff --git a/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c b/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c
index 1c689e11c61f..126c0f1d8442 100644
--- a/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c
+++ b/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c
@@ -81,7 +81,6 @@ static int dpaa_get_link_ksettings(struct net_device *net_dev,
struct ethtool_link_ksettings *cmd)
{
if (!net_dev->phydev) {
- netdev_dbg(net_dev, "phy device not initialized\n");
return 0;
}

@@ -96,7 +95,6 @@ static int dpaa_set_link_ksettings(struct net_device *net_dev,
int err;

if (!net_dev->phydev) {
- netdev_err(net_dev, "phy device not initialized\n");
return -ENODEV;
}

@@ -143,7 +141,6 @@ static int dpaa_nway_reset(struct net_device *net_dev)
int err;

if (!net_dev->phydev) {
- netdev_err(net_dev, "phy device not initialized\n");
return -ENODEV;
}

@@ -168,7 +165,6 @@ static void dpaa_get_pauseparam(struct net_device *net_dev,
mac_dev = priv->mac_dev;

if (!net_dev->phydev) {
- netdev_err(net_dev, "phy device not initialized\n");
return;
}

--
2.1.0