From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darren Hart Subject: Re: [PATCH v2 3/3] dell-wmi: Improve unknown hotkey handling Date: Thu, 3 Dec 2015 16:21:34 -0800 Message-ID: <20151204002134.GH1800@malice.jf.intel.com> References: <20151203233856.GC1800@malice.jf.intel.com> <20151204000721.GF1800@malice.jf.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from bombadil.infradead.org ([198.137.202.9]:43874 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752137AbbLDAVi (ORCPT ); Thu, 3 Dec 2015 19:21:38 -0500 Content-Disposition: inline In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Andy Lutomirski Cc: Andy Lutomirski , Matthew Garrett , Linux ACPI , platform-driver-x86@vger.kernel.org, Mario Limonciello , Pali =?iso-8859-1?Q?Roh=E1r?= On Thu, Dec 03, 2015 at 04:10:49PM -0800, Andy Lutomirski wrote: > On Thu, Dec 3, 2015 at 4:07 PM, Darren Hart wrote: > > On Thu, Dec 03, 2015 at 03:45:11PM -0800, Andy Lutomirski wrote: > >> On Thu, Dec 3, 2015 at 3:38 PM, Darren Hart wrote: > >> > On Mon, Nov 30, 2015 at 05:02:01PM -0800, Andy Lutomirski wrote: > >> >> If DMI lists a hotkey that we don't recognize, log and ignore it > >> >> instead of trying to map it to keycode 0. I haven't seen this happen, > >> >> but it will help maintain the key map in the future and it will help > >> >> avoid sending bogus events. > >> >> > >> >> This also improves the message that we log when we get an unknown key > >> >> event. > >> >> > >> >> Signed-off-by: Andy Lutomirski > >> >> --- > >> > > >> > Please include the individual patch changelogs here under --- in the patch > >> > itself - this helps me build confidence that I am indeed looking at the right > >> > patch and the expected changes are here. > >> > >> Will do in the future -- different maintainers have different > >> preferences here, I think. > > > > Oh really? > > > > This is the documented process in Documentation/SubmittingPatches Section 14) > > The canonical patch format... > > > > Ingo and the other -tip maintainers seem to like all the comments up > front on the cover letter. Or at least they haven't complained yet. > Ugh, well, I don't like to ask contributors to treat this subsystem special by sticking to the documentation as much as possible. I really do find the per-patch changelog to be very useful. Thanks for working with me on that. > Anyway, I'm giving 'git notes' a try to generate nice change comments for v3. If you get a good solution, that would be nice to add to the documentation to help automate the process. Anything we can do to avoid mechanical errors here is goodness. -- Darren Hart Intel Open Source Technology Center