RE: [PATCH 3/4] dell-wmi: Add information about other WMI event codes

Mario_Limonciello
2016-06-21T19:51:17Z

> -----Original Message-----
> From: Pali Rohár [mailto:pali.rohar@gmail.com]
> Sent: Wednesday, June 15, 2016 2:51 PM
> To: Limonciello, Mario <Mario_Limonciello@Dell.com>
> Cc: Gabriele Mazzotta <gabriele.mzt@gmail.com>; mjg59@srcf.ucam.org;
> dvhart@infradead.org; kernel@kempniu.pl; luto@kernel.org;
> alex.hung@canonical.com; platform-driver-x86@vger.kernel.org; linux-
> kernel@vger.kernel.org
> Subject: Re: [PATCH 3/4] dell-wmi: Add information about other WMI event
> codes
>
> On Wednesday 08 June 2016 12:44:44 Gabriele Mazzotta wrote:
> > On 08/06/2016 08:02, Mario_Limonciello@Dell.com wrote:
> > >> -----Original Message-----
> > >> From: Pali Rohár [mailto:pali.rohar@gmail.com]
> > >> Sent: Tuesday, June 7, 2016 6:00 PM
> > >> To: Gabriele Mazzotta <gabriele.mzt@gmail.com>; Limonciello, Mario
> > >> <Mario_Limonciello@Dell.com>
> > >> Cc: Matthew Garrett <mjg59@srcf.ucam.org>; Darren Hart
> > >> <dvhart@infradead.org>; Michał Kępień <kernel@kempniu.pl>; Andy
> > >> Lutomirski <luto@kernel.org>; Alex Hung
> > >> <alex.hung@canonical.com>; platform-driver- x86@vger.kernel.org;
> > >> linux-kernel@vger.kernel.org
> > >> Subject: Re: [PATCH 3/4] dell-wmi: Add information about other WMI
> > >> event codes
> > >>
> > >> On Friday 27 May 2016 00:04:23 Gabriele Mazzotta wrote:
> > >>> On 22/05/2016 13:36, Pali Rohár wrote:
> > >>>> ACPI DSDT tables have defined other WMI codes, but does not
> > >>>> contain any description when those codes are emitted. Some
> > >>>> other codes can be found in logs on internet. In this patch are
> > >>>> all which I saw, but lot of them are not tested properly (e.g.
> > >>>> for duplicate events with AT keyboard). Now we have all WMI
> > >>>> event codes at one place and in future after proper testing
> > >>>> those codes can be correctly enabled or
> > >>
> > >> disabled...
> > >>
> > >>>> Signed-off-by: Pali Rohár <pali.rohar@gmail.com>
> > >>>> ---
> > >>>>
> > >>>> drivers/platform/x86/dell-wmi.c | 32
> > >>
> > >> ++++++++++++++++++++++++++++++++
> > >>
> > >>>> 1 file changed, 32 insertions(+)
> > >>>>
> > >>>> diff --git a/drivers/platform/x86/dell-wmi.c
> > >>>> b/drivers/platform/x86/dell-wmi.c index 363d927..7aac1dc 100644
> > >>>> --- a/drivers/platform/x86/dell-wmi.c
> > >>>> +++ b/drivers/platform/x86/dell-wmi.c
> > >>>> @@ -110,6 +110,9 @@ static const struct key_entry
> > >>
> > >> dell_wmi_legacy_keymap[] __initconst = {
> > >>
> > >>>> /* BIOS error detected */
> > >>>> { KE_IGNORE, 0xe00d, { KEY_RESERVED } },
> > >>>>
> > >>>> + /* Unknown, defined in ACPI DSDT */
> > >>>> + /* { KE_IGNORE, 0xe00e, { KEY_RESERVED } }, */
> > >>>> +
> > >>>
> > >>> I'm interested in knowing what's the meaning of this 0xe00e. This
> > >>> event is sent multiple times when I suspend/resume my laptop and
> > >>> it's definitely not a keypress.
> > >>
> > >> From DSDT dumps which I have seen, I guess it could be something
> > >> with battery charging... but that is only my guess.
> > >>
> > >> Mario, do you have any idea, what these unknown events are?
> > >
> > > Off-hand I'm not sure, it would require some more digging.
> > >
> > > Can you please remind me what model numbers and BIOS combinations
> > > you have found e00e in DSDT and what context the events are
> > > actually happening? Anything released in the past two years?
> >
> > XPS13 9333, BIOS A07.
> >
> > I think I saw the event only after resuming from suspend and
> > it's sent four times in a row.
> >
> > As Pali says, it seems to be related to the battery. There are
> > three _Qxx ACPI methods in my DSDT sending this event: one stops
> > battery charging, one detaches the battery and the last one stores
> > a value on the GNVS.
>
> Mario, were you able to identify something?
>
Pali,

You aren't seeing this on the DSDT of your Latitude right?

Gabriele,

Your machine is from the year before XPS switched over to running
the Dell business client (eg Latitude, Precision, Optiplex) BIOS.

The EC in that machine does have support for "Battery Health" via
that scancode. On Windows it's used for relaying battery information
to an application called Quick Set.

Thanks,