* [PATCH 1/3] wireless: ipw2100: remove unnecessary pci_set_drvdata()
@ 2013-11-25 1:49 Jingoo Han
2013-11-25 1:51 ` [PATCH 2/3] wireless: prism54: " Jingoo Han
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Jingoo Han @ 2013-11-25 1:49 UTC (permalink / raw)
To: 'John W. Linville'
Cc: linux-wireless, 'Jingoo Han',
'Stanislav Yakovlev'
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>
---
drivers/net/wireless/ipw2x00/ipw2100.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/wireless/ipw2x00/ipw2100.c b/drivers/net/wireless/ipw2x00/ipw2100.c
index f8ab193..2f4233a 100644
--- a/drivers/net/wireless/ipw2x00/ipw2100.c
+++ b/drivers/net/wireless/ipw2x00/ipw2100.c
@@ -6362,7 +6362,6 @@ out:
&ipw2100_attribute_group);
free_libipw(dev, 0);
- pci_set_drvdata(pci_dev, NULL);
}
pci_iounmap(pci_dev, ioaddr);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH 2/3] wireless: prism54: remove unnecessary pci_set_drvdata()
2013-11-25 1:49 [PATCH 1/3] wireless: ipw2100: remove unnecessary pci_set_drvdata() Jingoo Han
@ 2013-11-25 1:51 ` Jingoo Han
2013-11-25 1:52 ` [PATCH 3/3] wireless: rt2x00: " Jingoo Han
2013-11-26 6:55 ` [PATCH 1/3] wireless: ipw2100: " Stanislav Yakovlev
2 siblings, 0 replies; 5+ messages in thread
From: Jingoo Han @ 2013-11-25 1:51 UTC (permalink / raw)
To: 'John W. Linville'; +Cc: linux-wireless, 'Jingoo Han'
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>
---
drivers/net/wireless/prism54/islpci_dev.c | 1 -
drivers/net/wireless/prism54/islpci_hotplug.c | 2 --
2 files changed, 3 deletions(-)
diff --git a/drivers/net/wireless/prism54/islpci_dev.c b/drivers/net/wireless/prism54/islpci_dev.c
index e05d9b4..7fa2898 100644
--- a/drivers/net/wireless/prism54/islpci_dev.c
+++ b/drivers/net/wireless/prism54/islpci_dev.c
@@ -914,7 +914,6 @@ islpci_setup(struct pci_dev *pdev)
do_islpci_free_memory:
islpci_free_memory(priv);
do_free_netdev:
- pci_set_drvdata(pdev, NULL);
free_netdev(ndev);
priv = NULL;
return NULL;
diff --git a/drivers/net/wireless/prism54/islpci_hotplug.c b/drivers/net/wireless/prism54/islpci_hotplug.c
index 9e68e0c..d7b9e63 100644
--- a/drivers/net/wireless/prism54/islpci_hotplug.c
+++ b/drivers/net/wireless/prism54/islpci_hotplug.c
@@ -199,7 +199,6 @@ prism54_probe(struct pci_dev *pdev, const struct pci_device_id *id)
do_unregister_netdev:
unregister_netdev(ndev);
islpci_free_memory(priv);
- pci_set_drvdata(pdev, NULL);
free_netdev(ndev);
priv = NULL;
do_pci_clear_mwi:
@@ -247,7 +246,6 @@ prism54_remove(struct pci_dev *pdev)
/* free the PCI memory and unmap the remapped page */
islpci_free_memory(priv);
- pci_set_drvdata(pdev, NULL);
free_netdev(ndev);
priv = NULL;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH 3/3] wireless: rt2x00: remove unnecessary pci_set_drvdata()
2013-11-25 1:49 [PATCH 1/3] wireless: ipw2100: remove unnecessary pci_set_drvdata() Jingoo Han
2013-11-25 1:51 ` [PATCH 2/3] wireless: prism54: " Jingoo Han
@ 2013-11-25 1:52 ` Jingoo Han
2013-11-25 14:15 ` Gertjan van Wingerde
2013-11-26 6:55 ` [PATCH 1/3] wireless: ipw2100: " Stanislav Yakovlev
2 siblings, 1 reply; 5+ messages in thread
From: Jingoo Han @ 2013-11-25 1:52 UTC (permalink / raw)
To: 'John W. Linville'
Cc: linux-wireless, 'Jingoo Han', 'Ivo van Doorn',
'Gertjan van Wingerde', 'Helmut Schaa'
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>
---
drivers/net/wireless/rt2x00/rt2x00pci.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/net/wireless/rt2x00/rt2x00pci.c b/drivers/net/wireless/rt2x00/rt2x00pci.c
index 25da20e..af72183 100644
--- a/drivers/net/wireless/rt2x00/rt2x00pci.c
+++ b/drivers/net/wireless/rt2x00/rt2x00pci.c
@@ -156,8 +156,6 @@ exit_release_regions:
exit_disable_device:
pci_disable_device(pci_dev);
- pci_set_drvdata(pci_dev, NULL);
-
return retval;
}
EXPORT_SYMBOL_GPL(rt2x00pci_probe);
@@ -177,7 +175,6 @@ void rt2x00pci_remove(struct pci_dev *pci_dev)
/*
* Free the PCI device data.
*/
- pci_set_drvdata(pci_dev, NULL);
pci_disable_device(pci_dev);
pci_release_regions(pci_dev);
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 3/3] wireless: rt2x00: remove unnecessary pci_set_drvdata()
2013-11-25 1:52 ` [PATCH 3/3] wireless: rt2x00: " Jingoo Han
@ 2013-11-25 14:15 ` Gertjan van Wingerde
0 siblings, 0 replies; 5+ messages in thread
From: Gertjan van Wingerde @ 2013-11-25 14:15 UTC (permalink / raw)
To: Jingoo Han
Cc: John W. Linville, linux-wireless@vger.kernel.org, Ivo van Doorn,
Helmut Schaa
On Mon, Nov 25, 2013 at 2:52 AM, Jingoo Han <jg1.han@samsung.com> wrote:
> 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>
Acked-by: Gertjan van Wingerde <gwingerde@gmail.com>
> ---
> drivers/net/wireless/rt2x00/rt2x00pci.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/net/wireless/rt2x00/rt2x00pci.c b/drivers/net/wireless/rt2x00/rt2x00pci.c
> index 25da20e..af72183 100644
> --- a/drivers/net/wireless/rt2x00/rt2x00pci.c
> +++ b/drivers/net/wireless/rt2x00/rt2x00pci.c
> @@ -156,8 +156,6 @@ exit_release_regions:
> exit_disable_device:
> pci_disable_device(pci_dev);
>
> - pci_set_drvdata(pci_dev, NULL);
> -
> return retval;
> }
> EXPORT_SYMBOL_GPL(rt2x00pci_probe);
> @@ -177,7 +175,6 @@ void rt2x00pci_remove(struct pci_dev *pci_dev)
> /*
> * Free the PCI device data.
> */
> - pci_set_drvdata(pci_dev, NULL);
> pci_disable_device(pci_dev);
> pci_release_regions(pci_dev);
> }
> --
> 1.7.10.4
>
>
--
---
Gertjan
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/3] wireless: ipw2100: remove unnecessary pci_set_drvdata()
2013-11-25 1:49 [PATCH 1/3] wireless: ipw2100: remove unnecessary pci_set_drvdata() Jingoo Han
2013-11-25 1:51 ` [PATCH 2/3] wireless: prism54: " Jingoo Han
2013-11-25 1:52 ` [PATCH 3/3] wireless: rt2x00: " Jingoo Han
@ 2013-11-26 6:55 ` Stanislav Yakovlev
2 siblings, 0 replies; 5+ messages in thread
From: Stanislav Yakovlev @ 2013-11-26 6:55 UTC (permalink / raw)
To: Jingoo Han; +Cc: John W. Linville, wireless
Hello Jingoo,
On 24 November 2013 17:49, Jingoo Han <jg1.han@samsung.com> wrote:
> 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>
> ---
Looks fine, thanks.
Stanislav.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-11-26 6:55 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-25 1:49 [PATCH 1/3] wireless: ipw2100: remove unnecessary pci_set_drvdata() Jingoo Han
2013-11-25 1:51 ` [PATCH 2/3] wireless: prism54: " Jingoo Han
2013-11-25 1:52 ` [PATCH 3/3] wireless: rt2x00: " Jingoo Han
2013-11-25 14:15 ` Gertjan van Wingerde
2013-11-26 6:55 ` [PATCH 1/3] wireless: ipw2100: " Stanislav Yakovlev
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox