brcmfmac: stop releasing sdio host in irq handler
authorFranky Lin <frankyl@broadcom.com>
Sat, 28 Apr 2012 01:56:56 +0000 (18:56 -0700)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 9 May 2012 01:53:53 +0000 (21:53 -0400)
brcmf_sdbrcm_isr doesn't access to the dongle through SDIO bus.
Stop releasing and claiming host in irq handler to eliminate
any potential risk.

Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c

index e925290b432bc104057be312664e322a18a66ac6..ea350200a7dbbc66b5694f9559c08320227bef22 100644 (file)
@@ -45,11 +45,7 @@ static void brcmf_sdioh_irqhandler(struct sdio_func *func)
 
        brcmf_dbg(TRACE, "***IRQHandler\n");
 
-       sdio_release_host(func);
-
        brcmf_sdbrcm_isr(sdiodev->bus);
-
-       sdio_claim_host(func);
 }
 
 /* dummy handler for SDIO function 2 interrupt */