On 31/05/2017 at 11:21:55 +0200, Alexandre Belloni wrote: > Hi Rafael, > > On 14/05/2017 at 01:55:32 +0200, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > > > > Commit eed4d47efe95 (ACPI / sleep: Ignore spurious SCI wakeups from > > suspend-to-idle) modified the core suspend-to-idle code to filter > > out spurious SCI interrupts received while suspended, which requires > > ACPI event source handlers to report wakeup events in a way that > > will trigger a wakeup from suspend to idle (or abort system suspends > > in progress, which is equivalent). > > > > That needs to be done in the rtc-cmos driver too, which was overlooked > > by the above commit, so do that now. > > > > Fixes: eed4d47efe95 (ACPI / sleep: Ignore spurious SCI wakeups from suspend-to-idle) > > Reported-by: David E. Box <david.e.box@xxxxxxxxxxxxxxx> > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > > --- > > drivers/rtc/rtc-cmos.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > Index: linux-pm/drivers/rtc/rtc-cmos.c > > =================================================================== > > --- linux-pm.orig/drivers/rtc/rtc-cmos.c > > +++ linux-pm/drivers/rtc/rtc-cmos.c > > @@ -1085,7 +1085,7 @@ static u32 rtc_handler(void *context) > > } > > spin_unlock_irqrestore(&rtc_lock, flags); > > > > - pm_wakeup_event(dev, 0); > > + pm_wakeup_hard_event(dev); > > acpi_clear_event(ACPI_EVENT_RTC); > > acpi_disable_event(ACPI_EVENT_RTC, 0); > > return ACPI_INTERRUPT_HANDLED; > > > > This seems good to me, do you expect it to go through my tree? > OK, I see it has already been pulled by Linus. -- Alexandre Belloni, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- 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
