Forgot to reply about some stuff.  Adding.

Tejun Heo wrote:
> Alan wrote:
>>> This patch makes amd74xx not configure udma mode higher than BIOS did.
>>> If BIOS configured the device <= udma44, udma33 is the maximum speed.
>> NAK
>>
>> The boot firmware for AMD/Nvidia chips is only run on PC, and the data is
>> only valid on some of them, in some cases, and not after a suspend/resume
>> cycle.

To prevent that, the value is cached on driver probe and written back on
driver detach on pata_amd, which is necessary even when there is no
suspend/resume as mode programming alters the content of the register.
amd74xx cannot be unloaded, so caching is enough.  Also, doesn't
suspend/resume cycle store and restore PCI space anyway?

Thanks.

-- 
tejun
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to