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? The CRC check appears to pass without issue when using the 0x800 offset on my device. -saul -- 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