Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1336292
| From | Pali Rohár <pali.rohar@gmail.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v3 5/5] dell-wmi: support Dell Inspiron M5110 |
| Date | 2016-02-17 12:50 +0100 |
| Message-ID | <r38Mi-5Nt-33@gated-at.bofh.it> (permalink) |
| References | <qTjpE-39M-21@gated-at.bofh.it> <r2PgC-KD-5@gated-at.bofh.it> <r2PgC-KD-9@gated-at.bofh.it> <r2PJE-1c4-9@gated-at.bofh.it> <r2VYJ-5w0-13@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Tuesday 16 February 2016 23:03:12 Michał Kępień wrote:
> > > /* 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.
--
Pali Rohár
pali.rohar@gmail.com
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v3 5/5] dell-wmi: support Dell Inspiron M5110 Michał Kępień <kernel@kempniu.pl> - 2016-02-16 16:00 +0100
Re: [PATCH v3 5/5] dell-wmi: support Dell Inspiron M5110 Pali Rohár <pali.rohar@gmail.com> - 2016-02-16 16:30 +0100
Re: [PATCH v3 5/5] dell-wmi: support Dell Inspiron M5110 Michał Kępień <kernel@kempniu.pl> - 2016-02-16 23:10 +0100
Re: [PATCH v3 5/5] dell-wmi: support Dell Inspiron M5110 Pali Rohár <pali.rohar@gmail.com> - 2016-02-17 12:50 +0100
Re: [PATCH v3 5/5] dell-wmi: support Dell Inspiron M5110 Michał Kępień <kernel@kempniu.pl> - 2016-02-17 13:10 +0100
Re: [PATCH v3 5/5] dell-wmi: support Dell Inspiron M5110 Pali Rohár <pali.rohar@gmail.com> - 2016-02-17 13:10 +0100
Re: [PATCH v3 5/5] dell-wmi: support Dell Inspiron M5110 Michał Kępień <kernel@kempniu.pl> - 2016-02-18 09:30 +0100
csiph-web