From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934171AbcBQMCF (ORCPT ); Wed, 17 Feb 2016 07:02:05 -0500 Received: from mail-lf0-f49.google.com ([209.85.215.49]:36482 "EHLO mail-lf0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756148AbcBQMCD (ORCPT ); Wed, 17 Feb 2016 07:02:03 -0500 Date: Wed, 17 Feb 2016 13:01:58 +0100 From: =?utf-8?B?TWljaGHFgiBLxJlwaWXFhA==?= To: Pali =?utf-8?B?Um9ow6Fy?= Cc: Matthew Garrett , Darren Hart , Darek Stojaczyk , platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 5/5] dell-wmi: support Dell Inspiron M5110 Message-ID: <20160217120158.GA3494@eudyptula.hq.kempniu.pl> References: <20160121090401.GR7192@pali> <1455634230-1487-1-git-send-email-kernel@kempniu.pl> <1455634230-1487-6-git-send-email-kernel@kempniu.pl> <20160216152246.GL1476@pali> <20160216220312.GB2741@eudyptula.hq.kempniu.pl> <20160217114230.GS1476@pali> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160217114230.GS1476@pali> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > > > > /* Shortcut and audio panel keys */ > > > > { KE_KEY, 0xe025, { KEY_PROG4 } }, > > > > { KE_IGNORE, 0xe026, { KEY_RESERVED } }, > > > > + { KE_KEY, 0xe029, { KEY_PROG4 } }, > > > > > > Hi! Above comment (Shortcut and audio panel keys) is not relevant for > > > this 0xe029, right? > > > > I believe it is relevant. The commit which added that comment > > (f1566f0d) was made by Seth Forshee to support Dell XPS L502X. The term > > "shortcut key" is used in the bug report linked from that commit [1], > > but the manual for that model [2] describes the hotkey in question as > > "Instant launch control" (page 43), so I believe both Seth's commit and > > my patch refer to the same thing. > > > > [1] https://bugs.launchpad.net/ubuntu/+source/linux/+bug/815914/comments/6 > > [2] http://downloads.dell.com/Manuals/all-products/esuprt_laptop/esuprt_xps_laptop/xps-l502x_setup%20guide_en-us.pdf > > Then please split above group of keys and add comment which key is > shortcut and which audio panel. Because now there will be three keys and > comment just for two -- in future will not know which code 0x... is > mapped to which key. Please note that except for the first 5 entries, the keymap is sorted by event code in ascending order. If I understand correctly, doing what you ask would break that ordering: /* Dell Instant Launch key */ { KE_KEY, 0xe025, { KEY_PROG4 } }, { KE_KEY, 0xe029, { KEY_PROG4 } }, /* Audio panel key */ { KE_IGNORE, 0xe026, { KEY_RESERVED } }, Is this acceptable? Or perhaps you had something else on your mind? -- Best regards, Michał Kępień