public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Re: Fw: ati-remote strangeness from 2.6.12 onwards
       [not found] <20050730173253.693484a2.akpm@osdl.org>
@ 2005-08-02  5:04 ` mdew
  2005-08-03  5:54   ` Pavel Machek
  2005-08-04 17:15   ` Andrew Morton
  0 siblings, 2 replies; 10+ messages in thread
From: mdew @ 2005-08-02  5:04 UTC (permalink / raw)
  To: linux-kernel; +Cc: linux-usb-devel, Vojtech Pavlik, Dmitry Torokhov

I discovered a minor change in 2.6.10-mm1, changing this value back
corrects the "ok" button issue.


diff -urN linux/drivers/usb/input/ati_remote.c
linux-2.6.11/drivers/usb/input/ati_remote.c
--- linux/drivers/usb/input/ati_remote.c        2005-08-02
17:56:26.000000000 +1200
+++ linux-2.6.11/drivers/usb/input/ati_remote.c 2005-08-02
17:54:34.000000000 +1200
@@ -263,7 +263,7 @@
        {KIND_FILTERED, 0xe4, 0x1f, EV_KEY, KEY_RIGHT, 1},      /* right */
        {KIND_FILTERED, 0xe7, 0x22, EV_KEY, KEY_DOWN, 1},       /* down */
        {KIND_FILTERED, 0xdf, 0x1a, EV_KEY, KEY_UP, 1},         /* up */
-       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_ENTER, 1},      /* "OK" */
+       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_OK, 1},         /* "OK" */
        {KIND_FILTERED, 0xce, 0x09, EV_KEY, KEY_VOLUMEDOWN, 1}, /* VOL + */
        {KIND_FILTERED, 0xcd, 0x08, EV_KEY, KEY_VOLUMEUP, 1},   /* VOL - */
        {KIND_FILTERED, 0xcf, 0x0a, EV_KEY, KEY_MUTE, 1},       /* MUTE  */


On 7/31/05, Andrew Morton <akpm@osdl.org> wrote:
> 
> 
> Begin forwarded message:
> 
> Date: Sun, 31 Jul 2005 12:18:23 +1200
> From: mdew <some.nzguy@gmail.com>
> To: linux-kernel <linux-kernel@vger.kernel.org>
> Subject: ati-remote strangeness from 2.6.12 onwards
> 
> 
> using 2.6.11 everything works fine, Upgrading too 2.6.13-rc3 I noticed 2 errors,
> 
> (1) When setting the HZ rating too 250 or 100 will cause the driver to
> excessfully repeat keys/accelerate when pressing a button, making it
> unusable :(
> 
> (2) the "Ok" button no longer works in anything after and including
> 2.6.12-rc1 (I've tested upto 2.6.13-rc3), 2.6.11 works fine. xbindkeys
> doesnt register any "ok" key presses on 2.6.12-rc1 onwards.
> 
> 2.6.11 xbindkeys responses (nothing shows up in -rc1)
> 
> mediabawx2:~# xbindkeys -mk
> Press combination of keys or/and click under the window.
> You can use one of the two lines after "NoCommand"
> in $HOME/.xbindkeysrc to bind a key.
> 
> --- Press "q" to stop. ---
> "NoCommand"
> m:0x0 + c:36
> Return
> "NoCommand"
> m:0x0 + c:36
> Return
> 
> 
> Thanks :)
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Fw: ati-remote strangeness from 2.6.12 onwards
  2005-08-02  5:04 ` Fw: ati-remote strangeness from 2.6.12 onwards mdew
@ 2005-08-03  5:54   ` Pavel Machek
  2005-08-03 10:11     ` mdew
  2005-08-04 17:15   ` Andrew Morton
  1 sibling, 1 reply; 10+ messages in thread
From: Pavel Machek @ 2005-08-03  5:54 UTC (permalink / raw)
  To: mdew; +Cc: linux-kernel, linux-usb-devel, Vojtech Pavlik, Dmitry Torokhov

Hi!

> I discovered a minor change in 2.6.10-mm1, changing this value back
> corrects the "ok" button issue.
> 
> 
> diff -urN linux/drivers/usb/input/ati_remote.c
> linux-2.6.11/drivers/usb/input/ati_remote.c
> --- linux/drivers/usb/input/ati_remote.c        2005-08-02
> 17:56:26.000000000 +1200
> +++ linux-2.6.11/drivers/usb/input/ati_remote.c 2005-08-02
> 17:54:34.000000000 +1200
> @@ -263,7 +263,7 @@
>         {KIND_FILTERED, 0xe4, 0x1f, EV_KEY, KEY_RIGHT, 1},      /* right */
>         {KIND_FILTERED, 0xe7, 0x22, EV_KEY, KEY_DOWN, 1},       /* down */
>         {KIND_FILTERED, 0xdf, 0x1a, EV_KEY, KEY_UP, 1},         /* up */
> -       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_ENTER, 1},      /* "OK" */
> +       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_OK, 1},         /* "OK" */
>         {KIND_FILTERED, 0xce, 0x09, EV_KEY, KEY_VOLUMEDOWN, 1}, /* VOL + */
>         {KIND_FILTERED, 0xcd, 0x08, EV_KEY, KEY_VOLUMEUP, 1},   /* VOL - */
>         {KIND_FILTERED, 0xcf, 0x0a, EV_KEY, KEY_MUTE, 1},       /* MUTE  */

I'd say that KEY_ENTER is perhaps more logical there? It is certainly
more usefull than "OK" key.
								Pavel

-- 
teflon -- maybe it is a trademark, but it should not be.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Fw: ati-remote strangeness from 2.6.12 onwards
  2005-08-03  5:54   ` Pavel Machek
@ 2005-08-03 10:11     ` mdew
  2005-08-03 11:41       ` [linux-usb-devel] " Frank Loeffler
  0 siblings, 1 reply; 10+ messages in thread
From: mdew @ 2005-08-03 10:11 UTC (permalink / raw)
  To: Pavel Machek
  Cc: linux-kernel, linux-usb-devel, Vojtech Pavlik, Dmitry Torokhov

[-- Attachment #1: Type: text/plain, Size: 2969 bytes --]

Hi Pavel,

Further testing, I initated xmodmap -e "keycode $X = p" (where X was
10->255), so theoritcally, all the buttons on the ati-remote should be
mapped to "p". I found the TV Button, The DVD Button, the CH-/+ and
the OK Button all non-working, every other button produced the "p".
xbindkeys -mk doesnt respond to those particular keys either.

these changes occured after 2.6.11, which have caused it not to
respond correctly

-	{KIND_FILTERED, 0xc8, 0x03, EV_KEY, KEY_PROG1, 1},      /* TV */
-	{KIND_FILTERED, 0xc9, 0x04, EV_KEY, KEY_PROG2, 1},      /* DVD */
+	{KIND_FILTERED, 0xc8, 0x03, EV_KEY, KEY_TV, 1},         /* TV */
+	{KIND_FILTERED, 0xc9, 0x04, EV_KEY, KEY_DVD, 1},        /* DVD */
-	{KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_ENTER, 1},      /* "OK" */
+	{KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_OK, 1},         /* "OK" */

couldnt get the channelup/down to work correctly, even reversing the
orginal patch doesnt help.

-	{KIND_FILTERED, 0xd1, 0x0c, EV_KEY, KEY_CHANNELUP, 1},  /* CH + */
-	{KIND_FILTERED, 0xd0, 0x0b, EV_KEY, KEY_CHANNELDOWN, 1},/* CH - */
+	{KIND_FILTERED, 0xd0, 0x0b, EV_KEY, KEY_CHANNELUP, 1},  /* CH + */
+	{KIND_FILTERED, 0xd1, 0x0c, EV_KEY, KEY_CHANNELDOWN, 1},/* CH - */

however this one change is fine.

-	{KIND_FILTERED, 0xea, 0x25, EV_KEY, KEY_PLAYCD, 1},     /* ( >) */
+	{KIND_FILTERED, 0xea, 0x25, EV_KEY, KEY_PLAY, 1},       /* ( >) */


My orginal patch was incorrect, heres a revised patch, fixes the OK
button and the TV/DVD button issue. Would be nice to get it included
into 2.6.13 ;-)

On 8/3/05, Pavel Machek <pavel@suse.cz> wrote:
> Hi!
> 
> > I discovered a minor change in 2.6.10-mm1, changing this value back
> > corrects the "ok" button issue.
> >
> >
> > diff -urN linux/drivers/usb/input/ati_remote.c
> > linux-2.6.11/drivers/usb/input/ati_remote.c
> > --- linux/drivers/usb/input/ati_remote.c        2005-08-02
> > 17:56:26.000000000 +1200
> > +++ linux-2.6.11/drivers/usb/input/ati_remote.c 2005-08-02
> > 17:54:34.000000000 +1200
> > @@ -263,7 +263,7 @@
> >         {KIND_FILTERED, 0xe4, 0x1f, EV_KEY, KEY_RIGHT, 1},      /* right */
> >         {KIND_FILTERED, 0xe7, 0x22, EV_KEY, KEY_DOWN, 1},       /* down */
> >         {KIND_FILTERED, 0xdf, 0x1a, EV_KEY, KEY_UP, 1},         /* up */
> > -       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_ENTER, 1},      /* "OK" */
> > +       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_OK, 1},         /* "OK" */
> >         {KIND_FILTERED, 0xce, 0x09, EV_KEY, KEY_VOLUMEDOWN, 1}, /* VOL + */
> >         {KIND_FILTERED, 0xcd, 0x08, EV_KEY, KEY_VOLUMEUP, 1},   /* VOL - */
> >         {KIND_FILTERED, 0xcf, 0x0a, EV_KEY, KEY_MUTE, 1},       /* MUTE  */
> 
> I'd say that KEY_ENTER is perhaps more logical there? It is certainly
> more usefull than "OK" key.
>                                                                 Pavel
> 
> --
> teflon -- maybe it is a trademark, but it should not be.
>

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: ati-remotefix.patch --]
[-- Type: text/x-patch; name="ati-remotefix.patch", Size: 1571 bytes --]

diff -ruN linux-2.6.12/drivers/usb/input/ati_remote.c linux-2.6.12-modified/drivers/usb/input/ati_remote.c
--- linux-2.6.12/drivers/usb/input/ati_remote.c	2005-06-18 07:48:29.000000000 +1200
+++ linux-2.6.12-modified/drivers/usb/input/ati_remote.c	2005-08-03 09:54:48.000000000 +1200
@@ -252,8 +252,8 @@
 	{KIND_FILTERED, 0xdd, 0x18, EV_KEY, KEY_KPENTER, 1},    /* "check" */
 	{KIND_FILTERED, 0xdb, 0x16, EV_KEY, KEY_MENU, 1},       /* "menu" */
 	{KIND_FILTERED, 0xc7, 0x02, EV_KEY, KEY_POWER, 1},      /* Power */
-	{KIND_FILTERED, 0xc8, 0x03, EV_KEY, KEY_TV, 1},         /* TV */
-	{KIND_FILTERED, 0xc9, 0x04, EV_KEY, KEY_DVD, 1},        /* DVD */
+	{KIND_FILTERED, 0xc8, 0x03, EV_KEY, KEY_PROG1, 1},      /* TV */
+	{KIND_FILTERED, 0xc9, 0x04, EV_KEY, KEY_PROG2, 1},      /* DVD */
 	{KIND_FILTERED, 0xca, 0x05, EV_KEY, KEY_WWW, 1},        /* WEB */
 	{KIND_FILTERED, 0xcb, 0x06, EV_KEY, KEY_BOOKMARKS, 1},  /* "book" */
 	{KIND_FILTERED, 0xcc, 0x07, EV_KEY, KEY_EDIT, 1},       /* "hand" */
@@ -263,7 +263,7 @@
 	{KIND_FILTERED, 0xe4, 0x1f, EV_KEY, KEY_RIGHT, 1},      /* right */
 	{KIND_FILTERED, 0xe7, 0x22, EV_KEY, KEY_DOWN, 1},       /* down */
 	{KIND_FILTERED, 0xdf, 0x1a, EV_KEY, KEY_UP, 1},         /* up */
-	{KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_OK, 1},         /* "OK" */
+	{KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_ENTER, 1},      /* "OK" */
 	{KIND_FILTERED, 0xce, 0x09, EV_KEY, KEY_VOLUMEDOWN, 1}, /* VOL + */
 	{KIND_FILTERED, 0xcd, 0x08, EV_KEY, KEY_VOLUMEUP, 1},   /* VOL - */
 	{KIND_FILTERED, 0xcf, 0x0a, EV_KEY, KEY_MUTE, 1},       /* MUTE  */

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-usb-devel] Re: Fw: ati-remote strangeness from 2.6.12 onwards
  2005-08-03 10:11     ` mdew
@ 2005-08-03 11:41       ` Frank Loeffler
  2005-08-04  1:15         ` Ryan Brown
  2005-08-04 20:27         ` Pavel Machek
  0 siblings, 2 replies; 10+ messages in thread
From: Frank Loeffler @ 2005-08-03 11:41 UTC (permalink / raw)
  To: mdew
  Cc: Pavel Machek, linux-kernel, linux-usb-devel, Vojtech Pavlik,
	Dmitry Torokhov

Hi 'mdew' (you do have a name, do you?),

mdew wrote:
> mapped to "p". I found the TV Button, The DVD Button, the CH-/+ and
> the OK Button all non-working, every other button produced the "p".

Could you please try 'showkey -s' from a console on all of those keys?

Pavel: I would think that 'more useful' is not really the same as 
'correct'. If you find it useful to map this key to 'ENTER', so you 
should remap it in userspace. It should not be KEY_ENTER in the kernel 
for at least two reasons:

- The key is labled 'ok' (and not enter). I assume the code KEY_OK is
   made for exactly that kind of key and certain applications might
   look for exactly this code.
- You might want to differentiate between this key and the ENTER key
   of your keyboard, at least I do. If the kernel is sending the same
   code for both keys, this is not possible in userspace.

Frank


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-usb-devel] Re: Fw: ati-remote strangeness from 2.6.12 onwards
  2005-08-03 11:41       ` [linux-usb-devel] " Frank Loeffler
@ 2005-08-04  1:15         ` Ryan Brown
  2005-08-04 20:27         ` Pavel Machek
  1 sibling, 0 replies; 10+ messages in thread
From: Ryan Brown @ 2005-08-04  1:15 UTC (permalink / raw)
  To: Frank Loeffler
  Cc: Pavel Machek, linux-kernel, linux-usb-devel, Vojtech Pavlik,
	Dmitry Torokhov

> mdew wrote:
> > mapped to "p". I found the TV Button, The DVD Button, the CH-/+ and
> > the OK Button all non-working, every other button produced the "p".
> 
> Could you please try 'showkey -s' from a console on all of those keys?

Without my patch, nothing shows up when pressing OK, TV and DVD, in showkey -s

# with my patch
mediabawx2:~# showkey -s
kb mode was XLATE

press any key (program terminates after 10s of last keypress)...
0x1c 0x9c    - OK
0xe0 0x1f 0xe0 0x9f   - TV
0xe0 0x17 0xe0 0x97   - DVD

> Pavel: I would think that 'more useful' is not really the same as
> 'correct'. If you find it useful to map this key to 'ENTER', so you
> should remap it in userspace. It should not be KEY_ENTER in the kernel
> for at least two reasons:
> 
> - The key is labled 'ok' (and not enter). I assume the code KEY_OK is
>    made for exactly that kind of key and certain applications might
>    look for exactly this code.
> - You might want to differentiate between this key and the ENTER key
>    of your keyboard, at least I do. If the kernel is sending the same
>    code for both keys, this is not possible in userspace.
> 
> Frank
> 
>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Fw: ati-remote strangeness from 2.6.12 onwards
  2005-08-02  5:04 ` Fw: ati-remote strangeness from 2.6.12 onwards mdew
  2005-08-03  5:54   ` Pavel Machek
@ 2005-08-04 17:15   ` Andrew Morton
  2005-08-04 22:23     ` Ryan Brown
  1 sibling, 1 reply; 10+ messages in thread
From: Andrew Morton @ 2005-08-04 17:15 UTC (permalink / raw)
  To: mdew; +Cc: linux-kernel, linux-usb-devel, vojtech, dtor_core

mdew <some.nzguy@gmail.com> wrote:
>
> I discovered a minor change in 2.6.10-mm1, changing this value back
>  corrects the "ok" button issue.
> 
> 
>  diff -urN linux/drivers/usb/input/ati_remote.c
>  linux-2.6.11/drivers/usb/input/ati_remote.c
>  --- linux/drivers/usb/input/ati_remote.c        2005-08-02
>  17:56:26.000000000 +1200
>  +++ linux-2.6.11/drivers/usb/input/ati_remote.c 2005-08-02
>  17:54:34.000000000 +1200
>  @@ -263,7 +263,7 @@
>          {KIND_FILTERED, 0xe4, 0x1f, EV_KEY, KEY_RIGHT, 1},      /* right */
>          {KIND_FILTERED, 0xe7, 0x22, EV_KEY, KEY_DOWN, 1},       /* down */
>          {KIND_FILTERED, 0xdf, 0x1a, EV_KEY, KEY_UP, 1},         /* up */
>  -       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_ENTER, 1},      /* "OK" */
>  +       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_OK, 1},         /* "OK" */
>          {KIND_FILTERED, 0xce, 0x09, EV_KEY, KEY_VOLUMEDOWN, 1}, /* VOL + */
>          {KIND_FILTERED, 0xcd, 0x08, EV_KEY, KEY_VOLUMEUP, 1},   /* VOL - */
>          {KIND_FILTERED, 0xcf, 0x0a, EV_KEY, KEY_MUTE, 1},       /* MUTE  */

This appears to be already applied in 2.6.12-rc5.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-usb-devel] Re: Fw: ati-remote strangeness from 2.6.12 onwards
  2005-08-03 11:41       ` [linux-usb-devel] " Frank Loeffler
  2005-08-04  1:15         ` Ryan Brown
@ 2005-08-04 20:27         ` Pavel Machek
  1 sibling, 0 replies; 10+ messages in thread
From: Pavel Machek @ 2005-08-04 20:27 UTC (permalink / raw)
  To: Frank Loeffler
  Cc: mdew, linux-kernel, linux-usb-devel, Vojtech Pavlik,
	Dmitry Torokhov

Hi!

> - You might want to differentiate between this key and the ENTER key
>   of your keyboard, at least I do. If the kernel is sending the same
>   code for both keys, this is not possible in userspace.

No, I think that you can still diferentiate between them ... they come
from different keyboard after all. See /dev/input/event*.

								Pavel
-- 
teflon -- maybe it is a trademark, but it should not be.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Fw: ati-remote strangeness from 2.6.12 onwards
  2005-08-04 17:15   ` Andrew Morton
@ 2005-08-04 22:23     ` Ryan Brown
  2005-08-04 22:44       ` Andrew Morton
  0 siblings, 1 reply; 10+ messages in thread
From: Ryan Brown @ 2005-08-04 22:23 UTC (permalink / raw)
  To: Andrew Morton; +Cc: linux-kernel, linux-usb-devel, vojtech, dtor_core

Sorry Andrew, but the diff was incorrectly made, the updated patch,
reverts the changes too TV, DVD and OK Button, from 2.6.12-rc1.

diff -ruN linux-2.6.12/drivers/usb/input/ati_remote.c
linux-2.6.12-modified/drivers/usb/input/ati_remote.c
--- linux-2.6.12/drivers/usb/input/ati_remote.c	2005-06-18
07:48:29.000000000 +1200
+++ linux-2.6.12-modified/drivers/usb/input/ati_remote.c	2005-08-03
09:54:48.000000000 +1200
@@ -252,8 +252,8 @@
 	{KIND_FILTERED, 0xdd, 0x18, EV_KEY, KEY_KPENTER, 1},    /* "check" */
 	{KIND_FILTERED, 0xdb, 0x16, EV_KEY, KEY_MENU, 1},       /* "menu" */
 	{KIND_FILTERED, 0xc7, 0x02, EV_KEY, KEY_POWER, 1},      /* Power */
-	{KIND_FILTERED, 0xc8, 0x03, EV_KEY, KEY_TV, 1},         /* TV */
-	{KIND_FILTERED, 0xc9, 0x04, EV_KEY, KEY_DVD, 1},        /* DVD */
+	{KIND_FILTERED, 0xc8, 0x03, EV_KEY, KEY_PROG1, 1},      /* TV */
+	{KIND_FILTERED, 0xc9, 0x04, EV_KEY, KEY_PROG2, 1},      /* DVD */
 	{KIND_FILTERED, 0xca, 0x05, EV_KEY, KEY_WWW, 1},        /* WEB */
 	{KIND_FILTERED, 0xcb, 0x06, EV_KEY, KEY_BOOKMARKS, 1},  /* "book" */
 	{KIND_FILTERED, 0xcc, 0x07, EV_KEY, KEY_EDIT, 1},       /* "hand" */
@@ -263,7 +263,7 @@
 	{KIND_FILTERED, 0xe4, 0x1f, EV_KEY, KEY_RIGHT, 1},      /* right */
 	{KIND_FILTERED, 0xe7, 0x22, EV_KEY, KEY_DOWN, 1},       /* down */
 	{KIND_FILTERED, 0xdf, 0x1a, EV_KEY, KEY_UP, 1},         /* up */
-	{KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_OK, 1},         /* "OK" */
+	{KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_ENTER, 1},      /* "OK" */
 	{KIND_FILTERED, 0xce, 0x09, EV_KEY, KEY_VOLUMEDOWN, 1}, /* VOL + */
 	{KIND_FILTERED, 0xcd, 0x08, EV_KEY, KEY_VOLUMEUP, 1},   /* VOL - */
 	{KIND_FILTERED, 0xcf, 0x0a, EV_KEY, KEY_MUTE, 1},       /* MUTE  */


On 8/5/05, Andrew Morton <akpm@osdl.org> wrote:
> mdew <some.nzguy@gmail.com> wrote:
> >
> > I discovered a minor change in 2.6.10-mm1, changing this value back
> >  corrects the "ok" button issue.
> >
> >
> >  diff -urN linux/drivers/usb/input/ati_remote.c
> >  linux-2.6.11/drivers/usb/input/ati_remote.c
> >  --- linux/drivers/usb/input/ati_remote.c        2005-08-02
> >  17:56:26.000000000 +1200
> >  +++ linux-2.6.11/drivers/usb/input/ati_remote.c 2005-08-02
> >  17:54:34.000000000 +1200
> >  @@ -263,7 +263,7 @@
> >          {KIND_FILTERED, 0xe4, 0x1f, EV_KEY, KEY_RIGHT, 1},      /* right */
> >          {KIND_FILTERED, 0xe7, 0x22, EV_KEY, KEY_DOWN, 1},       /* down */
> >          {KIND_FILTERED, 0xdf, 0x1a, EV_KEY, KEY_UP, 1},         /* up */
> >  -       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_ENTER, 1},      /* "OK" */
> >  +       {KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_OK, 1},         /* "OK" */
> >          {KIND_FILTERED, 0xce, 0x09, EV_KEY, KEY_VOLUMEDOWN, 1}, /* VOL + */
> >          {KIND_FILTERED, 0xcd, 0x08, EV_KEY, KEY_VOLUMEUP, 1},   /* VOL - */
> >          {KIND_FILTERED, 0xcf, 0x0a, EV_KEY, KEY_MUTE, 1},       /* MUTE  */
> 
> This appears to be already applied in 2.6.12-rc5.
>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Fw: ati-remote strangeness from 2.6.12 onwards
  2005-08-04 22:23     ` Ryan Brown
@ 2005-08-04 22:44       ` Andrew Morton
  2005-08-05  6:46         ` [linux-usb-devel] " Frank Loeffler
  0 siblings, 1 reply; 10+ messages in thread
From: Andrew Morton @ 2005-08-04 22:44 UTC (permalink / raw)
  To: Ryan Brown; +Cc: linux-kernel, linux-usb-devel, vojtech, dtor_core

Ryan Brown <some.nzguy@gmail.com> wrote:
>
> Sorry Andrew, but the diff was incorrectly made, the updated patch,
> reverts the changes too TV, DVD and OK Button, from 2.6.12-rc1.

hm, this was about 20 patches and 300 emails ago and I've forgotten what
we're discussing.  For poor old scatterbrains it really helps if people can
maintain a description of their patch alongside the patch itself.

> diff -ruN linux-2.6.12/drivers/usb/input/ati_remote.c
> linux-2.6.12-modified/drivers/usb/input/ati_remote.c
> --- linux-2.6.12/drivers/usb/input/ati_remote.c	2005-06-18
> 07:48:29.000000000 +1200
> +++ linux-2.6.12-modified/drivers/usb/input/ati_remote.c	2005-08-03
> 09:54:48.000000000 +1200
> @@ -252,8 +252,8 @@
>  	{KIND_FILTERED, 0xdd, 0x18, EV_KEY, KEY_KPENTER, 1},    /* "check" */
>  	{KIND_FILTERED, 0xdb, 0x16, EV_KEY, KEY_MENU, 1},       /* "menu" */
>  	{KIND_FILTERED, 0xc7, 0x02, EV_KEY, KEY_POWER, 1},      /* Power */
> -	{KIND_FILTERED, 0xc8, 0x03, EV_KEY, KEY_TV, 1},         /* TV */
> -	{KIND_FILTERED, 0xc9, 0x04, EV_KEY, KEY_DVD, 1},        /* DVD */
> +	{KIND_FILTERED, 0xc8, 0x03, EV_KEY, KEY_PROG1, 1},      /* TV */
> +	{KIND_FILTERED, 0xc9, 0x04, EV_KEY, KEY_PROG2, 1},      /* DVD */
>  	{KIND_FILTERED, 0xca, 0x05, EV_KEY, KEY_WWW, 1},        /* WEB */
>  	{KIND_FILTERED, 0xcb, 0x06, EV_KEY, KEY_BOOKMARKS, 1},  /* "book" */
>  	{KIND_FILTERED, 0xcc, 0x07, EV_KEY, KEY_EDIT, 1},       /* "hand" */
> @@ -263,7 +263,7 @@
>  	{KIND_FILTERED, 0xe4, 0x1f, EV_KEY, KEY_RIGHT, 1},      /* right */
>  	{KIND_FILTERED, 0xe7, 0x22, EV_KEY, KEY_DOWN, 1},       /* down */
>  	{KIND_FILTERED, 0xdf, 0x1a, EV_KEY, KEY_UP, 1},         /* up */
> -	{KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_OK, 1},         /* "OK" */
> +	{KIND_FILTERED, 0xe3, 0x1e, EV_KEY, KEY_ENTER, 1},      /* "OK" */
>  	{KIND_FILTERED, 0xce, 0x09, EV_KEY, KEY_VOLUMEDOWN, 1}, /* VOL + */
>  	{KIND_FILTERED, 0xcd, 0x08, EV_KEY, KEY_VOLUMEUP, 1},   /* VOL - */
>  	{KIND_FILTERED, 0xcf, 0x0a, EV_KEY, KEY_MUTE, 1},       /* MUTE  */

IOW: what does this (wordwrapped!) patch do?


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [linux-usb-devel] Re: Fw: ati-remote strangeness from 2.6.12 onwards
  2005-08-04 22:44       ` Andrew Morton
@ 2005-08-05  6:46         ` Frank Loeffler
  0 siblings, 0 replies; 10+ messages in thread
From: Frank Loeffler @ 2005-08-05  6:46 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Ryan Brown, linux-kernel, linux-usb-devel, vojtech, dtor_core

Hi,

Andrew Morton wrote:
> IOW: what does this (wordwrapped!) patch do?

It changes the keycode the kernel is sending for three keys. For normal 
keyboards there is usually no argument to which keycode to send. An 'a' 
would send the keycoe for an 'a'. This however is a remote control. The 
keys are labled 'OK', 'TV' and 'DVD'. Therefore the kernel currently 
sends the keycodes KEY_OK, KEY_TV and KEY_DVD. The patch changes this to 
KEY_ENTER, KEY_PROG1 and KEY_PROG2.

I do not know about the motivation of this patch, as the kernel 
currently _does_ send keycodes, maybe just not the ones the some users 
might want. IMHO this is an issue of remapping the keycodes in userspace 
and I would like to leave the kernel-codes alone. However, I might not 
see the whole problem here because it is working fine for me.

Btw, Pavel:

 > No, I think that you can still diferentiate between them ... they come
 > from different keyboard after all. See /dev/input/event*.

How can I tell the consoles of linux which keyboard to use? So far they 
all use all keyboards (which is my usual keyboard mixed with the remote 
control keys). (Yes, I searched google extensivly and no, I do not have 
X on that machine.)

Frank


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2005-08-05  6:49 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20050730173253.693484a2.akpm@osdl.org>
2005-08-02  5:04 ` Fw: ati-remote strangeness from 2.6.12 onwards mdew
2005-08-03  5:54   ` Pavel Machek
2005-08-03 10:11     ` mdew
2005-08-03 11:41       ` [linux-usb-devel] " Frank Loeffler
2005-08-04  1:15         ` Ryan Brown
2005-08-04 20:27         ` Pavel Machek
2005-08-04 17:15   ` Andrew Morton
2005-08-04 22:23     ` Ryan Brown
2005-08-04 22:44       ` Andrew Morton
2005-08-05  6:46         ` [linux-usb-devel] " Frank Loeffler

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox