[media] ati_remote: Sort buttons in top-to-bottom order
authorGeorge Spelvin <linux@horizon.com>
Sun, 11 May 2014 11:16:56 +0000 (08:16 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Sat, 26 Jul 2014 02:52:59 +0000 (23:52 -0300)
Since numerical order corresponds to top-left-to-bottom-right
order on the remote, this makes the table easier to read.

Signed-off-by: George Spelvin <linux@horizon.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/rc/keymaps/rc-ati-x10.c

index 81506440eded45d7b9db84462e9b0874eb38ca61..4e2cbbafe97adba05b61d141060baa8805ae50ec 100644 (file)
 #include <media/rc-map.h>
 
 static struct rc_map_table ati_x10[] = {
+       /* keyboard - Above the cursor pad */
+       { 0x00, KEY_A },
+       { 0x01, KEY_B },
+       { 0x02, KEY_POWER },      /* Power */
+
+       { 0x03, KEY_TV },         /* TV */
+       { 0x04, KEY_DVD },        /* DVD */
+       { 0x05, KEY_WWW },        /* WEB */
+       { 0x06, KEY_BOOKMARKS },  /* "book" */
+       { 0x07, KEY_EDIT },       /* "hand" */
+       /* Below the cursor pad */
+       { 0x09, KEY_VOLUMEDOWN }, /* VOL + */
+       { 0x08, KEY_VOLUMEUP },   /* VOL - */
+       { 0x0a, KEY_MUTE },       /* MUTE  */
+       { 0x0b, KEY_CHANNELUP },  /* CH + */
+       { 0x0c, KEY_CHANNELDOWN },/* CH - */
+
        { 0x0d, KEY_1 },
        { 0x0e, KEY_2 },
        { 0x0f, KEY_3 },
@@ -36,42 +53,36 @@ static struct rc_map_table ati_x10[] = {
        { 0x13, KEY_7 },
        { 0x14, KEY_8 },
        { 0x15, KEY_9 },
+       { 0x16, KEY_MENU },       /* "menu" */
        { 0x17, KEY_0 },
-       { 0x00, KEY_A },
-       { 0x01, KEY_B },
+       { 0x18, KEY_KPENTER },    /* "check" */
+
+       /* DVD navigation buttons */
        { 0x19, KEY_C },
+       { 0x1a, KEY_UP },         /* up */
        { 0x1b, KEY_D },
-       { 0x21, KEY_E },
-       { 0x23, KEY_F },
 
-       { 0x18, KEY_KPENTER },    /* "check" */
-       { 0x16, KEY_MENU },       /* "menu" */
-       { 0x02, KEY_POWER },      /* Power */
-       { 0x03, KEY_TV },         /* TV */
-       { 0x04, KEY_DVD },        /* DVD */
-       { 0x05, KEY_WWW },        /* WEB */
-       { 0x06, KEY_BOOKMARKS },  /* "book" */
-       { 0x07, KEY_EDIT },       /* "hand" */
        { 0x1c, KEY_COFFEE },     /* "timer" */
-       { 0x20, KEY_FRONT },      /* "max" */
        { 0x1d, KEY_LEFT },       /* left */
+       { 0x1e, KEY_OK },         /* "OK" */
        { 0x1f, KEY_RIGHT },      /* right */
+       { 0x20, KEY_FRONT },      /* "max" */
+
+       { 0x21, KEY_E },
        { 0x22, KEY_DOWN },       /* down */
-       { 0x1a, KEY_UP },         /* up */
-       { 0x1e, KEY_OK },         /* "OK" */
-       { 0x09, KEY_VOLUMEDOWN }, /* VOL + */
-       { 0x08, KEY_VOLUMEUP },   /* VOL - */
-       { 0x0a, KEY_MUTE },       /* MUTE  */
-       { 0x0b, KEY_CHANNELUP },  /* CH + */
-       { 0x0c, KEY_CHANNELDOWN },/* CH - */
-       { 0x27, KEY_RECORD },     /* ( o) red */
-       { 0x25, KEY_PLAY },       /* ( >) */
+       { 0x23, KEY_F },
+       /* Play/stop/pause buttons */
        { 0x24, KEY_REWIND },     /* (<<) */
+       { 0x25, KEY_PLAY },       /* ( >) */
        { 0x26, KEY_FORWARD },    /* (>>) */
+
+       { 0x27, KEY_RECORD },     /* ( o) red */
        { 0x28, KEY_STOP },       /* ([]) */
        { 0x29, KEY_PAUSE },      /* ('') */
-       { 0x2b, KEY_PREVIOUS },   /* (<-) */
+
+       /* Extra keys, not on the original ATI remote */
        { 0x2a, KEY_NEXT },       /* (>+) */
+       { 0x2b, KEY_PREVIOUS },   /* (<-) */
        { 0x2d, KEY_INFO },       /* PLAYING */
        { 0x2e, KEY_HOME },       /* TOP */
        { 0x2f, KEY_END },        /* END */