net: fec_main: Use dev_err() instead of pr_err()
authorFabio Estevam <festevam@gmail.com>
Fri, 7 Jun 2019 12:14:18 +0000 (09:14 -0300)
committerDavid S. Miller <davem@davemloft.net>
Sun, 9 Jun 2019 20:31:26 +0000 (13:31 -0700)
dev_err() is more appropriate for printing error messages inside
drivers, so switch to dev_err().

Signed-off-by: Fabio Estevam <festevam@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/fec_main.c

index 2ee72452ca76db9ae93fd18e137037fe946cb75a..9d459ccf251dd5e37cae0f566a5cf9491a40df02 100644 (file)
@@ -2446,30 +2446,31 @@ static int
 fec_enet_set_coalesce(struct net_device *ndev, struct ethtool_coalesce *ec)
 {
        struct fec_enet_private *fep = netdev_priv(ndev);
+       struct device *dev = &fep->pdev->dev;
        unsigned int cycle;
 
        if (!(fep->quirks & FEC_QUIRK_HAS_COALESCE))
                return -EOPNOTSUPP;
 
        if (ec->rx_max_coalesced_frames > 255) {
-               pr_err("Rx coalesced frames exceed hardware limitation\n");
+               dev_err(dev, "Rx coalesced frames exceed hardware limitation\n");
                return -EINVAL;
        }
 
        if (ec->tx_max_coalesced_frames > 255) {
-               pr_err("Tx coalesced frame exceed hardware limitation\n");
+               dev_err(dev, "Tx coalesced frame exceed hardware limitation\n");
                return -EINVAL;
        }
 
        cycle = fec_enet_us_to_itr_clock(ndev, fep->rx_time_itr);
        if (cycle > 0xFFFF) {
-               pr_err("Rx coalesced usec exceed hardware limitation\n");
+               dev_err(dev, "Rx coalesced usec exceed hardware limitation\n");
                return -EINVAL;
        }
 
        cycle = fec_enet_us_to_itr_clock(ndev, fep->tx_time_itr);
        if (cycle > 0xFFFF) {
-               pr_err("Rx coalesced usec exceed hardware limitation\n");
+               dev_err(dev, "Rx coalesced usec exceed hardware limitation\n");
                return -EINVAL;
        }