* [PATCH] brcmfmac: clear sdiodev->pdata if irq_of_parse_and_map fails
@ 2015-05-20 14:02 Henrik Eriksson
2015-05-20 14:57 ` Henrik Eriksson
0 siblings, 1 reply; 2+ messages in thread
From: Henrik Eriksson @ 2015-05-20 14:02 UTC (permalink / raw)
To: brcm80211-dev-list, linux-wireless
Clear the pdata pointer in sdiodev after freeing the memory. In my case
not doing so interferes with device initialization as request_irq of
sdiodev->pdata->oob_irq_nr fails, but even if not, leaving a wild
pointer is incorrect.
Signed-off-by: Henrik Eriksson <henriken@axis.com>
---
drivers/net/wireless/brcm80211/brcmfmac/of.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/wireless/brcm80211/brcmfmac/of.c b/drivers/net/wireless/brcm80211/brcmfmac/of.c
index c824570..71ead7b 100644
--- a/drivers/net/wireless/brcm80211/brcmfmac/of.c
+++ b/drivers/net/wireless/brcm80211/brcmfmac/of.c
@@ -43,6 +43,7 @@ void brcmf_of_probe(struct brcmf_sdio_dev *sdiodev)
if (!irq) {
brcmf_err("interrupt could not be mapped\n");
devm_kfree(dev, sdiodev->pdata);
+ sdiodev->pdata = NULL;
return;
}
irqf = irqd_get_trigger_type(irq_get_irq_data(irq));
--
2.0.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] brcmfmac: clear sdiodev->pdata if irq_of_parse_and_map fails
2015-05-20 14:02 [PATCH] brcmfmac: clear sdiodev->pdata if irq_of_parse_and_map fails Henrik Eriksson
@ 2015-05-20 14:57 ` Henrik Eriksson
0 siblings, 0 replies; 2+ messages in thread
From: Henrik Eriksson @ 2015-05-20 14:57 UTC (permalink / raw)
To: brcm80211-dev-list, linux-wireless
Please ignore this. [PATCH 1/6] brcmfmac: allow device tree node
without 'interrupts' property posted earlier today deals with the same
thing, in a better way.
On Wed, May 20, 2015 at 16:02:15 +0200, Henrik Eriksson wrote:
> Clear the pdata pointer in sdiodev after freeing the memory. In my case
> not doing so interferes with device initialization as request_irq of
> sdiodev->pdata->oob_irq_nr fails, but even if not, leaving a wild
> pointer is incorrect.
>
> Signed-off-by: Henrik Eriksson <henriken@axis.com>
> ---
> drivers/net/wireless/brcm80211/brcmfmac/of.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/wireless/brcm80211/brcmfmac/of.c b/drivers/net/wireless/brcm80211/brcmfmac/of.c
> index c824570..71ead7b 100644
> --- a/drivers/net/wireless/brcm80211/brcmfmac/of.c
> +++ b/drivers/net/wireless/brcm80211/brcmfmac/of.c
> @@ -43,6 +43,7 @@ void brcmf_of_probe(struct brcmf_sdio_dev *sdiodev)
> if (!irq) {
> brcmf_err("interrupt could not be mapped\n");
> devm_kfree(dev, sdiodev->pdata);
> + sdiodev->pdata = NULL;
> return;
> }
> irqf = irqd_get_trigger_type(irq_get_irq_data(irq));
--
henrik eriksson
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-05-20 14:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-20 14:02 [PATCH] brcmfmac: clear sdiodev->pdata if irq_of_parse_and_map fails Henrik Eriksson
2015-05-20 14:57 ` Henrik Eriksson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).