On Wed, 2010-12-01 at 13:15 -0600, Larry Finger wrote: > At one time, we thought that we had found BCM43xx devices with no SPROM. In the > one case that I remember, it was because the SPROM had been relocated. > > I now have the data from John's device that needs the revision fixup and I know > what is wrong - it is rev 2 with corrupted CRC. The defaulting to rev 1 is > getting almost everything wrong, including MAC address and vendor. My plan is to > write a better fixup routine. > > At the moment, we have some SPROM fallback code that has not been fully > implemented, and is probably not needed. Are there any objections to stripping > this code out of drivers/ssb/pci.c and drivers/ssb/sprom.c? Yes. The code is needed for bcm63xx embedded devices. The code that uses it currently is not in mainline, though. It can be found in the OpenWRT repositories. But I still think that the SPROM fallback mechanism should be replaced by a "platform data" based mechanism, or similar. Just removing it without replacement is not an option, because bcm63xx embedded really does not have an SPROM. The bcm63xx was the reason the fallback mechanism was implemented in the first place. See git logs for more details. -- Greetings Michael. -- 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
