On Thursday 31 May 2007 22:37, Shaohua Li wrote: I like this patch, it may save power while in S3... I gather that it should also automatically expose the D-state controls for these PNP devices in sysfs, which may come in very handy... > +static int pnpacpi_suspend(struct pnp_dev *dev, pm_message_t state) > +{ > + return acpi_bus_set_power((acpi_handle)dev->data, 3); > +} > + This should usually work, but... If the device is enabled as a wakeup device, this could disable its wakeup capability. So in that case, we need to check _S3W to make sure that we are not putting the device in a D-state deeper than it can still wake the system. -Len - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html