Message ID | 31830983.Z4sGoD3BOE@aspire.rjw.lan (mailing list archive) |
---|---|
State | Mainlined |
Delegated to: | Rafael Wysocki |
Headers | show |
Hi Rafael, On 14/05/2017 at 01:55:32 +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > 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@linux.intel.com> > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > --- > 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?
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@intel.com> > > > > 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@linux.intel.com> > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > --- > > 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.
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;