diff options
author | Francois Romieu <romieu@fr.zoreil.com> | 2012-03-09 05:53:42 -0500 |
---|---|---|
committer | Francois Romieu <romieu@fr.zoreil.com> | 2012-04-07 05:45:26 -0400 |
commit | ea8f2ed0f1b943e499e760feced4038f95b2b71f (patch) | |
tree | 41ff2615b5b1949582d27bb4e409c3b30541f799 | |
parent | 0c18acc1ed57d54195427d16316d5ea09d840b0e (diff) |
starfire: stop using net_device.{base_addr, irq}.
It's useless to check mem_start on a newly allocated device as well.
Signed-off-by: Francois Romieu <romieu@fr.zoreil.com>
Cc: Ion Badulescu <ionut@badula.org>
-rw-r--r-- | drivers/net/ethernet/adaptec/starfire.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/drivers/net/ethernet/adaptec/starfire.c b/drivers/net/ethernet/adaptec/starfire.c index d896816512ca..4f27bff32d18 100644 --- a/drivers/net/ethernet/adaptec/starfire.c +++ b/drivers/net/ethernet/adaptec/starfire.c | |||
@@ -753,9 +753,6 @@ static int __devinit starfire_init_one(struct pci_dev *pdev, | |||
753 | /* wait a little longer */ | 753 | /* wait a little longer */ |
754 | udelay(1000); | 754 | udelay(1000); |
755 | 755 | ||
756 | dev->base_addr = (unsigned long)base; | ||
757 | dev->irq = irq; | ||
758 | |||
759 | np = netdev_priv(dev); | 756 | np = netdev_priv(dev); |
760 | np->dev = dev; | 757 | np->dev = dev; |
761 | np->base = base; | 758 | np->base = base; |
@@ -773,8 +770,6 @@ static int __devinit starfire_init_one(struct pci_dev *pdev, | |||
773 | drv_flags = netdrv_tbl[chip_idx].drv_flags; | 770 | drv_flags = netdrv_tbl[chip_idx].drv_flags; |
774 | 771 | ||
775 | option = card_idx < MAX_UNITS ? options[card_idx] : 0; | 772 | option = card_idx < MAX_UNITS ? options[card_idx] : 0; |
776 | if (dev->mem_start) | ||
777 | option = dev->mem_start; | ||
778 | 773 | ||
779 | /* The lower four bits are the media type. */ | 774 | /* The lower four bits are the media type. */ |
780 | if (option & 0x200) | 775 | if (option & 0x200) |
@@ -909,13 +904,14 @@ static int netdev_open(struct net_device *dev) | |||
909 | const __be32 *fw_rx_data, *fw_tx_data; | 904 | const __be32 *fw_rx_data, *fw_tx_data; |
910 | struct netdev_private *np = netdev_priv(dev); | 905 | struct netdev_private *np = netdev_priv(dev); |
911 | void __iomem *ioaddr = np->base; | 906 | void __iomem *ioaddr = np->base; |
907 | const int irq = np->pci_dev->irq; | ||
912 | int i, retval; | 908 | int i, retval; |
913 | size_t tx_size, rx_size; | 909 | size_t tx_size, rx_size; |
914 | size_t tx_done_q_size, rx_done_q_size, tx_ring_size, rx_ring_size; | 910 | size_t tx_done_q_size, rx_done_q_size, tx_ring_size, rx_ring_size; |
915 | 911 | ||
916 | /* Do we ever need to reset the chip??? */ | 912 | /* Do we ever need to reset the chip??? */ |
917 | 913 | ||
918 | retval = request_irq(dev->irq, intr_handler, IRQF_SHARED, dev->name, dev); | 914 | retval = request_irq(irq, intr_handler, IRQF_SHARED, dev->name, dev); |
919 | if (retval) | 915 | if (retval) |
920 | return retval; | 916 | return retval; |
921 | 917 | ||
@@ -924,7 +920,7 @@ static int netdev_open(struct net_device *dev) | |||
924 | writel(1, ioaddr + PCIDeviceConfig); | 920 | writel(1, ioaddr + PCIDeviceConfig); |
925 | if (debug > 1) | 921 | if (debug > 1) |
926 | printk(KERN_DEBUG "%s: netdev_open() irq %d.\n", | 922 | printk(KERN_DEBUG "%s: netdev_open() irq %d.\n", |
927 | dev->name, dev->irq); | 923 | dev->name, irq); |
928 | 924 | ||
929 | /* Allocate the various queues. */ | 925 | /* Allocate the various queues. */ |
930 | if (!np->queue_mem) { | 926 | if (!np->queue_mem) { |
@@ -935,7 +931,7 @@ static int netdev_open(struct net_device *dev) | |||
935 | np->queue_mem_size = tx_done_q_size + rx_done_q_size + tx_ring_size + rx_ring_size; | 931 | np->queue_mem_size = tx_done_q_size + rx_done_q_size + tx_ring_size + rx_ring_size; |
936 | np->queue_mem = pci_alloc_consistent(np->pci_dev, np->queue_mem_size, &np->queue_mem_dma); | 932 | np->queue_mem = pci_alloc_consistent(np->pci_dev, np->queue_mem_size, &np->queue_mem_dma); |
937 | if (np->queue_mem == NULL) { | 933 | if (np->queue_mem == NULL) { |
938 | free_irq(dev->irq, dev); | 934 | free_irq(irq, dev); |
939 | return -ENOMEM; | 935 | return -ENOMEM; |
940 | } | 936 | } |
941 | 937 | ||
@@ -1962,7 +1958,7 @@ static int netdev_close(struct net_device *dev) | |||
1962 | } | 1958 | } |
1963 | } | 1959 | } |
1964 | 1960 | ||
1965 | free_irq(dev->irq, dev); | 1961 | free_irq(np->pci_dev->irq, dev); |
1966 | 1962 | ||
1967 | /* Free all the skbuffs in the Rx queue. */ | 1963 | /* Free all the skbuffs in the Rx queue. */ |
1968 | for (i = 0; i < RX_RING_SIZE; i++) { | 1964 | for (i = 0; i < RX_RING_SIZE; i++) { |