Re: [PATCH] platform/x86: intel-hid: Missing power button release on some Dell models

[Darren Hart]

On Mon, Jan 07, 2019 at 03:36:48PM +0000, Mario.Limonciello@xxxxxxxx wrote:
> 
> 
> > -----Original Message-----
> > From: platform-driver-x86-owner@xxxxxxxxxxxxxxx <platform-driver-x86-
> > owner@xxxxxxxxxxxxxxx> On Behalf Of Jérôme de Bretagne
> > Sent: Sunday, January 6, 2019 11:57 AM
> > To: Alex Hung
> > Cc: platform-driver-x86@xxxxxxxxxxxxxxx; Andy Shevchenko; Darren Hart;
> > Limonciello, Mario; Rafael J. Wysocki; Chih-Wei Huang; Tristian Celestin; linux-
> > kernel@xxxxxxxxxxxxxxx
> > Subject: [PATCH] platform/x86: intel-hid: Missing power button release on some
> > Dell models
> > 
> > 
> > [EXTERNAL EMAIL]
> > 
> > Power button suspend for some Dell models was added in:
> > 
> > commit 821b85366284 (intel-hid: Power button suspend on Dell Latitude 7275)

I've addressed this one, but please do run checkpatch and correct reported
errors in the future. It checks commit reference format.

> > 
> > by checking against the power button press notification (0xCE) to report
> > the power button press event. The corresponding power button release
> > notification (0xCF) was caught and ignored to stop it from being reported
> > as an "unknown event" in the logs.
> > 
> > The missing button release event is creating issues on Android-x86, as
> > reported on the project mailing list for a Dell Latitude 5175 model, since
> > the events are expected in down/up pairs.
> > 
> > Report the power button release event to fix this issue.
> > 
> > Link: https://groups.google.com/forum/#!topic/android-x86/aSwZK9Nf9Ro
> > Tested-by: Tristian Celestin <tristian.celestin@xxxxxxxxxxx>
> > Tested-by: Jérôme de Bretagne <jerome.debretagne@xxxxxxxxx>
> > Signed-off-by: Jérôme de Bretagne <jerome.debretagne@xxxxxxxxx>

Queued for testing, thanks for the patch.

-- 
Darren Hart
VMware Open Source Technology Center