* 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: [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-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: 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