Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s

From	Pali Rohár <pali.rohar@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s
Date	2016-07-27 09:30 +0200
Message-ID	<rZrrX-4Me-7@gated-at.bofh.it> (permalink)
References	<rXP5n-23Z-1@gated-at.bofh.it> <rZpqa-3Bu-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tuesday 26 July 2016 22:19:09 Darren Hart wrote:
> On Fri, Jul 22, 2016 at 03:01:24PM -0500, Mario Limonciello wrote:
> > The Dell Rugged 7202 has 3 programmable buttons (labeled P1, P2, P3)
> > and a detachable magnetic keyboard/mouse.
> > 
> > Signed-off-by: Mario Limonciello <mario_limonciello@dell.com>
> 
> +Pali
> 
> Pali, any concerns?

Hi! I'm not sure if KEY_PROG1/2/3 are good names here as we already use
those for other actions (see bios_to_linux_keycode). Also there is:

/* Wifi Catcher */
{ KE_KEY,    0xe011, { KEY_PROG2 } },

But probably we do not have better names...

Another small cosmetic change: align WMI codes, so 0x157 and 0x850 are
at some column (similar like are formatted other actions).

> > ---
> >  drivers/platform/x86/dell-wmi.c | 10 ++++++++++
> >  1 file changed, 10 insertions(+)
> > 
> > diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-wmi.c
> > index 15c6f11..6ba42d0 100644
> > --- a/drivers/platform/x86/dell-wmi.c
> > +++ b/drivers/platform/x86/dell-wmi.c
> > @@ -222,6 +222,16 @@ static const struct key_entry dell_wmi_extra_keymap[] __initconst = {
> >  
> >  	/* Stealth mode toggle */
> >  	{ KE_IGNORE, 0x155, { KEY_RESERVED } },
> > +
> > +	/* Rugged magnetic keyboard/mouse events */
> > +	{ KE_IGNORE, 0x156, { KEY_RESERVED } },
> > +	{ KE_IGNORE, 0x157, { KEY_RESERVED } },
> > +
> > +	/* Rugged programmable (P1/P2/P3 keys) */
> > +	{ KE_KEY, 0x850, { KEY_PROG1 } },
> > +	{ KE_KEY, 0x851, { KEY_PROG2 } },
> > +	{ KE_KEY, 0x852, { KEY_PROG3 } },
> > +
> >  };
> >  
> >  static struct input_dev *dell_wmi_input_dev;
> > -- 
> > 2.7.4
> > 
> > 
> 

-- 
Pali Rohár
pali.rohar@gmail.com

Thread

Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s Darren Hart <dvhart@infradead.org> - 2016-07-27 07:20 +0200
  Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s Pali Rohár <pali.rohar@gmail.com> - 2016-07-27 09:30 +0200
    Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s <Mario_Limonciello@Dell.com> - 2016-07-27 18:00 +0200
      Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s Pali Rohár <pali.rohar@gmail.com> - 2016-07-27 18:10 +0200
        Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s Darren Hart <dvhart@infradead.org> - 2016-07-27 18:40 +0200
        RE: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s <Mario_Limonciello@Dell.com> - 2016-07-27 21:10 +0200
          Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s Pali Rohár <pali.rohar@gmail.com> - 2016-07-27 22:20 +0200
            Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s Darren Hart <dvhart@infradead.org> - 2016-07-28 00:50 +0200
              Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-07-28 01:10 +0200
                RE: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s <Mario_Limonciello@Dell.com> - 2016-07-28 18:00 +0200
                Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-07-28 20:50 +0200
                Re: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s Pali Rohár <pali.rohar@gmail.com> - 2016-07-28 21:30 +0200
                RE: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s <Mario_Limonciello@Dell.com> - 2016-08-02 00:50 +0200
            RE: [PATCH] dell-wmi: Add events created by Dell Rugged 2-in-1s <Mario_Limonciello@Dell.com> - 2016-07-28 17:40 +0200

csiph-web