On 04/10/07 16:58 +0100, Alan Cox wrote: > > For kicks I switched my driver over to using PCI config registers > > instead of MSRs. And it still worked! > > > > When I changed my driver I did a quickie conversion from wrmsr() to > > pci_write_config_dword(). If I use pci_write_config_byte() instead > > (like pata_amd.c), things fail miserably. A register dump reveals > > that any value I write becomes a 0. Yay! > > At that point its probaly a good idea to take CS5536 out of AMD and use > your driver for it, whether in PCI or MSR mode (or perhaps supporting > either) Agreed. It sounds like there is still breakage in the BIOS - since the reference VSA code accounts for accesses of different alignments, but its probably not worth going back and forth about it. Like I said, if Alan and Jeff are game, we should go with the 5536 driver, and now that we've debated it extensively, I'm fine with the MSR accesses - I'm satisfied that we at least considered the alternative and dismissed it for technically sound reasons. Is there anything we can do to make the transition smooth for anybody who may be using pata_amd in 2.6.23? Jordan -- Jordan Crouse Systems Software Development Engineer Advanced Micro Devices, Inc. - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
