From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Tissoires Subject: Re: [PATCH v3 3/4] Input: gpio_keys - Allow suppression of input events for wakeup button presses Date: Mon, 6 Nov 2017 10:19:01 +0100 Message-ID: <20171106091901.GH9728@mail.corp.redhat.com> References: <20171030174050.30375-1-hdegoede@redhat.com> <20171030174050.30375-3-hdegoede@redhat.com> <20171030181706.huwka5foqxparsiq@dtor-ws> <20171030204809.mciyapkfqlihg3jd@dtor-ws> <7ca65f7f-60c4-04b7-2990-3037216f2be0@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Received: from mx1.redhat.com ([209.132.183.28]:35270 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750914AbdKFJTG (ORCPT ); Mon, 6 Nov 2017 04:19:06 -0500 Content-Disposition: inline In-Reply-To: <7ca65f7f-60c4-04b7-2990-3037216f2be0@redhat.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Hans de Goede Cc: Dmitry Torokhov , linux-input@vger.kernel.org On Nov 01 2017 or thereabouts, Hans de Goede wrote: > Hi, > > On 30-10-17 21:48, Dmitry Torokhov wrote: > > > > > > So both for consistency and because a timeout is racy I believe > > > that having the power button not send KEY_POWER after resume is > > > the proper solution here. > > > > > > Note that I've made this configurable and that soc_button_array > > > is the only driver getting this enabled, making the power-button > > > behavior on PC like devices consistent, without impacting any > > > other devices. > > > > I'd rather we did not make assumption in the kernel about behavior of > > userspace we happen to run on. The "PC like" devices change all the > > time, and one could even say that failure to deliver events by ACPI > > drivers on resume is a bug. > > > > Please teach userspace how to handle events coming during resume phase > > and what to do about them, and be done with it. One option is to say (in > > whatever implements your power policy) "we want to ignore next KEY_POWER > > event for the next NNN msec". You will need it ianyway if you decide to > > put your userspace onto, let's say, ASUS Flip (an ARM device using > > gpio-keys for power button and volume up/down buttons on the side of the > > case). > > Ok, I've written and submitted patches for this for gnome: > https://bugzilla.gnome.org/show_bug.cgi?id=789771 > > That still leaves the same issue for KDE, XFCE, mate, etc. though, > note I've no intention of fixing those. > Hi Hans, I roughly had a look at the series, and I am wondering, can't we at least take the cleanup patches (1/4 2/4 as far as i can tell)? Cheers, Benjamin > Regards, > > Hans