chelsio: constify cphy_ops structures
authorJulia Lawall <julia.lawall@lip6.fr>
Sun, 3 Jan 2016 13:09:37 +0000 (14:09 +0100)
committerDavid S. Miller <davem@davemloft.net>
Tue, 5 Jan 2016 02:58:28 +0000 (21:58 -0500)
The cphy_ops structures are never modified, so declare them as const.

Done with the help of Coccinelle.

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/chelsio/cxgb/cphy.h
drivers/net/ethernet/chelsio/cxgb/mv88e1xxx.c
drivers/net/ethernet/chelsio/cxgb/mv88x201x.c
drivers/net/ethernet/chelsio/cxgb/my3126.c
drivers/net/ethernet/chelsio/cxgb3/ael1002.c
drivers/net/ethernet/chelsio/cxgb3/aq100x.c
drivers/net/ethernet/chelsio/cxgb3/common.h
drivers/net/ethernet/chelsio/cxgb3/vsc8211.c

index a4d2a4c..bf43da6 100644 (file)
@@ -137,7 +137,7 @@ static inline int simple_mdio_write(struct cphy *cphy, int reg,
 
 /* Convenience initializer */
 static inline void cphy_init(struct cphy *phy, struct net_device *dev,
-                            int phy_addr, struct cphy_ops *phy_ops,
+                            int phy_addr, const struct cphy_ops *phy_ops,
                             const struct mdio_ops *mdio_ops)
 {
        struct adapter *adapter = netdev_priv(dev);
index 71018a4..76ce6e5 100644 (file)
@@ -337,7 +337,7 @@ static void mv88e1xxx_destroy(struct cphy *cphy)
        kfree(cphy);
 }
 
-static struct cphy_ops mv88e1xxx_ops = {
+static const struct cphy_ops mv88e1xxx_ops = {
        .destroy              = mv88e1xxx_destroy,
        .reset                = mv88e1xxx_reset,
        .interrupt_enable     = mv88e1xxx_interrupt_enable,
index d0cf611..7ddb301 100644 (file)
@@ -195,7 +195,7 @@ static void mv88x201x_destroy(struct cphy *cphy)
        kfree(cphy);
 }
 
-static struct cphy_ops mv88x201x_ops = {
+static const struct cphy_ops mv88x201x_ops = {
        .destroy           = mv88x201x_destroy,
        .reset             = mv88x201x_reset,
        .interrupt_enable  = mv88x201x_interrupt_enable,
index a683fd3..d546f46 100644 (file)
@@ -154,7 +154,7 @@ static void my3126_destroy(struct cphy *cphy)
        kfree(cphy);
 }
 
-static struct cphy_ops my3126_ops = {
+static const struct cphy_ops my3126_ops = {
        .destroy                = my3126_destroy,
        .reset                  = my3126_reset,
        .interrupt_enable       = my3126_interrupt_enable,
index 2028da9..dadf11e 100644 (file)
@@ -198,7 +198,7 @@ static int get_link_status_r(struct cphy *phy, int *link_ok, int *speed,
        return 0;
 }
 
-static struct cphy_ops ael1002_ops = {
+static const struct cphy_ops ael1002_ops = {
        .reset = ael1002_reset,
        .intr_enable = ael1002_intr_noop,
        .intr_disable = ael1002_intr_noop,
@@ -224,7 +224,7 @@ static int ael1006_reset(struct cphy *phy, int wait)
        return t3_phy_reset(phy, MDIO_MMD_PMAPMD, wait);
 }
 
-static struct cphy_ops ael1006_ops = {
+static const struct cphy_ops ael1006_ops = {
        .reset = ael1006_reset,
        .intr_enable = t3_phy_lasi_intr_enable,
        .intr_disable = t3_phy_lasi_intr_disable,
@@ -495,7 +495,7 @@ static int ael2005_intr_handler(struct cphy *phy)
        return ret ? ret : cphy_cause_link_change;
 }
 
-static struct cphy_ops ael2005_ops = {
+static const struct cphy_ops ael2005_ops = {
        .reset           = ael2005_reset,
        .intr_enable     = ael2005_intr_enable,
        .intr_disable    = ael2005_intr_disable,
@@ -801,7 +801,7 @@ static int ael2020_intr_handler(struct cphy *phy)
        return ret ? ret : cphy_cause_link_change;
 }
 
-static struct cphy_ops ael2020_ops = {
+static const struct cphy_ops ael2020_ops = {
        .reset           = ael2020_reset,
        .intr_enable     = ael2020_intr_enable,
        .intr_disable    = ael2020_intr_disable,
@@ -856,7 +856,7 @@ static int get_link_status_x(struct cphy *phy, int *link_ok, int *speed,
        return 0;
 }
 
-static struct cphy_ops qt2045_ops = {
+static const struct cphy_ops qt2045_ops = {
        .reset = ael1006_reset,
        .intr_enable = t3_phy_lasi_intr_enable,
        .intr_disable = t3_phy_lasi_intr_disable,
@@ -921,7 +921,7 @@ static int xaui_direct_power_down(struct cphy *phy, int enable)
        return 0;
 }
 
-static struct cphy_ops xaui_direct_ops = {
+static const struct cphy_ops xaui_direct_ops = {
        .reset = xaui_direct_reset,
        .intr_enable = ael1002_intr_noop,
        .intr_disable = ael1002_intr_noop,
index 341b7ef..6af5d20 100644 (file)
@@ -247,7 +247,7 @@ static int aq100x_get_link_status(struct cphy *phy, int *link_ok,
        return 0;
 }
 
-static struct cphy_ops aq100x_ops = {
+static const struct cphy_ops aq100x_ops = {
        .reset             = aq100x_reset,
        .intr_enable       = aq100x_intr_enable,
        .intr_disable      = aq100x_intr_disable,
index 4424809..1bd7d89 100644 (file)
@@ -575,7 +575,7 @@ static inline int t3_mdio_write(struct cphy *phy, int mmd, int reg,
 
 /* Convenience initializer */
 static inline void cphy_init(struct cphy *phy, struct adapter *adapter,
-                            int phy_addr, struct cphy_ops *phy_ops,
+                            int phy_addr, const struct cphy_ops *phy_ops,
                             const struct mdio_ops *mdio_ops,
                              unsigned int caps, const char *desc)
 {
index 4f9a1c2..8638ad4 100644 (file)
@@ -336,7 +336,7 @@ static int vsc8211_intr_handler(struct cphy *cphy)
        return cphy_cause;
 }
 
-static struct cphy_ops vsc8211_ops = {
+static const struct cphy_ops vsc8211_ops = {
        .reset = vsc8211_reset,
        .intr_enable = vsc8211_intr_enable,
        .intr_disable = vsc8211_intr_disable,
@@ -350,7 +350,7 @@ static struct cphy_ops vsc8211_ops = {
        .power_down = vsc8211_power_down,
 };
 
-static struct cphy_ops vsc8211_fiber_ops = {
+static const struct cphy_ops vsc8211_fiber_ops = {
        .reset = vsc8211_reset,
        .intr_enable = vsc8211_intr_enable,
        .intr_disable = vsc8211_intr_disable,