RE: Regression: Dell XPS 13 9360 keyboard no longer works

[Mario.Limonciello]

> -----Original Message-----
> From: platform-driver-x86-owner@xxxxxxxxxxxxxxx [mailto:platform-driver-x86-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Darren Hart
> Sent: Thursday, February 22, 2018 10:19 AM
> To: Limonciello, Mario <Mario_Limonciello@xxxxxxxx>
> Cc: notmart@xxxxxxxxx; jeremy@xxxxxxxxxx; pali.rohar@xxxxxxxxx;
> andriy.shevchenko@xxxxxxxxxxxxxxx; mjg59@xxxxxxxxxxxxx; platform-driver-
> x86@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: Regression: Dell XPS 13 9360 keyboard no longer works
> 
> On Thu, Feb 22, 2018 at 03:21:45PM +0000, Mario.Limonciello@xxxxxxxx wrote:
> > > -----Original Message-----
> > > From: Marco Martin [mailto:notmart@xxxxxxxxx]
> > > Sent: Thursday, February 22, 2018 9:16 AM
> > > To: Jeremy Cline <jeremy@xxxxxxxxxx>
> > > Cc: Limonciello, Mario <Mario_Limonciello@xxxxxxxx>; Pali Rohár
> > > <pali.rohar@xxxxxxxxx>; Andy Shevchenko
> <andriy.shevchenko@xxxxxxxxxxxxxxx>;
> > > Matthew Garrett <mjg59@xxxxxxxxxxxxx>; Darren Hart
> <dvhart@xxxxxxxxxxxxx>;
> > > Platform Driver <platform-driver-x86@xxxxxxxxxxxxxxx>; Linux Kernel Mailing
> List
> > > <linux-kernel@xxxxxxxxxxxxxxx>
> > > Subject: Re: Regression: Dell XPS 13 9360 keyboard no longer works
> > >
> > > On Thu, Feb 22, 2018 at 3:08 PM, Jeremy Cline <jeremy@xxxxxxxxxx> wrote:
> > > > Input driver version is 1.0.1
> > > > Input device ID: bus 0x19 vendor 0x0 product 0x0 version 0x0
> > > > Input device name: "Intel Virtual Button driver"
> > > > Supported events:
> > > >   Event type 0 (EV_SYN)
> > > >   Event type 1 (EV_KEY)
> > > >     Event code 114 (KEY_VOLUMEDOWN)
> > > >     Event code 115 (KEY_VOLUMEUP)
> > > >     Event code 116 (KEY_POWER)
> > > >     Event code 125 (KEY_LEFTMETA)
> > > >     Event code 240 (KEY_UNKNOWN)
> > > >     Event code 561 (?)
> > > >   Event type 4 (EV_MSC)
> > > >     Event code 4 (MSC_SCAN)
> > > >   Event type 5 (EV_SW)
> > > >     Event code 1 (SW_TABLET_MODE) state 1
> > > > Properties:
> > > > Testing ... (interrupt to exit)
> > >
> > > Indeed, it's thinking a switch exists and that is on, which is wrong,
> > > then i guess VGBS() reports a wrong status..
> > > Mario, do you have any ideas about that?
> > >
> > > --
> > > Marco Martin
> >
> > I guess that means we got this wrong and the patch should be reverted
> > until we figure this out.
> >
> > Jeremy,
> >
> > Can you please confirm what BIOS version you are on?
> > Also Is this a 9360 with 7th or 8th gen Intel CPU?
> >
> > I'll see if I can find out anything more relative to this.
> 
> Watching closely.
> 
> Mario, do you want me to revert immediately, or wait until you've had a couple
> rounds of Q&A here?
> 
> --

Darren,

Let's hold off until we've discussed a little more.  Jeremy is responsive and we might
get to the bottom quickly.

Btw you have a 9360 on hand still right?  Can you see if you can reproduce this problem too?

Thanks,