[media] rc-core: fix dib0700 scancode generation for RC5
authorDavid Härdeman <david@hardeman.nu>
Mon, 30 Mar 2015 20:51:01 +0000 (17:51 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Tue, 12 May 2015 21:27:25 +0000 (18:27 -0300)
commit af3a4a9bbeb0 ("[media] dib0700: NEC scancode cleanup") cleaned
up the NEC scancode logic but overlooked the RC5 case.

This patch brings the RC5 case in line with the NEC code and makes
the struct self-documenting.

Signed-off-by: David Härdeman <david@hardeman.nu>
Reported-by: David Cimbůrek <david.cimburek@gmail.com>
Cc: stable@vger.kernel.org
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/usb/dvb-usb/dib0700_core.c

index 2b40393836ffafa14c63c2a700bdd8072d8e0ed7..0d248ce02a9b2eb02b9da6fecbb6175319110574 100644 (file)
@@ -655,10 +655,20 @@ out:
 struct dib0700_rc_response {
        u8 report_id;
        u8 data_state;
-       u8 system;
-       u8 not_system;
-       u8 data;
-       u8 not_data;
+       union {
+               struct {
+                       u8 system;
+                       u8 not_system;
+                       u8 data;
+                       u8 not_data;
+               } nec;
+               struct {
+                       u8 not_used;
+                       u8 system;
+                       u8 data;
+                       u8 not_data;
+               } rc5;
+       };
 };
 #define RC_MSG_SIZE_V1_20 6
 
@@ -694,8 +704,8 @@ static void dib0700_rc_urb_completion(struct urb *purb)
 
        deb_data("IR ID = %02X state = %02X System = %02X %02X Cmd = %02X %02X (len %d)\n",
                 poll_reply->report_id, poll_reply->data_state,
-                poll_reply->system, poll_reply->not_system,
-                poll_reply->data, poll_reply->not_data,
+                poll_reply->nec.system, poll_reply->nec.not_system,
+                poll_reply->nec.data, poll_reply->nec.not_data,
                 purb->actual_length);
 
        switch (d->props.rc.core.protocol) {
@@ -704,30 +714,30 @@ static void dib0700_rc_urb_completion(struct urb *purb)
                toggle = 0;
 
                /* NEC protocol sends repeat code as 0 0 0 FF */
-               if (poll_reply->system     == 0x00 &&
-                   poll_reply->not_system == 0x00 &&
-                   poll_reply->data       == 0x00 &&
-                   poll_reply->not_data   == 0xff) {
+               if (poll_reply->nec.system     == 0x00 &&
+                   poll_reply->nec.not_system == 0x00 &&
+                   poll_reply->nec.data       == 0x00 &&
+                   poll_reply->nec.not_data   == 0xff) {
                        poll_reply->data_state = 2;
                        break;
                }
 
-               if ((poll_reply->data ^ poll_reply->not_data) != 0xff) {
+               if ((poll_reply->nec.data ^ poll_reply->nec.not_data) != 0xff) {
                        deb_data("NEC32 protocol\n");
-                       keycode = RC_SCANCODE_NEC32(poll_reply->system     << 24 |
-                                                    poll_reply->not_system << 16 |
-                                                    poll_reply->data       << 8  |
-                                                    poll_reply->not_data);
-               } else if ((poll_reply->system ^ poll_reply->not_system) != 0xff) {
+                       keycode = RC_SCANCODE_NEC32(poll_reply->nec.system     << 24 |
+                                                    poll_reply->nec.not_system << 16 |
+                                                    poll_reply->nec.data       << 8  |
+                                                    poll_reply->nec.not_data);
+               } else if ((poll_reply->nec.system ^ poll_reply->nec.not_system) != 0xff) {
                        deb_data("NEC extended protocol\n");
-                       keycode = RC_SCANCODE_NECX(poll_reply->system << 8 |
-                                                   poll_reply->not_system,
-                                                   poll_reply->data);
+                       keycode = RC_SCANCODE_NECX(poll_reply->nec.system << 8 |
+                                                   poll_reply->nec.not_system,
+                                                   poll_reply->nec.data);
 
                } else {
                        deb_data("NEC normal protocol\n");
-                       keycode = RC_SCANCODE_NEC(poll_reply->system,
-                                                  poll_reply->data);
+                       keycode = RC_SCANCODE_NEC(poll_reply->nec.system,
+                                                  poll_reply->nec.data);
                }
 
                break;
@@ -735,19 +745,19 @@ static void dib0700_rc_urb_completion(struct urb *purb)
                deb_data("RC5 protocol\n");
                protocol = RC_TYPE_RC5;
                toggle = poll_reply->report_id;
-               keycode = RC_SCANCODE_RC5(poll_reply->system, poll_reply->data);
+               keycode = RC_SCANCODE_RC5(poll_reply->rc5.system, poll_reply->rc5.data);
+
+               if ((poll_reply->rc5.data ^ poll_reply->rc5.not_data) != 0xff) {
+                       /* Key failed integrity check */
+                       err("key failed integrity check: %02x %02x %02x %02x",
+                           poll_reply->rc5.not_used, poll_reply->rc5.system,
+                           poll_reply->rc5.data, poll_reply->rc5.not_data);
+                       goto resubmit;
+               }
 
                break;
        }
 
-       if ((poll_reply->data + poll_reply->not_data) != 0xff) {
-               /* Key failed integrity check */
-               err("key failed integrity check: %02x %02x %02x %02x",
-                   poll_reply->system,  poll_reply->not_system,
-                   poll_reply->data, poll_reply->not_data);
-               goto resubmit;
-       }
-
        rc_keydown(d->rc_dev, protocol, keycode, toggle);
 
 resubmit: