net: dpaa2-eth: don't use -ENOTSUPP error code
authorVladimir Oltean <vladimir.oltean@nxp.com>
Tue, 29 Nov 2022 14:12:10 +0000 (16:12 +0200)
committerPaolo Abeni <pabeni@redhat.com>
Thu, 1 Dec 2022 12:40:21 +0000 (13:40 +0100)
dpaa2_eth_setup_dpni() is called from the probe path and
dpaa2_eth_set_link_ksettings() is propagated to user space.

include/linux/errno.h says that ENOTSUPP is "Defined for the NFSv3
protocol". Conventional wisdom has it to not use it in networking
drivers. Replace it with -EOPNOTSUPP.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Tested-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c
drivers/net/ethernet/freescale/dpaa2/dpaa2-ethtool.c

index 97e1856..515fcd1 100644 (file)
@@ -3791,7 +3791,7 @@ static int dpaa2_eth_setup_dpni(struct fsl_mc_device *ls_dev)
                dev_err(dev, "DPNI version %u.%u not supported, need >= %u.%u\n",
                        priv->dpni_ver_major, priv->dpni_ver_minor,
                        DPNI_VER_MAJOR, DPNI_VER_MINOR);
-               err = -ENOTSUPP;
+               err = -EOPNOTSUPP;
                goto close;
        }
 
index 32a38a0..ac3a7f2 100644 (file)
@@ -117,7 +117,7 @@ dpaa2_eth_set_link_ksettings(struct net_device *net_dev,
        struct dpaa2_eth_priv *priv = netdev_priv(net_dev);
 
        if (!dpaa2_eth_is_type_phy(priv))
-               return -ENOTSUPP;
+               return -EOPNOTSUPP;
 
        return phylink_ethtool_ksettings_set(priv->mac->phylink, link_settings);
 }