Re: [PATCH 08/13] [media] rc-winfast: Fix the keycode tables

[Balint Marton <cus@xxxxxxxxxx>]

> Hi,
>
> > One of the remotes has a picture available at:
> > 	http://lirc.sourceforge.net/remotes/leadtek/Y04G0004.jpg
> >
> > As there's one variant with a set direction keys plus vol/chann
> > keys, and the same table is used for both models, change it to
> > represent all keys, avoiding the usage of weird function keys.
> >
> > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> >
> > diff --git a/drivers/media/rc/keymaps/rc-winfast.c 
> > b/drivers/media/rc/keymaps/rc-winfast.c
> > index 2747db4..0062ca2 100644
> > --- a/drivers/media/rc/keymaps/rc-winfast.c
> > +++ b/drivers/media/rc/keymaps/rc-winfast.c
> > @@ -27,15 +27,15 @@ static struct rc_map_table winfast[] = {
> > 	{ 0x0e, KEY_8 },
> > 	{ 0x0f, KEY_9 },
> >
> > -	{ 0x00, KEY_POWER },
> > +	{ 0x00, KEY_POWER2 },
> > 	{ 0x1b, KEY_AUDIO },		/* Audio Source */
> > 	{ 0x02, KEY_TUNER },		/* TV/FM, not on Y0400052 */
> > 	{ 0x1e, KEY_VIDEO },		/* Video Source */
> > 	{ 0x16, KEY_INFO },		/* Display information */
> > -	{ 0x04, KEY_VOLUMEUP },
> > -	{ 0x08, KEY_VOLUMEDOWN },
> > -	{ 0x0c, KEY_CHANNELUP },
> > -	{ 0x10, KEY_CHANNELDOWN },
> > +	{ 0x04, KEY_LEFT },
> > +	{ 0x08, KEY_RIGHT },
> > +	{ 0x0c, KEY_UP },
> > +	{ 0x10, KEY_DOWN },
>
> Left and right key is now swapped on my remote. (Which is exactly the same 
> model by the way that was shown in Y04G0004.jpg.) Could you please swap the 
> two keys?
>
> Regards,
>  Marton

Sorry, just saw the fix commited to linux-next :)

Regards,
  Marton
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html