On Thu, Feb 22, 2018 at 04:28:57PM +0000, Mario.Limonciello@xxxxxxxx wrote: > > -----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? > I'm planning on it, yup. -- Darren Hart VMware Open Source Technology Center