[RFT][PATCH] b43: fix logic in GPIO configuration
Rafał Miłecki
zajec5 at gmail.com
Fri Mar 9 23:50:27 EST 2012
W dniu 10 marca 2012 00:17 użytkownik Hauke Mehrtens <hauke at hauke-m.de> napisał:
> On 03/07/2012 08:23 AM, Rafał Miłecki wrote:
>> W dniu 7 marca 2012 07:52 użytkownik Rafał Miłecki <zajec5 at gmail.com> napisał:
>>> W dniu 6 marca 2012 23:55 użytkownik Hauke Mehrtens <hauke at hauke-m.de> napisał:
>>>> On 03/06/2012 11:11 PM, Rafał Miłecki wrote:
>>>>> By using reverted mask we were taking over pins we were not supporsed to
>>>>> touch. After fixing this workaround for BCM5354 should not be needed
>>>>> anymore.
>>>>>
>>>>> Signed-off-by: Rafał Miłecki <zajec5 at gmail.com>
>>>>> ---
>>>>> drivers/net/wireless/b43/main.c | 6 ++----
>>>>> 1 files changed, 2 insertions(+), 4 deletions(-)
>>>>>
>>>>> diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
>>>>> index 1d633f3..8a89885 100644
>>>>> --- a/drivers/net/wireless/b43/main.c
>>>>> +++ b/drivers/net/wireless/b43/main.c
>>>>> @@ -2706,8 +2706,6 @@ static int b43_gpio_init(struct b43_wldev *dev)
>>>>> mask |= 0x0060;
>>>>> set |= 0x0060;
>>>>> }
>>>>> - if (dev->dev->chip_id == 0x5354)
>>>>> - set &= 0xff02;
>>>>> if (0 /* FIXME: conditional unknown */ ) {
>>>>> b43_write16(dev, B43_MMIO_GPIO_MASK,
>>>>> b43_read16(dev, B43_MMIO_GPIO_MASK)
>>>>> @@ -2730,7 +2728,7 @@ static int b43_gpio_init(struct b43_wldev *dev)
>>>>> case B43_BUS_BCMA:
>>>>> bcma_cc_write32(&dev->dev->bdev->bus->drv_cc, BCMA_CC_GPIOCTL,
>>>>> (bcma_cc_read32(&dev->dev->bdev->bus->drv_cc,
>>>>> - BCMA_CC_GPIOCTL) & mask) | set);
>>>>> + BCMA_CC_GPIOCTL) & ~mask) | set);
>>>>> break;
>>>>> #endif
>>>>> #ifdef CONFIG_B43_SSB
>>>>> @@ -2739,7 +2737,7 @@ static int b43_gpio_init(struct b43_wldev *dev)
>>>>> if (gpiodev)
>>>>> ssb_write32(gpiodev, B43_GPIO_CONTROL,
>>>>> (ssb_read32(gpiodev, B43_GPIO_CONTROL)
>>>>> - & mask) | set);
>>>>> + & ~mask) | set);
>>>>> break;
>>>>> #endif
>>>>> }
>>>>
>>>> Hi,
>>>>
>>>> This patch did not helped fixing my problem. The ucode still takes over
>>>> the GPIOs of the buttons with firmware version 666.2. I am getting the
>>>> following messages:
>>>>
>>>> [ 83.620000] b43-phy0: Loading firmware version 666.2 (2011-02-23
>>>> 01:15:07)
>>>> [ 83.624000] Init value of B43_GPIO_CONTROL: 0x0
>>>> [ 83.632000] b43-phy0 debug: b2062: Using crystal tab entry 19200 kHz.
>>>> [ 83.640000] hotplug_button: action: pressed, name: reset, seen: 17179652
>>>> [ 83.644000] hotplug_button: action: pressed, name: ses, seen: 17179652
>>>> [ 89.404000] b43-phy0 debug: Chip initialized
>>>
>>> Interesting/weird. I suspected there are some bits set in
>>> B43_GPIO_CONTROL we should not clean. And you just adjusted "set" to
>>> keep them set. But now it seems B43_GPIO_CONTROL is *zero* at the
>>> beginning and we actually have to *set* 0xFF00 to stop firmware
>>> touching them? Really tricky, I didn't suspect we may need to change
>>> default state of non-b43-related GPIOs.
>>
>> Ahh, sorry. For all the time I believed the code you added was:
>> set |= 0xff02;
>>
>> However the code you *really* added is:
>> set &= 0xff02;
>>
>> Sorry for messing. OK, let's short analyze that. By default we set "set" to:
>> set = 0x0000000F;
>> so you code basically makes it
>> set = 0x00000002;
>>
>> 0x2 (BIT 1) is wlan led on your device. So you let firmware use wlan
>> led, but you stop it from touching power led (0x1), reset button (0x4)
>> and ses button (0x8). OK, it makes sense.
> Yes
> It used the GPIO pins with the bit numbers set to 1, at least for the
> first 4 ones. I just made it just use the GPIO for the wlan LED. Should
> I still test some patches on the device?
>>
>> In the brcm80211 code there was unused si_gpioreserve function. I
>> guess it used to be called in some way like:
>> if (dev->dev->chip_id == 0x5354)
>> si_gpioreserve(sih, 0x2, 1+)
>
> Where did you find this code? I was unable to find any code adding some
> special gpio handling for the bcm5354 in my sources.
As I said, I've found only si_gpioreserve function. I didn't find
special handling of BCM5354, I only guess something like that was used
internally at Broadcom in their wl. Nothing like that was released
with brcm80211.
If you wish to see si_gpioreserve function, take a look at
commit a9533e7ea3c410fed2f4cd8b3e1e213e48529b75
Author: Henry Ptasinski <henryp at broadcom.com>
Date: Wed Sep 8 21:04:42 2010 -0700
Staging: Add initial release of brcm80211 - Broadcom 802.11n
wireless LAN driver.
--
Rafał
More information about the b43-dev
mailing list