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/via-ircc.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/via-ircc.c')
-rw-r--r-- | drivers/net/irda/via-ircc.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/drivers/net/irda/via-ircc.c b/drivers/net/irda/via-ircc.c index 84e609ea5fbb..74c78cf7a333 100644 --- a/drivers/net/irda/via-ircc.c +++ b/drivers/net/irda/via-ircc.c | |||
@@ -334,7 +334,7 @@ static __devinit int via_ircc_open(int i, chipio_t * info, unsigned int id) | |||
334 | if (dev == NULL) | 334 | if (dev == NULL) |
335 | return -ENOMEM; | 335 | return -ENOMEM; |
336 | 336 | ||
337 | self = dev->priv; | 337 | self = netdev_priv(dev); |
338 | self->netdev = dev; | 338 | self->netdev = dev; |
339 | spin_lock_init(&self->lock); | 339 | spin_lock_init(&self->lock); |
340 | 340 | ||
@@ -824,7 +824,7 @@ static int via_ircc_hard_xmit_sir(struct sk_buff *skb, | |||
824 | u16 iobase; | 824 | u16 iobase; |
825 | __u32 speed; | 825 | __u32 speed; |
826 | 826 | ||
827 | self = (struct via_ircc_cb *) dev->priv; | 827 | self = netdev_priv(dev); |
828 | IRDA_ASSERT(self != NULL, return 0;); | 828 | IRDA_ASSERT(self != NULL, return 0;); |
829 | iobase = self->io.fir_base; | 829 | iobase = self->io.fir_base; |
830 | 830 | ||
@@ -896,7 +896,7 @@ static int via_ircc_hard_xmit_fir(struct sk_buff *skb, | |||
896 | __u32 speed; | 896 | __u32 speed; |
897 | unsigned long flags; | 897 | unsigned long flags; |
898 | 898 | ||
899 | self = (struct via_ircc_cb *) dev->priv; | 899 | self = netdev_priv(dev); |
900 | iobase = self->io.fir_base; | 900 | iobase = self->io.fir_base; |
901 | 901 | ||
902 | if (self->st_fifo.len) | 902 | if (self->st_fifo.len) |
@@ -1349,7 +1349,7 @@ static int RxTimerHandler(struct via_ircc_cb *self, int iobase) | |||
1349 | static irqreturn_t via_ircc_interrupt(int dummy, void *dev_id) | 1349 | static irqreturn_t via_ircc_interrupt(int dummy, void *dev_id) |
1350 | { | 1350 | { |
1351 | struct net_device *dev = dev_id; | 1351 | struct net_device *dev = dev_id; |
1352 | struct via_ircc_cb *self = dev->priv; | 1352 | struct via_ircc_cb *self = netdev_priv(dev); |
1353 | int iobase; | 1353 | int iobase; |
1354 | u8 iHostIntType, iRxIntType, iTxIntType; | 1354 | u8 iHostIntType, iRxIntType, iTxIntType; |
1355 | 1355 | ||
@@ -1522,7 +1522,7 @@ static int via_ircc_net_open(struct net_device *dev) | |||
1522 | IRDA_DEBUG(3, "%s()\n", __func__); | 1522 | IRDA_DEBUG(3, "%s()\n", __func__); |
1523 | 1523 | ||
1524 | IRDA_ASSERT(dev != NULL, return -1;); | 1524 | IRDA_ASSERT(dev != NULL, return -1;); |
1525 | self = (struct via_ircc_cb *) dev->priv; | 1525 | self = netdev_priv(dev); |
1526 | self->stats.rx_packets = 0; | 1526 | self->stats.rx_packets = 0; |
1527 | IRDA_ASSERT(self != NULL, return 0;); | 1527 | IRDA_ASSERT(self != NULL, return 0;); |
1528 | iobase = self->io.fir_base; | 1528 | iobase = self->io.fir_base; |
@@ -1589,7 +1589,7 @@ static int via_ircc_net_close(struct net_device *dev) | |||
1589 | IRDA_DEBUG(3, "%s()\n", __func__); | 1589 | IRDA_DEBUG(3, "%s()\n", __func__); |
1590 | 1590 | ||
1591 | IRDA_ASSERT(dev != NULL, return -1;); | 1591 | IRDA_ASSERT(dev != NULL, return -1;); |
1592 | self = (struct via_ircc_cb *) dev->priv; | 1592 | self = netdev_priv(dev); |
1593 | IRDA_ASSERT(self != NULL, return 0;); | 1593 | IRDA_ASSERT(self != NULL, return 0;); |
1594 | 1594 | ||
1595 | /* Stop device */ | 1595 | /* Stop device */ |
@@ -1628,7 +1628,7 @@ static int via_ircc_net_ioctl(struct net_device *dev, struct ifreq *rq, | |||
1628 | int ret = 0; | 1628 | int ret = 0; |
1629 | 1629 | ||
1630 | IRDA_ASSERT(dev != NULL, return -1;); | 1630 | IRDA_ASSERT(dev != NULL, return -1;); |
1631 | self = dev->priv; | 1631 | self = netdev_priv(dev); |
1632 | IRDA_ASSERT(self != NULL, return -1;); | 1632 | IRDA_ASSERT(self != NULL, return -1;); |
1633 | IRDA_DEBUG(1, "%s(), %s, (cmd=0x%X)\n", __func__, dev->name, | 1633 | IRDA_DEBUG(1, "%s(), %s, (cmd=0x%X)\n", __func__, dev->name, |
1634 | cmd); | 1634 | cmd); |
@@ -1663,7 +1663,7 @@ static int via_ircc_net_ioctl(struct net_device *dev, struct ifreq *rq, | |||
1663 | static struct net_device_stats *via_ircc_net_get_stats(struct net_device | 1663 | static struct net_device_stats *via_ircc_net_get_stats(struct net_device |
1664 | *dev) | 1664 | *dev) |
1665 | { | 1665 | { |
1666 | struct via_ircc_cb *self = (struct via_ircc_cb *) dev->priv; | 1666 | struct via_ircc_cb *self = netdev_priv(dev); |
1667 | 1667 | ||
1668 | return &self->stats; | 1668 | return &self->stats; |
1669 | } | 1669 | } |