On Wed, Jan 11, 2012 at 9:00 AM, Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote: > --- wireless-testing-new.orig/drivers/bcma/sprom.c > +++ wireless-testing-new/drivers/bcma/sprom.c > @@ -230,6 +230,7 @@ int bcma_sprom_get(struct bcma_bus *bus) > * TODO: understand this condition and use it */ > offset = (bus->chipinfo.id == 0x4331) ? BCMA_CC_SPROM : > BCMA_CC_SPROM_PCIE6; > + pr_debug("SPROM offset 0x%x\n", offset); > bcma_sprom_read(bus, offset, sprom); > > if (bus->chipinfo.id == 0x4331) So I did that - the offset is printed out as 0x0830 (BCMA_CC_SPROM_PCIE6). And when I change the offset to 0x0800 (BCMA_CC_SPROM), the complaint about SPROM version goes away. IOW, these messages no longer exist: [ 0.815750] bcma: Unsupported SPROM revision: 255 [ 0.815804] bcma: No SPROM available but that doesn't actually make anything *work*. The brcms errors still remain the same. So the SPROM issue seems to be real, but irrelevant. I also noticed that the CONFIG_BRCMDBG onyl enables pr_debug(), but doesn't actually #define DEBUG, so pr_debug() gets compiled out anyway. I'll try with -DDEBUG in the brcm80211 subdirectory and see if I get more interesting output that way. Linus -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html