On 02/14/2012 06:29 PM, Rafał Miłecki wrote:
> W dniu 14 lutego 2012 17:22 użytkownik Saul St. John
> <saul.stjohn@xxxxxxxxx> napisał:
>> On Tue, Feb 14, 2012 at 7:34 AM, Rafał Miłecki <zajec5@xxxxxxxxx> wrote:
>>> W dniu 14 lutego 2012 05:01 użytkownik Saul St. John
>>> <saul.stjohn@xxxxxxxxx> napisał:
>>>> I don't know if this is correct in the general sense, but the wireless on my
>>>> mid-2010 MacBook Pro doesn't work without it.
>>>>
>>>> Signed-off-by: Saul St. John <saul.stjohn@xxxxxxxxx>
>>>> ---
>>>> drivers/bcma/sprom.c | 4 ++--
>>>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/drivers/bcma/sprom.c b/drivers/bcma/sprom.c
>>>> index 6f230fb..06c87b5 100644
>>>> --- a/drivers/bcma/sprom.c
>>>> +++ b/drivers/bcma/sprom.c
>>>> @@ -228,8 +228,8 @@ int bcma_sprom_get(struct bcma_bus *bus)
>>>> /* Most cards have SPROM moved by additional offset 0x30 (48 dwords).
>>>> * According to brcm80211 this applies to cards with PCIe rev >= 6
>>>> * TODO: understand this condition and use it */
>>>> - offset = (bus->chipinfo.id == 0x4331) ? BCMA_CC_SPROM :
>>>> - BCMA_CC_SPROM_PCIE6;
>>>> + offset = (bus->chipinfo.id == 0x4331 || bus->chipinfo.id == 43224) ?
>>>> + BCMA_CC_SPROM : BCMA_CC_SPROM_PCIE6;
>>>> bcma_sprom_read(bus, offset, sprom);
>>>>
>>>> if (bus->chipinfo.id == 0x4331)
>>>
>>> I'm quite sure it'll break my BCM43224. It's not chip-specific,
>>> probably some status bit specific.
>>>
>>> --
>>> Rafał
>>
>> My BCM43324 was broken by bmca up until "[PATCH] bcma: don't fail for
>> bad SPROM CRC." Even with that patch, I still get "bmca: Failed to get
>> SPROM: -71" in the dmesg log. Is that error harmless?
>
> It's harmless for brcmsmac, which doesn't use SPROM struct of bcma bus
> driver. This bug should be fixed and brcmsmac should be improved in
> many contexts: using SPROM, standard bcma module functions, dropping
> other cores initializing. For now you can live with this.
>
>
>> The CRC check appears to pass without issue when using the 0x800
>> offset on my device.
>
> The quick fix would be probably to implement in bcma two tries of
> reading SPROM. One for 0x800 second for 0x830 (base address). The real
> fix is to grab the real condition from specs/brcmsmac and implement it
> in bcma.
>
Hi,
I am unable to find the 0x830 address in brcmsmac or any other source
code from Broadcom.
This is the place where brcmsmac gets the offset:
/* determine core to read */
if (ai_get_ccrev(sih) < 32) {
core = ai_findcore(sih, BCMA_CORE_80211, 0);
sprom_offset = PCI_BAR0_SPROM_OFFSET; // (4 * 1024)
} else {
core = ai_findcore(sih, BCMA_CORE_CHIPCOMMON, 0);
sprom_offset = CHIPCREGOFFS(sromotp); // 0x800
}
What chipcommon rev does the device have?
This device is probably using the otp instead of a sprom and that is not
implemented in bcma.
The thing with the pcie core rev >= 6, is about sprom rev >= 4 being
bigger than the old ones which is fully implemented in bcma.
Hauke
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
