net: qualcomm: use net_device_ops instead of direct call
authorStefan Wahren <stefan.wahren@i2se.com>
Mon, 29 May 2017 11:57:13 +0000 (13:57 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 30 May 2017 17:57:28 +0000 (13:57 -0400)
There is no need to export qcaspi_netdev_open and qcaspi_netdev_close
because they are also accessible via the net_device_ops.

Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/qualcomm/qca_debug.c
drivers/net/ethernet/qualcomm/qca_spi.c
drivers/net/ethernet/qualcomm/qca_spi.h

index d145df98feffbca8e76cebff04be6955f5c3b324..92b6be9c44296820b758440c08131aa0eb507c03 100644 (file)
@@ -275,6 +275,7 @@ qcaspi_get_ringparam(struct net_device *dev, struct ethtool_ringparam *ring)
 static int
 qcaspi_set_ringparam(struct net_device *dev, struct ethtool_ringparam *ring)
 {
+       const struct net_device_ops *ops = dev->netdev_ops;
        struct qcaspi *qca = netdev_priv(dev);
 
        if ((ring->rx_pending) ||
@@ -283,13 +284,13 @@ qcaspi_set_ringparam(struct net_device *dev, struct ethtool_ringparam *ring)
                return -EINVAL;
 
        if (netif_running(dev))
-               qcaspi_netdev_close(dev);
+               ops->ndo_stop(dev);
 
        qca->txr.count = max_t(u32, ring->tx_pending, TX_RING_MIN_LEN);
        qca->txr.count = min_t(u16, qca->txr.count, TX_RING_MAX_LEN);
 
        if (netif_running(dev))
-               qcaspi_netdev_open(dev);
+               ops->ndo_open(dev);
 
        return 0;
 }
index 0c3fdeee27ead21aa4c296b164640e462e862747..7e039e37a3c723bd839cb4039f6c80efaf9f4b98 100644 (file)
@@ -603,7 +603,7 @@ qcaspi_intr_handler(int irq, void *data)
        return IRQ_HANDLED;
 }
 
-int
+static int
 qcaspi_netdev_open(struct net_device *dev)
 {
        struct qcaspi *qca = netdev_priv(dev);
@@ -640,7 +640,7 @@ qcaspi_netdev_open(struct net_device *dev)
        return 0;
 }
 
-int
+static int
 qcaspi_netdev_close(struct net_device *dev)
 {
        struct qcaspi *qca = netdev_priv(dev);
index 6e31a0e744a45b48fda518489504a447a7cc6d94..064853ddd678703398705c702307f4bafa920a1a 100644 (file)
@@ -108,7 +108,4 @@ struct qcaspi {
        u16 burst_len;
 };
 
-int qcaspi_netdev_open(struct net_device *dev);
-int qcaspi_netdev_close(struct net_device *dev);
-
 #endif /* _QCA_SPI_H */