previous version (it was checked into acpi-test on Jun-6) reverted, this version applied. thanks, -Len On Thursday 19 July 2007 22:03, Shaohua Li wrote: > applied after Rafel's 'PM: Update global suspend and hibernation operations framework' patch set > > Hook acpi d-state to pnp suspend/resume. > Signed-off-by: Shaohua Li <shaohua.li@xxxxxxxxx> > > Index: lgit/include/linux/pnp.h > =================================================================== > --- lgit.orig/include/linux/pnp.h 2007-07-19 09:38:25.000000000 +0800 > +++ lgit/include/linux/pnp.h 2007-07-19 14:24:30.000000000 +0800 > @@ -335,6 +335,10 @@ struct pnp_protocol { > int (*set)(struct pnp_dev *dev, struct pnp_resource_table *res); > int (*disable)(struct pnp_dev *dev); > > + /* protocol specific suspend/resume */ > + int (*suspend)(struct pnp_dev *dev, pm_message_t state); > + int (*resume)(struct pnp_dev *dev); > + > /* used by pnp layer only (look but don't touch) */ > unsigned char number; /* protocol number*/ > struct device dev; /* link to driver model */ > Index: lgit/drivers/pnp/driver.c > =================================================================== > --- lgit.orig/drivers/pnp/driver.c 2007-07-19 09:38:25.000000000 +0800 > +++ lgit/drivers/pnp/driver.c 2007-07-19 14:49:47.000000000 +0800 > @@ -167,6 +167,8 @@ static int pnp_bus_suspend(struct device > return error; > } > > + if (pnp_dev->protocol && pnp_dev->protocol->suspend) > + pnp_dev->protocol->suspend(pnp_dev, state); > return 0; > } > > @@ -179,6 +181,9 @@ static int pnp_bus_resume(struct device > if (!pnp_drv) > return 0; > > + if (pnp_dev->protocol && pnp_dev->protocol->resume) > + pnp_dev->protocol->resume(pnp_dev); > + > if (!(pnp_drv->flags & PNP_DRIVER_RES_DO_NOT_CHANGE)) { > error = pnp_start_dev(pnp_dev); > if (error) > Index: lgit/drivers/pnp/pnpacpi/core.c > =================================================================== > --- lgit.orig/drivers/pnp/pnpacpi/core.c 2007-07-19 09:38:25.000000000 +0800 > +++ lgit/drivers/pnp/pnpacpi/core.c 2007-07-19 14:50:13.000000000 +0800 > @@ -119,11 +119,25 @@ static int pnpacpi_disable_resources(str > return ACPI_FAILURE(status) ? -ENODEV : 0; > } > > +static int pnpacpi_suspend(struct pnp_dev *dev, pm_message_t state) > +{ > + return acpi_bus_set_power((acpi_handle)dev->data, > + acpi_pm_device_sleep_state(&dev->dev, > + device_may_wakeup(&dev->dev), NULL)); > +} > + > +static int pnpacpi_resume(struct pnp_dev *dev) > +{ > + return acpi_bus_set_power((acpi_handle)dev->data, ACPI_STATE_D0); > +} > + > static struct pnp_protocol pnpacpi_protocol = { > .name = "Plug and Play ACPI", > .get = pnpacpi_get_resources, > .set = pnpacpi_set_resources, > .disable = pnpacpi_disable_resources, > + .suspend = pnpacpi_suspend, > + .resume = pnpacpi_resume, > }; > > static int __init pnpacpi_add_device(struct acpi_device *device) > - 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