From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-gw1-out.broadcom.com ([216.31.210.62]:53640 "EHLO mail-gw1-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755797AbaFQLt6 (ORCPT ); Tue, 17 Jun 2014 07:49:58 -0400 Message-ID: <53A02B5A.3070409@broadcom.com> (sfid-20140617_135003_362997_26D6EA0B) Date: Tue, 17 Jun 2014 13:49:46 +0200 From: Arend van Spriel MIME-Version: 1.0 To: Hans de Goede , "John W. Linville" CC: Chen-Yu Tsai , , , devicetree , Subject: Re: [PATCH v2 4/4] brcmfmac: Fix OOB interrupt not working for BCM43362 References: <1402941407-8210-1-git-send-email-hdegoede@redhat.com> <1402941407-8210-5-git-send-email-hdegoede@redhat.com> In-Reply-To: <1402941407-8210-5-git-send-email-hdegoede@redhat.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 16-06-14 19:56, Hans de Goede wrote: > It has taken me a long long time to get the OOB interrupt working on the > AP6210 sdio wifi/bt module found on various Allwinner A20 boards. In the > end I found these magic register pokes in the cubietruck kernel tree: > https://github.com/cubieboard2/linux-sunxi/commit/7f08ba395617d17e7a711507503d89a50406fe7a > > I'm not entirely sure if this specific to the AP6210 module, or if this > should be done for all BCM43362 sdio devices. Hi Hans, I was still not sure about this one so I asked around. Apart from the nvram settings I emailed earlier one more is needed: sd_gpout=1 sd_oobonly=1 sd_gpval=1 *muxenab=0x11* Could you give that a try? Regards, Arend > Signed-off-by: Hans de Goede > --- > drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c b/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c > index 0fc707c..58e86a5 100644 > --- a/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c > +++ b/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c > @@ -39,7 +39,9 @@ > #include > #include > #include > +#include > #include > +#include "chip.h" > #include "dhd_bus.h" > #include "dhd_dbg.h" > #include "sdio_host.h" > @@ -119,6 +121,7 @@ int brcmf_sdiod_intr_register(struct brcmf_sdio_dev *sdiodev) > { > int ret = 0; > u8 data; > + u32 addr, gpiocontrol; > unsigned long flags; > > if ((sdiodev->pdata) && (sdiodev->pdata->oob_irq_supported)) { > @@ -148,6 +151,19 @@ int brcmf_sdiod_intr_register(struct brcmf_sdio_dev *sdiodev) > > sdio_claim_host(sdiodev->func[1]); > > + if (sdiodev->bus_if->chip == BCM43362_CHIP_ID) { > + /* assign GPIO to SDIO core */ > + addr = CORE_CC_REG(SI_ENUM_BASE, gpiocontrol); > + gpiocontrol = brcmf_sdiod_regrl(sdiodev, addr, &ret); > + gpiocontrol |= 0x2; > + brcmf_sdiod_regwl(sdiodev, addr, gpiocontrol, &ret); > + > + brcmf_sdiod_regwb(sdiodev, SBSDIO_GPIO_SELECT, 0xf, > + &ret); > + brcmf_sdiod_regwb(sdiodev, SBSDIO_GPIO_OUT, 0, &ret); > + brcmf_sdiod_regwb(sdiodev, SBSDIO_GPIO_EN, 0x2, &ret); > + } > + > /* must configure SDIO_CCCR_IENx to enable irq */ > data = brcmf_sdiod_regrb(sdiodev, SDIO_CCCR_IENx, &ret); > data |= 1 << SDIO_FUNC_1 | 1 << SDIO_FUNC_2 | 1; >