aboutsummaryrefslogtreecommitdiffstats
path: root/package/kernel/mac80211/patches/351-0004-brcmfmac-Fix-did-not-remove-int-handler-warning.patch
diff options
context:
space:
mode:
Diffstat (limited to 'package/kernel/mac80211/patches/351-0004-brcmfmac-Fix-did-not-remove-int-handler-warning.patch')
-rw-r--r--package/kernel/mac80211/patches/351-0004-brcmfmac-Fix-did-not-remove-int-handler-warning.patch83
1 files changed, 83 insertions, 0 deletions
diff --git a/package/kernel/mac80211/patches/351-0004-brcmfmac-Fix-did-not-remove-int-handler-warning.patch b/package/kernel/mac80211/patches/351-0004-brcmfmac-Fix-did-not-remove-int-handler-warning.patch
new file mode 100644
index 0000000000..3aabf590bb
--- /dev/null
+++ b/package/kernel/mac80211/patches/351-0004-brcmfmac-Fix-did-not-remove-int-handler-warning.patch
@@ -0,0 +1,83 @@
+From: Christian Daudt <csd@broadcom.com>
+Date: Wed, 11 May 2016 15:06:49 -0700
+Subject: [PATCH] brcmfmac: Fix 'did not remove int handler' warning
+
+brcmf_sdiod_intr_unregister call that removes both func1 and
+func2 interrupt handlers only called when brcmf_ops_sdio_remove
+is called for func 1 (which is the 2nd call) but sdio is expecting
+it to be removed at the end of each sdio_remove call.
+This is causing 'rmmod bcmrfmac' on a 4356-sdio chip to complain
+with:
+WARNING: driver brcmfmac did not remove its interrupt handler!
+
+The modification makes calling brcmf_sdiod_intr_unregister multiple
+times harmless by clearing the variables that track if interrupt
+handlers have been installed, and then calls it on every
+brcmf_ops_sdio_remove call instead of just remove for func 1.
+
+Signed-off-by: Christian Daudt <csd@broadcom.com>
+Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
+---
+
+--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c
++++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c
+@@ -172,7 +172,7 @@ int brcmf_sdiod_intr_register(struct brc
+ return 0;
+ }
+
+-int brcmf_sdiod_intr_unregister(struct brcmf_sdio_dev *sdiodev)
++void brcmf_sdiod_intr_unregister(struct brcmf_sdio_dev *sdiodev)
+ {
+
+ brcmf_dbg(SDIO, "Entering oob=%d sd=%d\n",
+@@ -195,6 +195,7 @@ int brcmf_sdiod_intr_unregister(struct b
+ }
+ free_irq(pdata->oob_irq_nr, &sdiodev->func[1]->dev);
+ sdiodev->irq_en = false;
++ sdiodev->oob_irq_requested = false;
+ }
+
+ if (sdiodev->sd_irq_requested) {
+@@ -202,9 +203,8 @@ int brcmf_sdiod_intr_unregister(struct b
+ sdio_release_irq(sdiodev->func[2]);
+ sdio_release_irq(sdiodev->func[1]);
+ sdio_release_host(sdiodev->func[1]);
++ sdiodev->sd_irq_requested = false;
+ }
+-
+- return 0;
+ }
+
+ void brcmf_sdiod_change_state(struct brcmf_sdio_dev *sdiodev,
+@@ -1200,12 +1200,17 @@ static void brcmf_ops_sdio_remove(struct
+ brcmf_dbg(SDIO, "sdio device ID: 0x%04x\n", func->device);
+ brcmf_dbg(SDIO, "Function: %d\n", func->num);
+
+- if (func->num != 1)
+- return;
+-
+ bus_if = dev_get_drvdata(&func->dev);
+ if (bus_if) {
+ sdiodev = bus_if->bus_priv.sdio;
++
++ /* start by unregistering irqs */
++ brcmf_sdiod_intr_unregister(sdiodev);
++
++ if (func->num != 1)
++ return;
++
++ /* only proceed with rest of cleanup if func 1 */
+ brcmf_sdiod_remove(sdiodev);
+
+ dev_set_drvdata(&sdiodev->func[1]->dev, NULL);
+--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.h
++++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.h
+@@ -294,7 +294,7 @@ struct sdpcmd_regs {
+
+ /* Register/deregister interrupt handler. */
+ int brcmf_sdiod_intr_register(struct brcmf_sdio_dev *sdiodev);
+-int brcmf_sdiod_intr_unregister(struct brcmf_sdio_dev *sdiodev);
++void brcmf_sdiod_intr_unregister(struct brcmf_sdio_dev *sdiodev);
+
+ /* sdio device register access interface */
+ u8 brcmf_sdiod_regrb(struct brcmf_sdio_dev *sdiodev, u32 addr, int *ret);