Re: device_suspend() levels [was Re: [patch] ACPI work on aic7xxx]

Nathan Bryant
2004-07-20T18:34:29Z

Benjamin Herrenschmidt wrote:

> 2 comments here:
>
> - The low level bus state (PCI D state for example) and the "linux"
> state should be 2 different entities.
>
> - For PCI, we probably want a hook so the arch can implement it's own
> version of pci_set_power_state() so that ACPI can use it's own trickery
> there.

Ok, so the takeaway message for driver writers is to treat the
pci_dev->suspend() state parameter as an opaque value as far as
possible, and just pass it on to the other layers