On 12/20/2009 03:53 PM, Luis R. Rodriguez wrote: >> I should note 0x40 starts with vendor specific PCI config space so you >> cannot guarantee different PCI devices use 0x41 will be used the same >> for different devices. The documentation for the ath9k PCI-E devices >> used that entry for something completely different but what I did not >> do is try to very and ensure PCI devices do not use it it for the >> same. I am told though that although this is PCI vendor space some >> devices may still use similar private PCI config spaces on different >> devices which just follows a practice. At this point we now have not >> only b43, ipw and ath9k follow this but also prism54 and I think p54 >> uses this. I'll note I *highly* doubt this is used for the same thing >> on all these devices and was just code copied from other Linux >> drivers. In the case of Atheros Linux drivers I know it was copied >> form Intel drivers, which is why I started questioning it all. >> >> Anyway, if it helps, that's great :) but it cannot be concluded its >> all for the same thing unless you have proper documentation as this is >> in PCI vendor space which *can* vary depending on device and vendor. > > Larry, does this actually fix something or is this code purely speculative? I got on this idea because I logged all PCI configuration read/write values and discovered that the Broadcom wl driver did a read/write on this location. A later examination of the open-source part of that driver showed that they copied it from ipw2100. If anyone knows the layout of the private area of the configuration space, they should; however, they may be just propagating the code as you say. This change does not fix the DMA error problem with Atom processors in Netbooks. We have had one report of DMA errors in a Core Duo system - AFAIK, the only non-Atom processor with the problem. That user has not tested this patch. Larry -- 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
