diff options
author | Wang Chen <wangchen@cn.fujitsu.com> | 2008-11-13 02:38:14 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-11-13 02:38:14 -0500 |
commit | 4cf1653aa90c6320dc8032443b5e322820aa28b1 (patch) | |
tree | 7920008d3f953defdcda8c81d39da963290d93ad /drivers/net/irda/w83977af_ir.c | |
parent | 454d7c9b14e20fd1949e2686e9de4a2926e01476 (diff) |
netdevice: safe convert to netdev_priv() #part-2
We have some reasons to kill netdev->priv:
1. netdev->priv is equal to netdev_priv().
2. netdev_priv() wraps the calculation of netdev->priv's offset, obviously
netdev_priv() is more flexible than netdev->priv.
But we cann't kill netdev->priv, because so many drivers reference to it
directly.
This patch is a safe convert for netdev->priv to netdev_priv(netdev).
Since all of the netdev->priv is only for read.
But it is too big to be sent in one mail.
I split it to 4 parts and make every part smaller than 100,000 bytes,
which is max size allowed by vger.
Signed-off-by: Wang Chen <wangchen@cn.fujitsu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/irda/w83977af_ir.c')
-rw-r--r-- | drivers/net/irda/w83977af_ir.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index fa600200ac01..161d591aeb97 100644 --- a/drivers/net/irda/w83977af_ir.c +++ b/drivers/net/irda/w83977af_ir.c | |||
@@ -178,7 +178,7 @@ int w83977af_open(int i, unsigned int iobase, unsigned int irq, | |||
178 | goto err_out; | 178 | goto err_out; |
179 | } | 179 | } |
180 | 180 | ||
181 | self = dev->priv; | 181 | self = netdev_priv(dev); |
182 | spin_lock_init(&self->lock); | 182 | spin_lock_init(&self->lock); |
183 | 183 | ||
184 | 184 | ||
@@ -497,7 +497,7 @@ int w83977af_hard_xmit(struct sk_buff *skb, struct net_device *dev) | |||
497 | __u8 set; | 497 | __u8 set; |
498 | int mtt; | 498 | int mtt; |
499 | 499 | ||
500 | self = (struct w83977af_ir *) dev->priv; | 500 | self = netdev_priv(dev); |
501 | 501 | ||
502 | iobase = self->io.fir_base; | 502 | iobase = self->io.fir_base; |
503 | 503 | ||
@@ -1118,7 +1118,7 @@ static irqreturn_t w83977af_interrupt(int irq, void *dev_id) | |||
1118 | __u8 set, icr, isr; | 1118 | __u8 set, icr, isr; |
1119 | int iobase; | 1119 | int iobase; |
1120 | 1120 | ||
1121 | self = dev->priv; | 1121 | self = netdev_priv(dev); |
1122 | 1122 | ||
1123 | iobase = self->io.fir_base; | 1123 | iobase = self->io.fir_base; |
1124 | 1124 | ||
@@ -1191,7 +1191,7 @@ static int w83977af_net_open(struct net_device *dev) | |||
1191 | IRDA_DEBUG(0, "%s()\n", __func__ ); | 1191 | IRDA_DEBUG(0, "%s()\n", __func__ ); |
1192 | 1192 | ||
1193 | IRDA_ASSERT(dev != NULL, return -1;); | 1193 | IRDA_ASSERT(dev != NULL, return -1;); |
1194 | self = (struct w83977af_ir *) dev->priv; | 1194 | self = netdev_priv(dev); |
1195 | 1195 | ||
1196 | IRDA_ASSERT(self != NULL, return 0;); | 1196 | IRDA_ASSERT(self != NULL, return 0;); |
1197 | 1197 | ||
@@ -1255,7 +1255,7 @@ static int w83977af_net_close(struct net_device *dev) | |||
1255 | 1255 | ||
1256 | IRDA_ASSERT(dev != NULL, return -1;); | 1256 | IRDA_ASSERT(dev != NULL, return -1;); |
1257 | 1257 | ||
1258 | self = (struct w83977af_ir *) dev->priv; | 1258 | self = netdev_priv(dev); |
1259 | 1259 | ||
1260 | IRDA_ASSERT(self != NULL, return 0;); | 1260 | IRDA_ASSERT(self != NULL, return 0;); |
1261 | 1261 | ||
@@ -1302,7 +1302,7 @@ static int w83977af_net_ioctl(struct net_device *dev, struct ifreq *rq, int cmd) | |||
1302 | 1302 | ||
1303 | IRDA_ASSERT(dev != NULL, return -1;); | 1303 | IRDA_ASSERT(dev != NULL, return -1;); |
1304 | 1304 | ||
1305 | self = dev->priv; | 1305 | self = netdev_priv(dev); |
1306 | 1306 | ||
1307 | IRDA_ASSERT(self != NULL, return -1;); | 1307 | IRDA_ASSERT(self != NULL, return -1;); |
1308 | 1308 | ||
@@ -1338,7 +1338,7 @@ out: | |||
1338 | 1338 | ||
1339 | static struct net_device_stats *w83977af_net_get_stats(struct net_device *dev) | 1339 | static struct net_device_stats *w83977af_net_get_stats(struct net_device *dev) |
1340 | { | 1340 | { |
1341 | struct w83977af_ir *self = (struct w83977af_ir *) dev->priv; | 1341 | struct w83977af_ir *self = netdev_priv(dev); |
1342 | 1342 | ||
1343 | return &self->stats; | 1343 | return &self->stats; |
1344 | } | 1344 | } |