> -----Original Message----- > From: Andy Shevchenko [mailto:[email protected]] > Sent: Tuesday, August 1, 2017 7:37 AM > To: Rafael J. Wysocki <[email protected]> > Cc: Rafael J. Wysocki <[email protected]>; Platform Driver <platform-driver- > [email protected]>; Darren Hart <[email protected]>; LKML <linux- > [email protected]>; Linux ACPI <[email protected]>; Andy > Shevchenko <[email protected]>; Jérôme de Bretagne > <[email protected]>; Limonciello, Mario > <[email protected]>; Alex Hung <[email protected]> > Subject: Re: [PATCH] platform/x86: intel-hid: Wake up Dell Latitude 7275 from > suspend-to-idle > > On Tue, Aug 1, 2017 at 2:56 PM, Rafael J. Wysocki <[email protected]> wrote: > > On Tue, Aug 1, 2017 at 1:21 AM, Andy Shevchenko > > <[email protected]> wrote: > >> On Tue, Aug 1, 2017 at 12:46 AM, Rafael J. Wysocki <[email protected]> > >> wrote: > >>> On Friday, July 28, 2017 02:06:36 AM Rafael J. Wysocki wrote: > > >>> Please note that this change is requisite for > >>> > >>> https://patchwork.kernel.org/patch/9873159/ > >>> > >>> so are there any objections or concerns? > >> > >> Not from my side, > >> > >> Acked-by: Andy Shevchenko <[email protected]> > > > > OK, thanks! > > > > I'm going to route it through the PM tree then if that's not a problem. > > Mario, are you okay with this change? >
Thanks for checking. I spent a little time this morning trying to walk through the ASL as attached to the Bugzilla entry and I think this is the correct approach. Acked-By: Mario Limonciello <[email protected]> Jérôme, I have one question though. These events should be happening as a pair. Press: 0xCE, Release: 0xCF. What happens with the event on power button release? Is that showing a message in the log during wakeup from S2I? Something like "unknown event 0xCF"? If so, it would be good to also catch and ignore that too. Thanks,
