On Thursday, 26 of June 2008, Jesse Barnes wrote: > On Monday, June 23, 2008 10:34 am Rafael J. Wysocki wrote: > > Hi, > > > > The purpose of the following series of patches is to rework the power > > management of PCI devices so that the handling of their wake-up > > functionality is consistent with the other PM operations and so that the > > user space can manage the wake-up functionality of PCI devices using the > > /sys/devices/.../power/wakeup interface regardless of whether the wake-up > > function is based on the native PCI mechanism (PME#), or it is handled by > > the platform (eg. ACPI). > > > > Apart from the ACPI and PCI changes necessary for this purpose, the series > > includes some clean-ups that are not strictly required, but make the code > > more straightforward and (IMHO) easier to follow. > > > > Please review. > > This patchset looks really nice, and makes pci_enable_wake and such much more > readable, IMO. Thanks! > How much testing has it seen? A little. I've been using it on my HP nx6325 for some days with no visible problems. > Do you think it's appropriate for 2.6.27? Well, I think the patches need to be modified to handle the case of devices that can wake up both through PME# and the ACPI-based mechanism. Also, I need to make ACPI enable the wake-up GPEs for PCI devices that are supposed to be handled by it. For this reasons I'm not sure they will be sufficiently tested before the 2.6.27 merge window. Perhaps it's better to target the patchset at 2.6.28 and make sure it gets some serious testing during the .27 cycle. Thanks, Rafael -- 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
