lkml.org 
[lkml]   [2015]   [Apr]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] Input: atkbd: Fix release quirk for Dell models
On Wed, Apr 08, 2015 at 10:54:20PM +0200, Pali Rohár wrote:
> On Monday 06 April 2015 18:20:56 Dmitry Torokhov wrote:
> > On Mon, Apr 06, 2015 at 10:06:10AM +0200, Pali Rohár wrote:
> > > On Monday 06 April 2015 01:53:43 Dmitry Torokhov wrote:
> > > > On Mon, Apr 06, 2015 at 12:00:32AM +0200, Pali Rohár wrote:
> > > > > On Sunday 05 April 2015 23:48:33 Dmitry Torokhov wrote:
> > > > > > Hi Pali,
> > > > > >
> > > > > > On Sun, Mar 29, 2015 at 04:36:19PM +0200, Pali Rohár
> > > > > > wrote:
> > > > > > > This patch fixes commit 61579ba83934 ("Input: atkbd
> > > > > > > - expand Latitude's force release quirk to other
> > > > > > > Dells"). Before that commit release quirks were
> > > > > > > called for all Dell Latitude models. After that
> > > > > > > commit only for Portable Dell devices. But lot of
> > > > > > > Latitude models are Laptop or Notebook DMI devices
> > > > > > > so quirks are not called.
> > > > > > >
> > > > > > > Release quirks are still needed also for new Dell
> > > > > > > Latitude models, so this patch enables quirks for
> > > > > > > all Portable, Laptop, Notebook and Sub-Notebook
> > > > > > > Dell devices.
> > > > > >
> > > > > > Does Dell use all these types for their laptops? What
> > > > > > models do you know that need this quirk?
> > > > > >
> > > > > > Thanks.
> > > > >
> > > > > I do not if Dell use all types, but months ago Matthew
> > > > > wrote to include also other numbers not only 9 (Laptop)
> > > > > and you agreed.
> > > >
> > > > Hmm, I tried looking back but I could not quite find the
> > > > discussion.
> > >
> > > Search for Message-ID: <20141222175632.GB18556@dtor-ws>
> > > (this should be unique identifier for emails)
> >
> > Ah, indeed. Sorry, at this time I completely forgot that we
> > are controlling this from userspace nowadays.
> >
> > > > > I do not know exact list of models which needs these
> > > > > quirks, but before that commit (61579ba83934) it was
> > > > > used for all Latitude models. At least I see that
> > > > > switches do not generate release events and on older
> > > > > Latitude machines some Fn keys do not generate them
> > > > > too.
> > > >
> > > > So the question is still: what models do need this quirk
> > > > and what chassis type they are used.
> > >
> > > My model:
> > >
> > > $ cat /sys/class/dmi/id/chassis_type
> > > 9
> > > $ cat /sys/class/dmi/id/sys_vendor
> > > Dell Inc.
> > > $ cat /sys/class/dmi/id/product_name
> > > Latitude E6440
> > >
> > > > Regardless, I have not accepted quirks for force_release
> > > > and atkbd keymaps for many years now as the task to
> > > > adjust both keymap and force_release list has been
> > > > offloaded to udev.
> > > >
> > > > Thanks.
> > >
> > > Ok, and what to do with kernel regressions? As before commit
> > > 61579ba83934 it worked, because quirk was called for all
> > > Latitude (independently on chassis_type).
> >
> > No, this is not a kernel regression because it would mean that
> > your laptop worked properly with then current kernel and then
> > stopped working. Given that the change went in in 2008 and
> > your laptop is a bit newer than that you can't claim kernel
> > regression here; as far as I can see the change was
> > reasonable for the lineage we had back then, or at least
> > noone notified us for about 6 years.
> >
> > So please do adjust udev rules, there are quite a bit for
> > tweaks for Dell and other manufacturer's laptop keyboards
> > there).
> >
> > Thanks.
>
> Ok, thanks for information. Maybe you can add some comment to
> source code or documentation, that this part of code is now
> handled by userspace. So other people send patches to correct
> place/projects in future.

Fair enough. How about below?

Thanks.

--
Dmitry

Input: atkbd - document "no new force-release quirks" policy

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>

To save people some time let's document that we do not want new quirks for
"force-release" keys in the kernel and that they should patch userspace
(udev) instead.

Suggested-by: Pali Rohár <pali.rohar@gmail.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
drivers/input/keyboard/atkbd.c | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/drivers/input/keyboard/atkbd.c b/drivers/input/keyboard/atkbd.c
index 387c51f..ec876b5 100644
--- a/drivers/input/keyboard/atkbd.c
+++ b/drivers/input/keyboard/atkbd.c
@@ -1653,6 +1653,12 @@ static int __init atkbd_deactivate_fixup(const struct dmi_system_id *id)
return 1;
}

+/*
+ * NOTE: do not add any more "force release" quirks to this table. The
+ * task of adjusting list of keys that should be "released" automatically
+ * by the driver is now delegated to userspace tools, such as udev, so
+ * submit such quirks there.
+ */
static const struct dmi_system_id atkbd_dmi_quirk_table[] __initconst = {
{
.matches = {

\
 
 \ /
  Last update: 2015-04-09 19:21    [W:0.159 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site