From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755077AbbLDBQM (ORCPT ); Thu, 3 Dec 2015 20:16:12 -0500 Received: from bombadil.infradead.org ([198.137.202.9]:56632 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755051AbbLDBQK (ORCPT ); Thu, 3 Dec 2015 20:16:10 -0500 Date: Thu, 3 Dec 2015 17:16:06 -0800 From: Darren Hart To: =?utf-8?B?TWljaGHFgiBLxJlwaWXFhA==?= Cc: Matthew Garrett , Pali =?iso-8859-1?Q?Roh=E1r?= , platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org, Andy Lutomirski Subject: Re: [PATCH v2] dell-wmi: process Dell Instant Launch hotkey on Dell Vostro V131 Message-ID: <20151204011606.GI1800@malice.jf.intel.com> References: <20151130211542.GE30553@malice.jf.intel.com> <1654e7bcde98f1cf89f698a1e359110c06c6fcd4.1448999372.git.kernel@kempniu.pl> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1654e7bcde98f1cf89f698a1e359110c06c6fcd4.1448999372.git.kernel@kempniu.pl> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 01, 2015 at 08:51:34PM +0100, Michał Kępień wrote: > On some laptop models (e.g. Dell Vostro V131), pressing the Dell Instant > Launch hotkey does not raise an i8042 interrupt - only WMI event 0xe025 > is generated. As there seems to be no ACPI method or SMI call to > determine without possible side effects whether a given machine is > capable of simulating a keypress when this hotkey is pressed, DMI > matching is used to whitelist the models for which an input event should > be generated when WMI event 0xe025 is received. > > Signed-off-by: Michał Kępień > --- > Changes from v1: > > - Use DMI matching instead of a module parameter > - Change flag name to improve readability > > Darren, I am aware this patch conflicts with Andy's WMI rework series > posted yesterday, so please just let me know if you want me to rebase. Thanks for the heads' up. Andy's series is longer and likely going to need more review from more people, so I don't necessarily want to hold this one up on it - unless it makes sense to include it in that series so they can evolve together. I'll leave that to you and Andy to decide. This looks fine to me, and if Pali will ack it, I'll move it from for-review to testing and Andy will need to update patch 14/14 to accomodate - unless you guys decide to include this in his. For now, this is queued to for-review. Thanks! -- Darren Hart Intel Open Source Technology Center