net: dsa: b53: constify b53_io_ops structures

JuliaLawall · davem330 · commit 0dff88d39fd0 · 2016-08-09T15:09:40.000-07:00
The b53_io_ops structures are never modified, so declare them as const.

Done with the help of Coccinelle.

Signed-off-by: Julia Lawall &lt;Julia.Lawall@lip6.fr&gt;
Acked-by: Florian Fainelli &lt;f.fainelli@gmail.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/dsa/b53/b53_common.c
@@ -1681,7 +1681,8 @@ static int b53_switch_init(struct b53_device *dev)
 	return 0;
 }
 
-struct b53_device *b53_switch_alloc(struct device *base, struct b53_io_ops *ops,
+struct b53_device *b53_switch_alloc(struct device *base,
+				    const struct b53_io_ops *ops,
 				    void *priv)
 {
 	struct dsa_switch *ds;
diff --git a/drivers/net/dsa/b53/b53_mdio.c b/drivers/net/dsa/b53/b53_mdio.c
@@ -267,7 +267,7 @@ static int b53_mdio_phy_write16(struct b53_device *dev, int addr, int reg,
 	return mdiobus_write_nested(bus, addr, reg, value);
 }
 
-static struct b53_io_ops b53_mdio_ops = {
+static const struct b53_io_ops b53_mdio_ops = {
 	.read8 = b53_mdio_read8,
 	.read16 = b53_mdio_read16,
 	.read32 = b53_mdio_read32,
diff --git a/drivers/net/dsa/b53/b53_mmap.c b/drivers/net/dsa/b53/b53_mmap.c
@@ -208,7 +208,7 @@ static int b53_mmap_write64(struct b53_device *dev, u8 page, u8 reg,
 	return 0;
 }
 
-static struct b53_io_ops b53_mmap_ops = {
+static const struct b53_io_ops b53_mmap_ops = {
 	.read8 = b53_mmap_read8,
 	.read16 = b53_mmap_read16,
 	.read32 = b53_mmap_read32,
diff --git a/drivers/net/dsa/b53/b53_priv.h b/drivers/net/dsa/b53/b53_priv.h
@@ -182,7 +182,8 @@ static inline int is_cpu_port(struct b53_device *dev, int port)
 	return dev->cpu_port;
 }
 
-struct b53_device *b53_switch_alloc(struct device *base, struct b53_io_ops *ops,
+struct b53_device *b53_switch_alloc(struct device *base,
+				    const struct b53_io_ops *ops,
 				    void *priv);
 
 int b53_switch_detect(struct b53_device *dev);
diff --git a/drivers/net/dsa/b53/b53_spi.c b/drivers/net/dsa/b53/b53_spi.c
@@ -270,7 +270,7 @@ static int b53_spi_write64(struct b53_device *dev, u8 page, u8 reg, u64 value)
 	return spi_write(spi, txbuf, sizeof(txbuf));
 }
 
-static struct b53_io_ops b53_spi_ops = {
+static const struct b53_io_ops b53_spi_ops = {
 	.read8 = b53_spi_read8,
 	.read16 = b53_spi_read16,
 	.read32 = b53_spi_read32,
diff --git a/drivers/net/dsa/b53/b53_srab.c b/drivers/net/dsa/b53/b53_srab.c
@@ -344,7 +344,7 @@ static int b53_srab_write64(struct b53_device *dev, u8 page, u8 reg,
 	return ret;
 }
 
-static struct b53_io_ops b53_srab_ops = {
+static const struct b53_io_ops b53_srab_ops = {
 	.read8 = b53_srab_read8,
 	.read16 = b53_srab_read16,
 	.read32 = b53_srab_read32,

Original file line number	Diff line number	Diff line change
`@@ -1681,7 +1681,8 @@ static int b53_switch_init(struct b53_device *dev)`
`1681`	`1681`	`return 0;`
`1682`	`1682`	`}`
`1683`	`1683`
`1684`		`-struct b53_device b53_switch_alloc(struct device base, struct b53_io_ops *ops,`
	`1684`	`+struct b53_device b53_switch_alloc(struct device base,`
	`1685`	`+ const struct b53_io_ops *ops,`
`1685`	`1686`	`void *priv)`
`1686`	`1687`	`{`
`1687`	`1688`	`struct dsa_switch *ds;`
Original file line number	Diff line number	Diff line change
`@@ -267,7 +267,7 @@ static int b53_mdio_phy_write16(struct b53_device *dev, int addr, int reg,`
`267`	`267`	`return mdiobus_write_nested(bus, addr, reg, value);`
`268`	`268`	`}`
`269`	`269`
`270`		`-static struct b53_io_ops b53_mdio_ops = {`
	`270`	`+static const struct b53_io_ops b53_mdio_ops = {`
`271`	`271`	`.read8 = b53_mdio_read8,`
`272`	`272`	`.read16 = b53_mdio_read16,`
`273`	`273`	`.read32 = b53_mdio_read32,`
Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,7 @@ static int b53_mmap_write64(struct b53_device *dev, u8 page, u8 reg,`
`208`	`208`	`return 0;`
`209`	`209`	`}`
`210`	`210`
`211`		`-static struct b53_io_ops b53_mmap_ops = {`
	`211`	`+static const struct b53_io_ops b53_mmap_ops = {`
`212`	`212`	`.read8 = b53_mmap_read8,`
`213`	`213`	`.read16 = b53_mmap_read16,`
`214`	`214`	`.read32 = b53_mmap_read32,`
Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,8 @@ static inline int is_cpu_port(struct b53_device *dev, int port)`
`182`	`182`	`return dev->cpu_port;`
`183`	`183`	`}`
`184`	`184`
`185`		`-struct b53_device b53_switch_alloc(struct device base, struct b53_io_ops *ops,`
	`185`	`+struct b53_device b53_switch_alloc(struct device base,`
	`186`	`+ const struct b53_io_ops *ops,`
`186`	`187`	`void *priv);`
`187`	`188`
`188`	`189`	`int b53_switch_detect(struct b53_device *dev);`
Original file line number	Diff line number	Diff line change
`@@ -270,7 +270,7 @@ static int b53_spi_write64(struct b53_device *dev, u8 page, u8 reg, u64 value)`
`270`	`270`	`return spi_write(spi, txbuf, sizeof(txbuf));`
`271`	`271`	`}`
`272`	`272`
`273`		`-static struct b53_io_ops b53_spi_ops = {`
	`273`	`+static const struct b53_io_ops b53_spi_ops = {`
`274`	`274`	`.read8 = b53_spi_read8,`
`275`	`275`	`.read16 = b53_spi_read16,`
`276`	`276`	`.read32 = b53_spi_read32,`
Original file line number	Diff line number	Diff line change
`@@ -344,7 +344,7 @@ static int b53_srab_write64(struct b53_device *dev, u8 page, u8 reg,`
`344`	`344`	`return ret;`
`345`	`345`	`}`
`346`	`346`
`347`		`-static struct b53_io_ops b53_srab_ops = {`
	`347`	`+static const struct b53_io_ops b53_srab_ops = {`
`348`	`348`	`.read8 = b53_srab_read8,`
`349`	`349`	`.read16 = b53_srab_read16,`
`350`	`350`	`.read32 = b53_srab_read32,`