[media] ati_remote: fix check for a weird byte
authorAnssi Hannula <anssi.hannula@iki.fi>
Sat, 6 Aug 2011 21:18:10 +0000 (18:18 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Thu, 22 Sep 2011 13:52:35 +0000 (10:52 -0300)
The ati_remote_dump() function tries to not print "Weird byte" warning
for 1-byte responses that contain 0xff or 0x00, but it doesn't work
properly as it simply falls back to the "Weird data" warning in the else
clause.

Fix that by adding an inner if clause.

Signed-off-by: Anssi Hannula <anssi.hannula@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/rc/ati_remote.c

index 7a47d7e73b57ddc88b523b347e6a77afb43ff1a8..92f2b1bcef4a7f6747cee2a58609229c563e5514 100644 (file)
@@ -273,9 +273,10 @@ static struct usb_driver ati_remote_driver = {
 static void ati_remote_dump(struct device *dev, unsigned char *data,
                            unsigned int len)
 {
-       if ((len == 1) && (data[0] != (unsigned char)0xff) && (data[0] != 0x00))
-               dev_warn(dev, "Weird byte 0x%02x\n", data[0]);
-       else if (len == 4)
+       if (len == 1) {
+               if (data[0] != (unsigned char)0xff && data[0] != 0x00)
+                       dev_warn(dev, "Weird byte 0x%02x\n", data[0]);
+       } else if (len == 4)
                dev_warn(dev, "Weird key %02x %02x %02x %02x\n",
                     data[0], data[1], data[2], data[3]);
        else