Input: gtco - use sign_extend32() for sign extension
authorMartin Kepplinger <martink@posteo.de>
Sat, 24 Jan 2015 00:40:53 +0000 (16:40 -0800)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Tue, 27 Jan 2015 07:07:45 +0000 (23:07 -0800)
Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/tablet/gtco.c

index 858045694e9d73f8ade7fe9d2327ec6594cb12c6..3a7f3a4a439635064a4e8ca78e1e7860b9072a47 100644 (file)
@@ -59,7 +59,7 @@ Scott Hill shill@gtcocalcomp.com
 #include <asm/uaccess.h>
 #include <asm/unaligned.h>
 #include <asm/byteorder.h>
-
+#include <linux/bitops.h>
 
 #include <linux/usb/input.h>
 
@@ -614,7 +614,6 @@ static void gtco_urb_callback(struct urb *urbinfo)
        struct input_dev  *inputdev;
        int               rc;
        u32               val = 0;
-       s8                valsigned = 0;
        char              le_buffer[2];
 
        inputdev = device->inputdevice;
@@ -665,20 +664,11 @@ static void gtco_urb_callback(struct urb *urbinfo)
                        /* Fall thru */
                case 4:
                        /* Tilt */
+                       input_report_abs(inputdev, ABS_TILT_X,
+                                        sign_extend32(device->buffer[6], 6));
 
-                       /* Sign extend these 7 bit numbers.  */
-                       if (device->buffer[6] & 0x40)
-                               device->buffer[6] |= 0x80;
-
-                       if (device->buffer[7] & 0x40)
-                               device->buffer[7] |= 0x80;
-
-
-                       valsigned = (device->buffer[6]);
-                       input_report_abs(inputdev, ABS_TILT_X, (s32)valsigned);
-
-                       valsigned = (device->buffer[7]);
-                       input_report_abs(inputdev, ABS_TILT_Y, (s32)valsigned);
+                       input_report_abs(inputdev, ABS_TILT_Y,
+                                        sign_extend32(device->buffer[7], 6));
 
                        /* Fall thru */
                case 2: