On Wed, Sep 26, 2012 at 7:59 AM, Huang Ying <ying.huang@xxxxxxxxx> wrote: > Sorry for bothering. It appears that this patch should go through ACPI > tree because code changed are under drivers/acpi. > > A set of power resources may be shared by multiple devices. When all > devices share one set of power resources is put into D3_COLD state, > the power resources will be turned off. When one of the device is > waked, the power resource will be turned on and all devices share it > will be powered on to D0uninitialized state. These devices should be > resumed, so that they can get opportunity to go to low power state > later. > > Signed-off-by: Huang Ying <ying.huang@xxxxxxxxx> > Acked-by: Rafael J. Wysocki <rjw@xxxxxxx> > Cc: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> I had applied this to my for-linus branch, headed for v3.7, but had to drop it because it introduced these warnings: drivers/acpi/pci_bind.c: In function 'acpi_pci_unbind': drivers/acpi/pci_bind.c:48:2: warning: passing argument 1 of 'acpi_power_resource_unregister_device' from incompatible pointer type [enabled by default] include/acpi/acpi_bus.h:346:6: note: expected 'struct device *' but argument is of type 'struct pci_dev *' drivers/acpi/pci_bind.c: In function 'acpi_pci_bind': drivers/acpi/pci_bind.c:75:2: warning: passing argument 1 of 'acpi_power_resource_register_device' from incompatible pointer type [enabled by default] include/acpi/acpi_bus.h:345:5: note: expected 'struct device *' but argument is of type 'struct pci_dev *' Apparently this wasn't tested :-( > --- > drivers/acpi/pci_bind.c | 2 ++ > 1 file changed, 2 insertions(+) > > --- a/drivers/acpi/pci_bind.c > +++ b/drivers/acpi/pci_bind.c > @@ -45,6 +45,7 @@ static int acpi_pci_unbind(struct acpi_d > > device_set_run_wake(&dev->dev, false); > pci_acpi_remove_pm_notifier(device); > + acpi_power_resource_unregister_device(dev, device->handle); > > if (!dev->subordinate) > goto out; > @@ -71,6 +72,7 @@ static int acpi_pci_bind(struct acpi_dev > return 0; > > pci_acpi_add_pm_notifier(device, dev); > + acpi_power_resource_register_device(dev, device->handle); > if (device->wakeup.flags.run_wake) > device_set_run_wake(&dev->dev, true); > -- 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
