* [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
@ 2011-02-25 5:19 ` Keng-Yu Lin
0 siblings, 0 replies; 17+ messages in thread
From: Keng-Yu Lin @ 2011-02-25 5:19 UTC (permalink / raw)
To: Corentin Chary, Matthew Garrett, acpi4asus-user,
platform-driver-x86, linux-kernel
Cc: tyson.chen, Keng-Yu Lin
Signed-off-by: Keng-Yu Lin <keng-yu.lin@canonical.com>
---
drivers/platform/x86/eeepc-wmi.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
https://bugs.freedesktop.org/show_bug.cgi?id=31333
diff --git a/drivers/platform/x86/eeepc-wmi.c b/drivers/platform/x86/eeepc-wmi.c
index d399775..da00cbc 100644
--- a/drivers/platform/x86/eeepc-wmi.c
+++ b/drivers/platform/x86/eeepc-wmi.c
@@ -113,7 +113,7 @@ static const struct key_entry eeepc_wmi_keymap[] = {
{ KE_KEY, 0x32, { KEY_MUTE } },
{ KE_KEY, 0x5c, { KEY_F15 } }, /* Power Gear key */
{ KE_KEY, 0x5d, { KEY_WLAN } },
- { KE_KEY, 0x6b, { KEY_F13 } }, /* Disable Touchpad */
+ { KE_KEY, 0x6b, { KEY_F21 } }, /* Disable Touchpad */
{ KE_KEY, 0x82, { KEY_CAMERA } },
{ KE_KEY, 0x83, { KEY_CAMERA_ZOOMIN } },
{ KE_KEY, 0x88, { KEY_WLAN } },
--
1.7.1
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
@ 2011-02-25 5:19 ` Keng-Yu Lin
0 siblings, 0 replies; 17+ messages in thread
From: Keng-Yu Lin @ 2011-02-25 5:19 UTC (permalink / raw)
To: Corentin Chary, Matthew Garrett,
acpi4asus-user-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
platform-driver-x86-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: tyson.chen-Z7WLFzj8eWMS+FvcfC7Uqw, Keng-Yu Lin
Signed-off-by: Keng-Yu Lin <keng-yu.lin-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
---
drivers/platform/x86/eeepc-wmi.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
https://bugs.freedesktop.org/show_bug.cgi?id=31333
diff --git a/drivers/platform/x86/eeepc-wmi.c b/drivers/platform/x86/eeepc-wmi.c
index d399775..da00cbc 100644
--- a/drivers/platform/x86/eeepc-wmi.c
+++ b/drivers/platform/x86/eeepc-wmi.c
@@ -113,7 +113,7 @@ static const struct key_entry eeepc_wmi_keymap[] = {
{ KE_KEY, 0x32, { KEY_MUTE } },
{ KE_KEY, 0x5c, { KEY_F15 } }, /* Power Gear key */
{ KE_KEY, 0x5d, { KEY_WLAN } },
- { KE_KEY, 0x6b, { KEY_F13 } }, /* Disable Touchpad */
+ { KE_KEY, 0x6b, { KEY_F21 } }, /* Disable Touchpad */
{ KE_KEY, 0x82, { KEY_CAMERA } },
{ KE_KEY, 0x83, { KEY_CAMERA_ZOOMIN } },
{ KE_KEY, 0x88, { KEY_WLAN } },
--
1.7.1
------------------------------------------------------------------------------
Free Software Download: Index, Search & Analyze Logs and other IT data in
Real-Time with Splunk. Collect, index and harness all the fast moving IT data
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business
insights. http://p.sf.net/sfu/splunk-dev2dev
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
2011-02-25 5:19 ` Keng-Yu Lin
(?)
@ 2011-02-25 6:52 ` Corentin Chary
2011-02-25 7:18 ` Dmitry Torokhov
2011-02-25 14:12 ` Matthew Garrett
-1 siblings, 2 replies; 17+ messages in thread
From: Corentin Chary @ 2011-02-25 6:52 UTC (permalink / raw)
To: Keng-Yu Lin
Cc: Matthew Garrett, acpi4asus-user, platform-driver-x86,
linux-kernel, tyson.chen, Dmitry Torokhov
On Fri, Feb 25, 2011 at 6:19 AM, Keng-Yu Lin <keng-yu.lin@canonical.com> wrote:
> Signed-off-by: Keng-Yu Lin <keng-yu.lin@canonical.com>
> ---
> drivers/platform/x86/eeepc-wmi.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> https://bugs.freedesktop.org/show_bug.cgi?id=31333
>
> diff --git a/drivers/platform/x86/eeepc-wmi.c b/drivers/platform/x86/eeepc-wmi.c
> index d399775..da00cbc 100644
> --- a/drivers/platform/x86/eeepc-wmi.c
> +++ b/drivers/platform/x86/eeepc-wmi.c
> @@ -113,7 +113,7 @@ static const struct key_entry eeepc_wmi_keymap[] = {
> { KE_KEY, 0x32, { KEY_MUTE } },
> { KE_KEY, 0x5c, { KEY_F15 } }, /* Power Gear key */
> { KE_KEY, 0x5d, { KEY_WLAN } },
> - { KE_KEY, 0x6b, { KEY_F13 } }, /* Disable Touchpad */
> + { KE_KEY, 0x6b, { KEY_F21 } }, /* Disable Touchpad */
> { KE_KEY, 0x82, { KEY_CAMERA } },
> { KE_KEY, 0x83, { KEY_CAMERA_ZOOMIN } },
> { KE_KEY, 0x88, { KEY_WLAN } },
Hum, I'm not very comfortable with this patch now that we got
KEY_TOUCHPAD_TOGGLE.
I know, it doesn't work with current Xorg, but if we continue to
workaround that, it'll never be fixed.
Dmitry, Matthew, any advice ?
--
Corentin Chary
http://xf.iksaif.net
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
2011-02-25 6:52 ` Corentin Chary
@ 2011-02-25 7:18 ` Dmitry Torokhov
2011-02-25 9:22 ` Keng-Yü Lin
2011-02-25 15:14 ` Chris Bagwell
2011-02-25 14:12 ` Matthew Garrett
1 sibling, 2 replies; 17+ messages in thread
From: Dmitry Torokhov @ 2011-02-25 7:18 UTC (permalink / raw)
To: Corentin Chary
Cc: Keng-Yu Lin, Matthew Garrett, acpi4asus-user,
platform-driver-x86, linux-kernel, tyson.chen
On Fri, Feb 25, 2011 at 07:52:11AM +0100, Corentin Chary wrote:
> On Fri, Feb 25, 2011 at 6:19 AM, Keng-Yu Lin <keng-yu.lin@canonical.com> wrote:
> > Signed-off-by: Keng-Yu Lin <keng-yu.lin@canonical.com>
> > ---
> > drivers/platform/x86/eeepc-wmi.c | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > https://bugs.freedesktop.org/show_bug.cgi?id=31333
> >
> > diff --git a/drivers/platform/x86/eeepc-wmi.c b/drivers/platform/x86/eeepc-wmi.c
> > index d399775..da00cbc 100644
> > --- a/drivers/platform/x86/eeepc-wmi.c
> > +++ b/drivers/platform/x86/eeepc-wmi.c
> > @@ -113,7 +113,7 @@ static const struct key_entry eeepc_wmi_keymap[] = {
> > { KE_KEY, 0x32, { KEY_MUTE } },
> > { KE_KEY, 0x5c, { KEY_F15 } }, /* Power Gear key */
> > { KE_KEY, 0x5d, { KEY_WLAN } },
> > - { KE_KEY, 0x6b, { KEY_F13 } }, /* Disable Touchpad */
> > + { KE_KEY, 0x6b, { KEY_F21 } }, /* Disable Touchpad */
> > { KE_KEY, 0x82, { KEY_CAMERA } },
> > { KE_KEY, 0x83, { KEY_CAMERA_ZOOMIN } },
> > { KE_KEY, 0x88, { KEY_WLAN } },
>
> Hum, I'm not very comfortable with this patch now that we got
> KEY_TOUCHPAD_TOGGLE.
> I know, it doesn't work with current Xorg, but if we continue to
> workaround that, it'll never be fixed.
>
> Dmitry, Matthew, any advice ?
I believe that the kernel should use KEY_TOUCHPAD_TOGGLE and userspace
should use udev's key re-mapping facilities while X is unable to process
keycodes above 255 to adjust to the keycode du jour. If all kernel
drivers were to use KEY_TOUCHPAD_TOGGLE then remapping would be much
easier (the utility would simply check capabilities for
KEY_TOUCHPAD_TOGGLE and do the remap).
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
2011-02-25 7:18 ` Dmitry Torokhov
@ 2011-02-25 9:22 ` Keng-Yü Lin
2011-02-25 9:33 ` Dmitry Torokhov
2011-02-25 9:53 ` Martin Pitt
2011-02-25 15:14 ` Chris Bagwell
1 sibling, 2 replies; 17+ messages in thread
From: Keng-Yü Lin @ 2011-02-25 9:22 UTC (permalink / raw)
To: Dmitry Torokhov, Martin Pitt
Cc: Corentin Chary, Matthew Garrett, acpi4asus-user,
platform-driver-x86, linux-kernel, tyson.chen
On Fri, Feb 25, 2011 at 3:18 PM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> I believe that the kernel should use KEY_TOUCHPAD_TOGGLE and userspace
> should use udev's key re-mapping facilities while X is unable to process
> keycodes above 255 to adjust to the keycode du jour. If all kernel
> drivers were to use KEY_TOUCHPAD_TOGGLE then remapping would be much
> easier (the utility would simply check capabilities for
> KEY_TOUCHPAD_TOGGLE and do the remap).
Can udev re-map a key code to another?
Am I correct that udev can only re-map a scan code to a specified key code.
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
@ 2011-02-25 9:33 ` Dmitry Torokhov
0 siblings, 0 replies; 17+ messages in thread
From: Dmitry Torokhov @ 2011-02-25 9:33 UTC (permalink / raw)
To: Keng-Yü Lin
Cc: Martin Pitt, Corentin Chary, Matthew Garrett, acpi4asus-user,
platform-driver-x86, linux-kernel, tyson.chen
On Fri, Feb 25, 2011 at 05:22:43PM +0800, Keng-Yü Lin wrote:
> On Fri, Feb 25, 2011 at 3:18 PM, Dmitry Torokhov
> <dmitry.torokhov@gmail.com> wrote:
> > I believe that the kernel should use KEY_TOUCHPAD_TOGGLE and userspace
> > should use udev's key re-mapping facilities while X is unable to process
> > keycodes above 255 to adjust to the keycode du jour. If all kernel
> > drivers were to use KEY_TOUCHPAD_TOGGLE then remapping would be much
> > easier (the utility would simply check capabilities for
> > KEY_TOUCHPAD_TOGGLE and do the remap).
>
> Can udev re-map a key code to another?
Not directly.
>
> Am I correct that udev can only re-map a scan code to a specified key code.
Right, so once you determined via capabilities that device emits
KEY_TOUCHPAD_TOGGLE you query the keymap (using EVIOCGKEYCODE2; by
index) and replace all entries that are mapped to KEY_TOUCHPAD_TOGGLE to
whatever you want.
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
@ 2011-02-25 9:33 ` Dmitry Torokhov
0 siblings, 0 replies; 17+ messages in thread
From: Dmitry Torokhov @ 2011-02-25 9:33 UTC (permalink / raw)
To: Keng-Yü Lin
Cc: Martin Pitt, acpi4asus-user-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
platform-driver-x86-u79uwXL29TY76Z2rM5mHXA,
tyson.chen-Z7WLFzj8eWMS+FvcfC7Uqw, Corentin Chary,
Matthew Garrett
On Fri, Feb 25, 2011 at 05:22:43PM +0800, Keng-Yü Lin wrote:
> On Fri, Feb 25, 2011 at 3:18 PM, Dmitry Torokhov
> <dmitry.torokhov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> > I believe that the kernel should use KEY_TOUCHPAD_TOGGLE and userspace
> > should use udev's key re-mapping facilities while X is unable to process
> > keycodes above 255 to adjust to the keycode du jour. If all kernel
> > drivers were to use KEY_TOUCHPAD_TOGGLE then remapping would be much
> > easier (the utility would simply check capabilities for
> > KEY_TOUCHPAD_TOGGLE and do the remap).
>
> Can udev re-map a key code to another?
Not directly.
>
> Am I correct that udev can only re-map a scan code to a specified key code.
Right, so once you determined via capabilities that device emits
KEY_TOUCHPAD_TOGGLE you query the keymap (using EVIOCGKEYCODE2; by
index) and replace all entries that are mapped to KEY_TOUCHPAD_TOGGLE to
whatever you want.
Thanks.
--
Dmitry
------------------------------------------------------------------------------
Free Software Download: Index, Search & Analyze Logs and other IT data in
Real-Time with Splunk. Collect, index and harness all the fast moving IT data
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business
insights. http://p.sf.net/sfu/splunk-dev2dev
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
2011-02-25 9:22 ` Keng-Yü Lin
2011-02-25 9:33 ` Dmitry Torokhov
@ 2011-02-25 9:53 ` Martin Pitt
1 sibling, 0 replies; 17+ messages in thread
From: Martin Pitt @ 2011-02-25 9:53 UTC (permalink / raw)
To: Keng-Yü Lin
Cc: Dmitry Torokhov, Corentin Chary, Matthew Garrett, acpi4asus-user,
platform-driver-x86, linux-kernel, tyson.chen
Keng-Yü Lin [2011-02-25 17:22 +0800]:
> Can udev re-map a key code to another?
Not directly, as udev itself doesn't actually process the input
events. It just pokes the scan code -> keycode map into the kernel.
What it can do is to re-map an existing key (scan code) to a different
key code, which provides the same effect in most cases.
Martin
--
Martin Pitt | http://www.piware.de
Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org)
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
2011-02-25 6:52 ` Corentin Chary
2011-02-25 7:18 ` Dmitry Torokhov
@ 2011-02-25 14:12 ` Matthew Garrett
1 sibling, 0 replies; 17+ messages in thread
From: Matthew Garrett @ 2011-02-25 14:12 UTC (permalink / raw)
To: Corentin Chary
Cc: Keng-Yu Lin, acpi4asus-user, platform-driver-x86, linux-kernel,
tyson.chen, Dmitry Torokhov
On Fri, Feb 25, 2011 at 07:52:11AM +0100, Corentin Chary wrote:
> Hum, I'm not very comfortable with this patch now that we got
> KEY_TOUCHPAD_TOGGLE.
> I know, it doesn't work with current Xorg, but if we continue to
> workaround that, it'll never be fixed.
>
> Dmitry, Matthew, any advice ?
Yeah, I'd go with Dmitry's idea for now.
--
Matthew Garrett | mjg59@srcf.ucam.org
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
2011-02-25 7:18 ` Dmitry Torokhov
2011-02-25 9:22 ` Keng-Yü Lin
@ 2011-02-25 15:14 ` Chris Bagwell
2011-02-25 22:25 ` Joey Lee
2011-03-01 3:29 ` Keng-Yü Lin
1 sibling, 2 replies; 17+ messages in thread
From: Chris Bagwell @ 2011-02-25 15:14 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Corentin Chary, Keng-Yu Lin, Matthew Garrett, acpi4asus-user,
platform-driver-x86, linux-kernel, tyson.chen
On Fri, Feb 25, 2011 at 1:18 AM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> On Fri, Feb 25, 2011 at 07:52:11AM +0100, Corentin Chary wrote:
>> On Fri, Feb 25, 2011 at 6:19 AM, Keng-Yu Lin <keng-yu.lin@canonical.com> wrote:
>> > Signed-off-by: Keng-Yu Lin <keng-yu.lin@canonical.com>
>> > ---
>> > drivers/platform/x86/eeepc-wmi.c | 2 +-
>> > 1 files changed, 1 insertions(+), 1 deletions(-)
>> >
>> > https://bugs.freedesktop.org/show_bug.cgi?id=31333
>> >
>> > diff --git a/drivers/platform/x86/eeepc-wmi.c b/drivers/platform/x86/eeepc-wmi.c
>> > index d399775..da00cbc 100644
>> > --- a/drivers/platform/x86/eeepc-wmi.c
>> > +++ b/drivers/platform/x86/eeepc-wmi.c
>> > @@ -113,7 +113,7 @@ static const struct key_entry eeepc_wmi_keymap[] = {
>> > { KE_KEY, 0x32, { KEY_MUTE } },
>> > { KE_KEY, 0x5c, { KEY_F15 } }, /* Power Gear key */
>> > { KE_KEY, 0x5d, { KEY_WLAN } },
>> > - { KE_KEY, 0x6b, { KEY_F13 } }, /* Disable Touchpad */
>> > + { KE_KEY, 0x6b, { KEY_F21 } }, /* Disable Touchpad */
>> > { KE_KEY, 0x82, { KEY_CAMERA } },
>> > { KE_KEY, 0x83, { KEY_CAMERA_ZOOMIN } },
>> > { KE_KEY, 0x88, { KEY_WLAN } },
>>
>> Hum, I'm not very comfortable with this patch now that we got
>> KEY_TOUCHPAD_TOGGLE.
>> I know, it doesn't work with current Xorg, but if we continue to
>> workaround that, it'll never be fixed.
>>
>> Dmitry, Matthew, any advice ?
>
> I believe that the kernel should use KEY_TOUCHPAD_TOGGLE and userspace
> should use udev's key re-mapping facilities while X is unable to process
> keycodes above 255 to adjust to the keycode du jour. If all kernel
> drivers were to use KEY_TOUCHPAD_TOGGLE then remapping would be much
> easier (the utility would simply check capabilities for
> KEY_TOUCHPAD_TOGGLE and do the remap).
I submitted last month a patch to udev to do the scancode-to-F21
mapping. So that part should be covered for near term.
I also agree if we are going to change kernel side, we should change
to KEY_TOUCHPAD_TOGGLE in both eeepc-laptop and eeepc-wmi.
http://git.kernel.org/?p=linux/hotplug/udev.git;a=commit;h=0d8b6be5f5238360d2d8e220d6da280f842c5358
Chris
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
2011-02-25 15:14 ` Chris Bagwell
@ 2011-02-25 22:25 ` Joey Lee
2011-03-01 3:29 ` Keng-Yü Lin
1 sibling, 0 replies; 17+ messages in thread
From: Joey Lee @ 2011-02-25 22:25 UTC (permalink / raw)
To: chris
Cc: keng-yu.lin, tyson.chen, corentin.chary, dmitry.torokhov,
acpi4asus-user, Joey Lee, mjg, linux-kernel, platform-driver-x86
於 五,2011-02-25 於 09:14 -0600,Chris Bagwell 提到:
> On Fri, Feb 25, 2011 at 1:18 AM, Dmitry Torokhov
> <dmitry.torokhov@gmail.com> wrote:
> > On Fri, Feb 25, 2011 at 07:52:11AM +0100, Corentin Chary wrote:
> >> On Fri, Feb 25, 2011 at 6:19 AM, Keng-Yu Lin <keng-yu.lin@canonical.com> wrote:
> >> > Signed-off-by: Keng-Yu Lin <keng-yu.lin@canonical.com>
> >> > ---
> >> > drivers/platform/x86/eeepc-wmi.c | 2 +-
> >> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >> >
> >> > https://bugs.freedesktop.org/show_bug.cgi?id=31333
> >> >
> >> > diff --git a/drivers/platform/x86/eeepc-wmi.c b/drivers/platform/x86/eeepc-wmi.c
> >> > index d399775..da00cbc 100644
> >> > --- a/drivers/platform/x86/eeepc-wmi.c
> >> > +++ b/drivers/platform/x86/eeepc-wmi.c
> >> > @@ -113,7 +113,7 @@ static const struct key_entry eeepc_wmi_keymap[] = {
> >> > { KE_KEY, 0x32, { KEY_MUTE } },
> >> > { KE_KEY, 0x5c, { KEY_F15 } }, /* Power Gear key */
> >> > { KE_KEY, 0x5d, { KEY_WLAN } },
> >> > - { KE_KEY, 0x6b, { KEY_F13 } }, /* Disable Touchpad */
> >> > + { KE_KEY, 0x6b, { KEY_F21 } }, /* Disable Touchpad */
> >> > { KE_KEY, 0x82, { KEY_CAMERA } },
> >> > { KE_KEY, 0x83, { KEY_CAMERA_ZOOMIN } },
> >> > { KE_KEY, 0x88, { KEY_WLAN } },
> >>
> >> Hum, I'm not very comfortable with this patch now that we got
> >> KEY_TOUCHPAD_TOGGLE.
> >> I know, it doesn't work with current Xorg, but if we continue to
> >> workaround that, it'll never be fixed.
> >>
> >> Dmitry, Matthew, any advice ?
> >
> > I believe that the kernel should use KEY_TOUCHPAD_TOGGLE and userspace
> > should use udev's key re-mapping facilities while X is unable to process
> > keycodes above 255 to adjust to the keycode du jour. If all kernel
> > drivers were to use KEY_TOUCHPAD_TOGGLE then remapping would be much
> > easier (the utility would simply check capabilities for
> > KEY_TOUCHPAD_TOGGLE and do the remap).
>
> I submitted last month a patch to udev to do the scancode-to-F21
> mapping. So that part should be covered for near term.
>
> I also agree if we are going to change kernel side, we should change
> to KEY_TOUCHPAD_TOGGLE in both eeepc-laptop and eeepc-wmi.
>
> http://git.kernel.org/?p=linux/hotplug/udev.git;a=commit;h=0d8b6be5f5238360d2d8e220d6da280f842c5358
>
Found acer-wmi also have Touchpad toggle but mapping to F22
{KE_KEY, 0x82, {KEY_F22} }, /* Touch Pad On/Off */
Will also send patch to udev keymap first, then change the wmi driver
mapping to KEY_TOUCHPAD_TOGGLE.
Thank's
Joey Lee
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
@ 2011-02-25 22:25 ` Joey Lee
0 siblings, 0 replies; 17+ messages in thread
From: Joey Lee @ 2011-02-25 22:25 UTC (permalink / raw)
To: chris
Cc: keng-yu.lin, tyson.chen, corentin.chary, dmitry.torokhov,
acpi4asus-user, Joey Lee, mjg, linux-kernel, platform-driver-x86
於 五,2011-02-25 於 09:14 -0600,Chris Bagwell 提到:
> On Fri, Feb 25, 2011 at 1:18 AM, Dmitry Torokhov
> <dmitry.torokhov@gmail.com> wrote:
> > On Fri, Feb 25, 2011 at 07:52:11AM +0100, Corentin Chary wrote:
> >> On Fri, Feb 25, 2011 at 6:19 AM, Keng-Yu Lin <keng-yu.lin@canonical.com> wrote:
> >> > Signed-off-by: Keng-Yu Lin <keng-yu.lin@canonical.com>
> >> > ---
> >> > drivers/platform/x86/eeepc-wmi.c | 2 +-
> >> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >> >
> >> > https://bugs.freedesktop.org/show_bug.cgi?id=31333
> >> >
> >> > diff --git a/drivers/platform/x86/eeepc-wmi.c b/drivers/platform/x86/eeepc-wmi.c
> >> > index d399775..da00cbc 100644
> >> > --- a/drivers/platform/x86/eeepc-wmi.c
> >> > +++ b/drivers/platform/x86/eeepc-wmi.c
> >> > @@ -113,7 +113,7 @@ static const struct key_entry eeepc_wmi_keymap[] = {
> >> > { KE_KEY, 0x32, { KEY_MUTE } },
> >> > { KE_KEY, 0x5c, { KEY_F15 } }, /* Power Gear key */
> >> > { KE_KEY, 0x5d, { KEY_WLAN } },
> >> > - { KE_KEY, 0x6b, { KEY_F13 } }, /* Disable Touchpad */
> >> > + { KE_KEY, 0x6b, { KEY_F21 } }, /* Disable Touchpad */
> >> > { KE_KEY, 0x82, { KEY_CAMERA } },
> >> > { KE_KEY, 0x83, { KEY_CAMERA_ZOOMIN } },
> >> > { KE_KEY, 0x88, { KEY_WLAN } },
> >>
> >> Hum, I'm not very comfortable with this patch now that we got
> >> KEY_TOUCHPAD_TOGGLE.
> >> I know, it doesn't work with current Xorg, but if we continue to
> >> workaround that, it'll never be fixed.
> >>
> >> Dmitry, Matthew, any advice ?
> >
> > I believe that the kernel should use KEY_TOUCHPAD_TOGGLE and userspace
> > should use udev's key re-mapping facilities while X is unable to process
> > keycodes above 255 to adjust to the keycode du jour. If all kernel
> > drivers were to use KEY_TOUCHPAD_TOGGLE then remapping would be much
> > easier (the utility would simply check capabilities for
> > KEY_TOUCHPAD_TOGGLE and do the remap).
>
> I submitted last month a patch to udev to do the scancode-to-F21
> mapping. So that part should be covered for near term.
>
> I also agree if we are going to change kernel side, we should change
> to KEY_TOUCHPAD_TOGGLE in both eeepc-laptop and eeepc-wmi.
>
> http://git.kernel.org/?p=linux/hotplug/udev.git;a=commit;h=0d8b6be5f5238360d2d8e220d6da280f842c5358
>
Found acer-wmi also have Touchpad toggle but mapping to F22
{KE_KEY, 0x82, {KEY_F22} }, /* Touch Pad On/Off */
Will also send patch to udev keymap first, then change the wmi driver
mapping to KEY_TOUCHPAD_TOGGLE.
Thank's
Joey Lee
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] eeepc-wmi: set the touchpad toggle key code to F21
2011-02-25 15:14 ` Chris Bagwell
2011-02-25 22:25 ` Joey Lee
@ 2011-03-01 3:29 ` Keng-Yü Lin
1 sibling, 0 replies; 17+ messages in thread
From: Keng-Yü Lin @ 2011-03-01 3:29 UTC (permalink / raw)
To: Chris Bagwell
Cc: Dmitry Torokhov, Corentin Chary, Matthew Garrett, acpi4asus-user,
platform-driver-x86, linux-kernel, tyson.chen
On Fri, Feb 25, 2011 at 11:14 PM, Chris Bagwell <chris@cnpbagwell.com> wrote:
>
> I submitted last month a patch to udev to do the scancode-to-F21
> mapping. So that part should be covered for near term.
>
> I also agree if we are going to change kernel side, we should change
> to KEY_TOUCHPAD_TOGGLE in both eeepc-laptop and eeepc-wmi.
>
> http://git.kernel.org/?p=linux/hotplug/udev.git;a=commit;h=0d8b6be5f5238360d2d8e220d6da280f842c5358
>
> Chris
>
Thanks Chris for the information. I also tested; it works.
I will re-send a eeepc-wmi patch with KEY_TOUCHPAD_TOGGLE.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v2] eeepc-wmi: set the touchpad toggle key code to KEY_TOUCHPAD_TOGGLE
@ 2011-03-01 4:56 ` Keng-Yu Lin
0 siblings, 0 replies; 17+ messages in thread
From: Keng-Yu Lin @ 2011-03-01 4:56 UTC (permalink / raw)
To: Corentin Chary, Matthew Garrett, acpi4asus-user,
platform-driver-x86, linux-kernel
Cc: Chris Bagwell, tyson.chen, Martin Pitt, Keng-Yu Lin
Signed-off-by: Keng-Yu Lin <keng-yu.lin@canonical.com>
---
drivers/platform/x86/eeepc-wmi.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/platform/x86/eeepc-wmi.c b/drivers/platform/x86/eeepc-wmi.c
index d399775..a9f75e2 100644
--- a/drivers/platform/x86/eeepc-wmi.c
+++ b/drivers/platform/x86/eeepc-wmi.c
@@ -113,7 +113,7 @@ static const struct key_entry eeepc_wmi_keymap[] = {
{ KE_KEY, 0x32, { KEY_MUTE } },
{ KE_KEY, 0x5c, { KEY_F15 } }, /* Power Gear key */
{ KE_KEY, 0x5d, { KEY_WLAN } },
- { KE_KEY, 0x6b, { KEY_F13 } }, /* Disable Touchpad */
+ { KE_KEY, 0x6b, { KEY_TOUCHPAD_TOGGLE } }, /* Toggle Touchpad */
{ KE_KEY, 0x82, { KEY_CAMERA } },
{ KE_KEY, 0x83, { KEY_CAMERA_ZOOMIN } },
{ KE_KEY, 0x88, { KEY_WLAN } },
--
1.7.1
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH v2] eeepc-wmi: set the touchpad toggle key code to KEY_TOUCHPAD_TOGGLE
@ 2011-03-01 4:56 ` Keng-Yu Lin
0 siblings, 0 replies; 17+ messages in thread
From: Keng-Yu Lin @ 2011-03-01 4:56 UTC (permalink / raw)
To: Corentin Chary, Matthew Garrett,
acpi4asus-user-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
platform-driver-x86-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Chris Bagwell, tyson.chen-Z7WLFzj8eWMS+FvcfC7Uqw, Keng-Yu Lin,
Martin Pitt
Signed-off-by: Keng-Yu Lin <keng-yu.lin-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
---
drivers/platform/x86/eeepc-wmi.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/platform/x86/eeepc-wmi.c b/drivers/platform/x86/eeepc-wmi.c
index d399775..a9f75e2 100644
--- a/drivers/platform/x86/eeepc-wmi.c
+++ b/drivers/platform/x86/eeepc-wmi.c
@@ -113,7 +113,7 @@ static const struct key_entry eeepc_wmi_keymap[] = {
{ KE_KEY, 0x32, { KEY_MUTE } },
{ KE_KEY, 0x5c, { KEY_F15 } }, /* Power Gear key */
{ KE_KEY, 0x5d, { KEY_WLAN } },
- { KE_KEY, 0x6b, { KEY_F13 } }, /* Disable Touchpad */
+ { KE_KEY, 0x6b, { KEY_TOUCHPAD_TOGGLE } }, /* Toggle Touchpad */
{ KE_KEY, 0x82, { KEY_CAMERA } },
{ KE_KEY, 0x83, { KEY_CAMERA_ZOOMIN } },
{ KE_KEY, 0x88, { KEY_WLAN } },
--
1.7.1
------------------------------------------------------------------------------
Free Software Download: Index, Search & Analyze Logs and other IT data in
Real-Time with Splunk. Collect, index and harness all the fast moving IT data
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business
insights. http://p.sf.net/sfu/splunk-dev2dev
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH v2] eeepc-wmi: set the touchpad toggle key code to KEY_TOUCHPAD_TOGGLE
@ 2011-03-11 17:31 ` Matthew Garrett
0 siblings, 0 replies; 17+ messages in thread
From: Matthew Garrett @ 2011-03-11 17:31 UTC (permalink / raw)
To: Keng-Yu Lin
Cc: Corentin Chary, acpi4asus-user, platform-driver-x86,
linux-kernel, Chris Bagwell, tyson.chen, Martin Pitt
Applied, thanks.
--
Matthew Garrett | mjg59@srcf.ucam.org
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2] eeepc-wmi: set the touchpad toggle key code to KEY_TOUCHPAD_TOGGLE
@ 2011-03-11 17:31 ` Matthew Garrett
0 siblings, 0 replies; 17+ messages in thread
From: Matthew Garrett @ 2011-03-11 17:31 UTC (permalink / raw)
To: Keng-Yu Lin
Cc: acpi4asus-user-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f, Chris Bagwell,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
platform-driver-x86-u79uwXL29TY76Z2rM5mHXA,
tyson.chen-Z7WLFzj8eWMS+FvcfC7Uqw, Martin Pitt
Applied, thanks.
--
Matthew Garrett | mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org
------------------------------------------------------------------------------
Colocation vs. Managed Hosting
A question and answer guide to determining the best fit
for your organization - today and in the future.
http://p.sf.net/sfu/internap-sfd2d
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2011-03-11 17:31 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-25 5:19 [PATCH] eeepc-wmi: set the touchpad toggle key code to F21 Keng-Yu Lin
2011-02-25 5:19 ` Keng-Yu Lin
2011-02-25 6:52 ` Corentin Chary
2011-02-25 7:18 ` Dmitry Torokhov
2011-02-25 9:22 ` Keng-Yü Lin
2011-02-25 9:33 ` Dmitry Torokhov
2011-02-25 9:33 ` Dmitry Torokhov
2011-02-25 9:53 ` Martin Pitt
2011-02-25 15:14 ` Chris Bagwell
2011-02-25 22:25 ` Joey Lee
2011-02-25 22:25 ` Joey Lee
2011-03-01 3:29 ` Keng-Yü Lin
2011-02-25 14:12 ` Matthew Garrett
2011-03-01 4:56 ` [PATCH v2] eeepc-wmi: set the touchpad toggle key code to KEY_TOUCHPAD_TOGGLE Keng-Yu Lin
2011-03-01 4:56 ` Keng-Yu Lin
2011-03-11 17:31 ` Matthew Garrett
2011-03-11 17:31 ` Matthew Garrett
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.