diff options
author | Wang Chen <wangchen@cn.fujitsu.com> | 2008-11-13 02:37:49 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-11-13 02:37:49 -0500 |
commit | 454d7c9b14e20fd1949e2686e9de4a2926e01476 (patch) | |
tree | fae8bd1bfb5fd496977a0639ad71c54a2ee278ae /drivers/net/lp486e.c | |
parent | 7a12122c7a00347da9669cfcde82954c9e3d6f5e (diff) |
netdevice: safe convert to netdev_priv() #part-1
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/lp486e.c')
-rw-r--r-- | drivers/net/lp486e.c | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/drivers/net/lp486e.c b/drivers/net/lp486e.c index d6b0036abafa..4d1a059921c6 100644 --- a/drivers/net/lp486e.c +++ b/drivers/net/lp486e.c | |||
@@ -390,7 +390,7 @@ i596_timeout(struct net_device *dev, char *msg, int ct) { | |||
390 | struct i596_private *lp; | 390 | struct i596_private *lp; |
391 | int boguscnt = ct; | 391 | int boguscnt = ct; |
392 | 392 | ||
393 | lp = (struct i596_private *) dev->priv; | 393 | lp = netdev_priv(dev); |
394 | while (lp->scb.command) { | 394 | while (lp->scb.command) { |
395 | if (--boguscnt == 0) { | 395 | if (--boguscnt == 0) { |
396 | printk("%s: %s timed out - stat %4.4x, cmd %4.4x\n", | 396 | printk("%s: %s timed out - stat %4.4x, cmd %4.4x\n", |
@@ -411,7 +411,7 @@ init_rx_bufs(struct net_device *dev, int num) { | |||
411 | int i; | 411 | int i; |
412 | // struct i596_rbd *rbd; | 412 | // struct i596_rbd *rbd; |
413 | 413 | ||
414 | lp = (struct i596_private *) dev->priv; | 414 | lp = netdev_priv(dev); |
415 | lp->scb.pa_rfd = I596_NULL; | 415 | lp->scb.pa_rfd = I596_NULL; |
416 | 416 | ||
417 | for (i = 0; i < num; i++) { | 417 | for (i = 0; i < num; i++) { |
@@ -468,7 +468,7 @@ remove_rx_bufs(struct net_device *dev) { | |||
468 | struct i596_private *lp; | 468 | struct i596_private *lp; |
469 | struct i596_rfd *rfd; | 469 | struct i596_rfd *rfd; |
470 | 470 | ||
471 | lp = (struct i596_private *) dev->priv; | 471 | lp = netdev_priv(dev); |
472 | lp->rx_tail->pa_next = I596_NULL; | 472 | lp->rx_tail->pa_next = I596_NULL; |
473 | 473 | ||
474 | do { | 474 | do { |
@@ -517,7 +517,7 @@ CLEAR_INT(void) { | |||
517 | /* selftest or dump */ | 517 | /* selftest or dump */ |
518 | static void | 518 | static void |
519 | i596_port_do(struct net_device *dev, int portcmd, char *cmdname) { | 519 | i596_port_do(struct net_device *dev, int portcmd, char *cmdname) { |
520 | struct i596_private *lp = dev->priv; | 520 | struct i596_private *lp = netdev_priv(dev); |
521 | u16 *outp; | 521 | u16 *outp; |
522 | int i, m; | 522 | int i, m; |
523 | 523 | ||
@@ -541,7 +541,7 @@ i596_port_do(struct net_device *dev, int portcmd, char *cmdname) { | |||
541 | 541 | ||
542 | static int | 542 | static int |
543 | i596_scp_setup(struct net_device *dev) { | 543 | i596_scp_setup(struct net_device *dev) { |
544 | struct i596_private *lp = dev->priv; | 544 | struct i596_private *lp = netdev_priv(dev); |
545 | int boguscnt; | 545 | int boguscnt; |
546 | 546 | ||
547 | /* Setup SCP, ISCP, SCB */ | 547 | /* Setup SCP, ISCP, SCB */ |
@@ -622,7 +622,7 @@ init_i596(struct net_device *dev) { | |||
622 | if (i596_scp_setup(dev)) | 622 | if (i596_scp_setup(dev)) |
623 | return 1; | 623 | return 1; |
624 | 624 | ||
625 | lp = (struct i596_private *) dev->priv; | 625 | lp = netdev_priv(dev); |
626 | lp->scb.command = 0; | 626 | lp->scb.command = 0; |
627 | 627 | ||
628 | memcpy ((void *)lp->i596_config, init_setup, 14); | 628 | memcpy ((void *)lp->i596_config, init_setup, 14); |
@@ -704,7 +704,7 @@ i596_rx_one(struct net_device *dev, struct i596_private *lp, | |||
704 | 704 | ||
705 | static int | 705 | static int |
706 | i596_rx(struct net_device *dev) { | 706 | i596_rx(struct net_device *dev) { |
707 | struct i596_private *lp = (struct i596_private *) dev->priv; | 707 | struct i596_private *lp = netdev_priv(dev); |
708 | struct i596_rfd *rfd; | 708 | struct i596_rfd *rfd; |
709 | int frames = 0; | 709 | int frames = 0; |
710 | 710 | ||
@@ -737,7 +737,7 @@ i596_cleanup_cmd(struct net_device *dev) { | |||
737 | struct i596_private *lp; | 737 | struct i596_private *lp; |
738 | struct i596_cmd *cmd; | 738 | struct i596_cmd *cmd; |
739 | 739 | ||
740 | lp = (struct i596_private *) dev->priv; | 740 | lp = netdev_priv(dev); |
741 | while (lp->cmd_head) { | 741 | while (lp->cmd_head) { |
742 | cmd = (struct i596_cmd *)lp->cmd_head; | 742 | cmd = (struct i596_cmd *)lp->cmd_head; |
743 | 743 | ||
@@ -805,7 +805,7 @@ static void i596_reset(struct net_device *dev, struct i596_private *lp, int ioad | |||
805 | } | 805 | } |
806 | 806 | ||
807 | static void i596_add_cmd(struct net_device *dev, struct i596_cmd *cmd) { | 807 | static void i596_add_cmd(struct net_device *dev, struct i596_cmd *cmd) { |
808 | struct i596_private *lp = dev->priv; | 808 | struct i596_private *lp = netdev_priv(dev); |
809 | int ioaddr = dev->base_addr; | 809 | int ioaddr = dev->base_addr; |
810 | unsigned long flags; | 810 | unsigned long flags; |
811 | 811 | ||
@@ -911,7 +911,7 @@ static int i596_start_xmit (struct sk_buff *skb, struct net_device *dev) { | |||
911 | 911 | ||
912 | static void | 912 | static void |
913 | i596_tx_timeout (struct net_device *dev) { | 913 | i596_tx_timeout (struct net_device *dev) { |
914 | struct i596_private *lp = dev->priv; | 914 | struct i596_private *lp = netdev_priv(dev); |
915 | int ioaddr = dev->base_addr; | 915 | int ioaddr = dev->base_addr; |
916 | 916 | ||
917 | /* Transmitter timeout, serious problems. */ | 917 | /* Transmitter timeout, serious problems. */ |
@@ -969,7 +969,7 @@ static int __init lp486e_probe(struct net_device *dev) { | |||
969 | return -EBUSY; | 969 | return -EBUSY; |
970 | } | 970 | } |
971 | 971 | ||
972 | lp = (struct i596_private *) dev->priv; | 972 | lp = netdev_priv(dev); |
973 | spin_lock_init(&lp->cmd_lock); | 973 | spin_lock_init(&lp->cmd_lock); |
974 | 974 | ||
975 | /* | 975 | /* |
@@ -1146,7 +1146,7 @@ static irqreturn_t | |||
1146 | i596_interrupt(int irq, void *dev_instance) | 1146 | i596_interrupt(int irq, void *dev_instance) |
1147 | { | 1147 | { |
1148 | struct net_device *dev = dev_instance; | 1148 | struct net_device *dev = dev_instance; |
1149 | struct i596_private *lp = dev->priv; | 1149 | struct i596_private *lp = netdev_priv(dev); |
1150 | unsigned short status, ack_cmd = 0; | 1150 | unsigned short status, ack_cmd = 0; |
1151 | int frames_in = 0; | 1151 | int frames_in = 0; |
1152 | 1152 | ||
@@ -1214,7 +1214,7 @@ i596_interrupt(int irq, void *dev_instance) | |||
1214 | } | 1214 | } |
1215 | 1215 | ||
1216 | static int i596_close(struct net_device *dev) { | 1216 | static int i596_close(struct net_device *dev) { |
1217 | struct i596_private *lp = dev->priv; | 1217 | struct i596_private *lp = netdev_priv(dev); |
1218 | 1218 | ||
1219 | netif_stop_queue(dev); | 1219 | netif_stop_queue(dev); |
1220 | 1220 | ||
@@ -1241,7 +1241,7 @@ static int i596_close(struct net_device *dev) { | |||
1241 | */ | 1241 | */ |
1242 | 1242 | ||
1243 | static void set_multicast_list(struct net_device *dev) { | 1243 | static void set_multicast_list(struct net_device *dev) { |
1244 | struct i596_private *lp = dev->priv; | 1244 | struct i596_private *lp = netdev_priv(dev); |
1245 | struct i596_cmd *cmd; | 1245 | struct i596_cmd *cmd; |
1246 | 1246 | ||
1247 | if (i596_debug > 1) | 1247 | if (i596_debug > 1) |