Hi Greg ! I was looking at our "generic" pci_save_state() and pci_restore_state() and I have various concerns with them, I was wondering what you though about them... - We should always write the command register after all the BARs, typically that mean write it back _last_ - We shouldn't write to the BIST register, it is defined as having side effects and writing to it any value may trigger a BIST on the card, with all the possible bad consequences that has - What about saving/restoring more registers ? I'm not sure wether it should be the responsibility of the driver to save and restore things above dword 15, but we should at least deal with the case of P2P bridges who have more "standard" registers In addition, we currently have no mecanism to save/restore the state of P2P bridges. Shouldn't we do that in pci_device_suspend() if there is no driver attached ? Ben.
