diff options
author | Jingoo Han <jg1.han@samsung.com> | 2013-10-20 22:09:26 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-10-21 17:21:00 -0400 |
commit | 3c5b9caf3237b16a9e5019f523b284f056bc4f74 (patch) | |
tree | c8c150ceb9f711302398ba85d2804a70933006cd /drivers/net | |
parent | 5eccdfaabcf44f71702ba033d9c24821111749e1 (diff) |
net: tulip: remove unnecessary pci_set_drvdata()
The driver core clears the driver data to NULL after device_release
or on probe failure. Thus, it is not needed to manually clear the
device driver data to NULL.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/ethernet/dec/tulip/de2104x.c | 1 | ||||
-rw-r--r-- | drivers/net/ethernet/dec/tulip/dmfe.c | 3 | ||||
-rw-r--r-- | drivers/net/ethernet/dec/tulip/tulip_core.c | 1 | ||||
-rw-r--r-- | drivers/net/ethernet/dec/tulip/uli526x.c | 2 | ||||
-rw-r--r-- | drivers/net/ethernet/dec/tulip/winbond-840.c | 3 | ||||
-rw-r--r-- | drivers/net/ethernet/dec/tulip/xircom_cb.c | 2 |
6 files changed, 0 insertions, 12 deletions
diff --git a/drivers/net/ethernet/dec/tulip/de2104x.c b/drivers/net/ethernet/dec/tulip/de2104x.c index eaab73cf27ca..38148b0e3a95 100644 --- a/drivers/net/ethernet/dec/tulip/de2104x.c +++ b/drivers/net/ethernet/dec/tulip/de2104x.c | |||
@@ -2110,7 +2110,6 @@ static void de_remove_one(struct pci_dev *pdev) | |||
2110 | iounmap(de->regs); | 2110 | iounmap(de->regs); |
2111 | pci_release_regions(pdev); | 2111 | pci_release_regions(pdev); |
2112 | pci_disable_device(pdev); | 2112 | pci_disable_device(pdev); |
2113 | pci_set_drvdata(pdev, NULL); | ||
2114 | free_netdev(dev); | 2113 | free_netdev(dev); |
2115 | } | 2114 | } |
2116 | 2115 | ||
diff --git a/drivers/net/ethernet/dec/tulip/dmfe.c b/drivers/net/ethernet/dec/tulip/dmfe.c index 83139307861c..5ad9e3e3c0b8 100644 --- a/drivers/net/ethernet/dec/tulip/dmfe.c +++ b/drivers/net/ethernet/dec/tulip/dmfe.c | |||
@@ -523,7 +523,6 @@ err_out_res: | |||
523 | err_out_disable: | 523 | err_out_disable: |
524 | pci_disable_device(pdev); | 524 | pci_disable_device(pdev); |
525 | err_out_free: | 525 | err_out_free: |
526 | pci_set_drvdata(pdev, NULL); | ||
527 | free_netdev(dev); | 526 | free_netdev(dev); |
528 | 527 | ||
529 | return err; | 528 | return err; |
@@ -548,8 +547,6 @@ static void dmfe_remove_one(struct pci_dev *pdev) | |||
548 | db->buf_pool_ptr, db->buf_pool_dma_ptr); | 547 | db->buf_pool_ptr, db->buf_pool_dma_ptr); |
549 | pci_release_regions(pdev); | 548 | pci_release_regions(pdev); |
550 | free_netdev(dev); /* free board information */ | 549 | free_netdev(dev); /* free board information */ |
551 | |||
552 | pci_set_drvdata(pdev, NULL); | ||
553 | } | 550 | } |
554 | 551 | ||
555 | DMFE_DBUG(0, "dmfe_remove_one() exit", 0); | 552 | DMFE_DBUG(0, "dmfe_remove_one() exit", 0); |
diff --git a/drivers/net/ethernet/dec/tulip/tulip_core.c b/drivers/net/ethernet/dec/tulip/tulip_core.c index 4e8cfa2ac803..add05f14b38b 100644 --- a/drivers/net/ethernet/dec/tulip/tulip_core.c +++ b/drivers/net/ethernet/dec/tulip/tulip_core.c | |||
@@ -1939,7 +1939,6 @@ static void tulip_remove_one(struct pci_dev *pdev) | |||
1939 | pci_iounmap(pdev, tp->base_addr); | 1939 | pci_iounmap(pdev, tp->base_addr); |
1940 | free_netdev (dev); | 1940 | free_netdev (dev); |
1941 | pci_release_regions (pdev); | 1941 | pci_release_regions (pdev); |
1942 | pci_set_drvdata (pdev, NULL); | ||
1943 | 1942 | ||
1944 | /* pci_power_off (pdev, -1); */ | 1943 | /* pci_power_off (pdev, -1); */ |
1945 | } | 1944 | } |
diff --git a/drivers/net/ethernet/dec/tulip/uli526x.c b/drivers/net/ethernet/dec/tulip/uli526x.c index 93845afe1cea..a5397b130724 100644 --- a/drivers/net/ethernet/dec/tulip/uli526x.c +++ b/drivers/net/ethernet/dec/tulip/uli526x.c | |||
@@ -429,7 +429,6 @@ err_out_release: | |||
429 | err_out_disable: | 429 | err_out_disable: |
430 | pci_disable_device(pdev); | 430 | pci_disable_device(pdev); |
431 | err_out_free: | 431 | err_out_free: |
432 | pci_set_drvdata(pdev, NULL); | ||
433 | free_netdev(dev); | 432 | free_netdev(dev); |
434 | 433 | ||
435 | return err; | 434 | return err; |
@@ -450,7 +449,6 @@ static void uli526x_remove_one(struct pci_dev *pdev) | |||
450 | db->buf_pool_ptr, db->buf_pool_dma_ptr); | 449 | db->buf_pool_ptr, db->buf_pool_dma_ptr); |
451 | pci_release_regions(pdev); | 450 | pci_release_regions(pdev); |
452 | pci_disable_device(pdev); | 451 | pci_disable_device(pdev); |
453 | pci_set_drvdata(pdev, NULL); | ||
454 | free_netdev(dev); | 452 | free_netdev(dev); |
455 | } | 453 | } |
456 | 454 | ||
diff --git a/drivers/net/ethernet/dec/tulip/winbond-840.c b/drivers/net/ethernet/dec/tulip/winbond-840.c index c7b04ecf5b49..62fe512bb216 100644 --- a/drivers/net/ethernet/dec/tulip/winbond-840.c +++ b/drivers/net/ethernet/dec/tulip/winbond-840.c | |||
@@ -468,7 +468,6 @@ static int w840_probe1(struct pci_dev *pdev, const struct pci_device_id *ent) | |||
468 | return 0; | 468 | return 0; |
469 | 469 | ||
470 | err_out_cleardev: | 470 | err_out_cleardev: |
471 | pci_set_drvdata(pdev, NULL); | ||
472 | pci_iounmap(pdev, ioaddr); | 471 | pci_iounmap(pdev, ioaddr); |
473 | err_out_free_res: | 472 | err_out_free_res: |
474 | pci_release_regions(pdev); | 473 | pci_release_regions(pdev); |
@@ -1542,8 +1541,6 @@ static void w840_remove1(struct pci_dev *pdev) | |||
1542 | pci_iounmap(pdev, np->base_addr); | 1541 | pci_iounmap(pdev, np->base_addr); |
1543 | free_netdev(dev); | 1542 | free_netdev(dev); |
1544 | } | 1543 | } |
1545 | |||
1546 | pci_set_drvdata(pdev, NULL); | ||
1547 | } | 1544 | } |
1548 | 1545 | ||
1549 | #ifdef CONFIG_PM | 1546 | #ifdef CONFIG_PM |
diff --git a/drivers/net/ethernet/dec/tulip/xircom_cb.c b/drivers/net/ethernet/dec/tulip/xircom_cb.c index 9b84cb04fe5f..ab7ebac6fbea 100644 --- a/drivers/net/ethernet/dec/tulip/xircom_cb.c +++ b/drivers/net/ethernet/dec/tulip/xircom_cb.c | |||
@@ -289,7 +289,6 @@ out: | |||
289 | err_unmap: | 289 | err_unmap: |
290 | pci_iounmap(pdev, private->ioaddr); | 290 | pci_iounmap(pdev, private->ioaddr); |
291 | reg_fail: | 291 | reg_fail: |
292 | pci_set_drvdata(pdev, NULL); | ||
293 | dma_free_coherent(d, 8192, private->tx_buffer, private->tx_dma_handle); | 292 | dma_free_coherent(d, 8192, private->tx_buffer, private->tx_dma_handle); |
294 | tx_buf_fail: | 293 | tx_buf_fail: |
295 | dma_free_coherent(d, 8192, private->rx_buffer, private->rx_dma_handle); | 294 | dma_free_coherent(d, 8192, private->rx_buffer, private->rx_dma_handle); |
@@ -317,7 +316,6 @@ static void xircom_remove(struct pci_dev *pdev) | |||
317 | 316 | ||
318 | unregister_netdev(dev); | 317 | unregister_netdev(dev); |
319 | pci_iounmap(pdev, card->ioaddr); | 318 | pci_iounmap(pdev, card->ioaddr); |
320 | pci_set_drvdata(pdev, NULL); | ||
321 | dma_free_coherent(d, 8192, card->tx_buffer, card->tx_dma_handle); | 319 | dma_free_coherent(d, 8192, card->tx_buffer, card->tx_dma_handle); |
322 | dma_free_coherent(d, 8192, card->rx_buffer, card->rx_dma_handle); | 320 | dma_free_coherent(d, 8192, card->rx_buffer, card->rx_dma_handle); |
323 | free_netdev(dev); | 321 | free_netdev(dev); |