Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input
authorLinus Torvalds <torvalds@linux-foundation.org>
Fri, 25 Mar 2016 23:39:05 +0000 (16:39 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 25 Mar 2016 23:39:05 +0000 (16:39 -0700)
Pull more input updates from Dmitry Torokhov:
 "Second round of updates for the input subsystem.

  The BYD PS/2 protocol driver now uses absolute reporting mode and
  should behave more like other touchpads; Synaptics driver needed to
  extend one of its quirks to a newer firmware version, and a few USB
  drivers got tightened up checks for the contents of their descriptors"

* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input:
  Input: sur40 - fix DMA on stack
  Input: ati_remote2 - fix crashes on detecting device with invalid descriptor
  Input: synaptics - handle spurious release of trackstick buttons, again
  Input: synaptics-rmi4 - remove check of Non-NULL array
  Input: byd - enable absolute mode
  Input: ims-pcu - sanity check against missing interfaces
  Input: melfas_mip4 - add hw_version sysfs attribute

drivers/input/misc/ati_remote2.c
drivers/input/misc/ims-pcu.c
drivers/input/mouse/byd.c
drivers/input/mouse/psmouse-base.c
drivers/input/mouse/synaptics.c
drivers/input/rmi4/rmi_driver.c
drivers/input/touchscreen/melfas_mip4.c
drivers/input/touchscreen/sur40.c

index cfd58e8..1c5914c 100644 (file)
@@ -817,26 +817,49 @@ static int ati_remote2_probe(struct usb_interface *interface, const struct usb_d
 
        ar2->udev = udev;
 
+       /* Sanity check, first interface must have an endpoint */
+       if (alt->desc.bNumEndpoints < 1 || !alt->endpoint) {
+               dev_err(&interface->dev,
+                       "%s(): interface 0 must have an endpoint\n", __func__);
+               r = -ENODEV;
+               goto fail1;
+       }
        ar2->intf[0] = interface;
        ar2->ep[0] = &alt->endpoint[0].desc;
 
+       /* Sanity check, the device must have two interfaces */
        ar2->intf[1] = usb_ifnum_to_if(udev, 1);
+       if ((udev->actconfig->desc.bNumInterfaces < 2) || !ar2->intf[1]) {
+               dev_err(&interface->dev, "%s(): need 2 interfaces, found %d\n",
+                       __func__, udev->actconfig->desc.bNumInterfaces);
+               r = -ENODEV;
+               goto fail1;
+       }
+
        r = usb_driver_claim_interface(&ati_remote2_driver, ar2->intf[1], ar2);
        if (r)
                goto fail1;
+
+       /* Sanity check, second interface must have an endpoint */
        alt = ar2->intf[1]->cur_altsetting;
+       if (alt->desc.bNumEndpoints < 1 || !alt->endpoint) {
+               dev_err(&interface->dev,
+                       "%s(): interface 1 must have an endpoint\n", __func__);
+               r = -ENODEV;
+               goto fail2;
+       }
        ar2->ep[1] = &alt->endpoint[0].desc;
 
        r = ati_remote2_urb_init(ar2);
        if (r)
-               goto fail2;
+               goto fail3;
 
        ar2->channel_mask = channel_mask;
        ar2->mode_mask = mode_mask;
 
        r = ati_remote2_setup(ar2, ar2->channel_mask);
        if (r)
-               goto fail2;
+               goto fail3;
 
        usb_make_path(udev, ar2->phys, sizeof(ar2->phys));
        strlcat(ar2->phys, "/input0", sizeof(ar2->phys));
@@ -845,11 +868,11 @@ static int ati_remote2_probe(struct usb_interface *interface, const struct usb_d
 
        r = sysfs_create_group(&udev->dev.kobj, &ati_remote2_attr_group);
        if (r)
-               goto fail2;
+               goto fail3;
 
        r = ati_remote2_input_init(ar2);
        if (r)
-               goto fail3;
+               goto fail4;
 
        usb_set_intfdata(interface, ar2);
 
@@ -857,10 +880,11 @@ static int ati_remote2_probe(struct usb_interface *interface, const struct usb_d
 
        return 0;
 
- fail3:
+ fail4:
        sysfs_remove_group(&udev->dev.kobj, &ati_remote2_attr_group);
- fail2:
+ fail3:
        ati_remote2_urb_cleanup(ar2);
+ fail2:
        usb_driver_release_interface(&ati_remote2_driver, ar2->intf[1]);
  fail1:
        kfree(ar2);
index ac1fa5f..9c0ea36 100644 (file)
@@ -1663,6 +1663,8 @@ static int ims_pcu_parse_cdc_data(struct usb_interface *intf, struct ims_pcu *pc
 
        pcu->ctrl_intf = usb_ifnum_to_if(pcu->udev,
                                         union_desc->bMasterInterface0);
+       if (!pcu->ctrl_intf)
+               return -EINVAL;
 
        alt = pcu->ctrl_intf->cur_altsetting;
        pcu->ep_ctrl = &alt->endpoint[0].desc;
@@ -1670,6 +1672,8 @@ static int ims_pcu_parse_cdc_data(struct usb_interface *intf, struct ims_pcu *pc
 
        pcu->data_intf = usb_ifnum_to_if(pcu->udev,
                                         union_desc->bSlaveInterface0);
+       if (!pcu->data_intf)
+               return -EINVAL;
 
        alt = pcu->data_intf->cur_altsetting;
        if (alt->desc.bNumEndpoints != 2) {
index 9425e0f..fdc243c 100644 (file)
 #include <linux/input.h>
 #include <linux/libps2.h>
 #include <linux/serio.h>
+#include <linux/slab.h>
 
 #include "psmouse.h"
 #include "byd.h"
 
+/* PS2 Bits */
 #define PS2_Y_OVERFLOW BIT_MASK(7)
 #define PS2_X_OVERFLOW BIT_MASK(6)
 #define PS2_Y_SIGN     BIT_MASK(5)
 #define PS2_LEFT       BIT_MASK(0)
 
 /*
- * The touchpad reports gestures in the last byte of each packet. It can take
- * any of the following values:
+ * BYD pad constants
  */
 
-/* One-finger scrolling in one of the edge scroll zones. */
-#define BYD_SCROLLUP           0xCA
-#define BYD_SCROLLDOWN         0x36
-#define BYD_SCROLLLEFT         0xCB
-#define BYD_SCROLLRIGHT                0x35
-/* Two-finger scrolling. */
-#define BYD_2DOWN              0x2B
-#define BYD_2UP                        0xD5
-#define BYD_2LEFT              0xD6
-#define BYD_2RIGHT             0x2A
-/* Pinching in or out. */
-#define BYD_ZOOMOUT            0xD8
-#define BYD_ZOOMIN             0x28
-/* Three-finger swipe. */
-#define BYD_3UP                        0xD3
-#define BYD_3DOWN              0x2D
-#define BYD_3LEFT              0xD4
-#define BYD_3RIGHT             0x2C
-/* Four-finger swipe. */
-#define BYD_4UP                        0xCD
-#define BYD_4DOWN              0x33
+/*
+ * True device resolution is unknown, however experiments show the
+ * resolution is about 111 units/mm.
+ * Absolute coordinate packets are in the range 0-255 for both X and Y
+ * we pick ABS_X/ABS_Y dimensions which are multiples of 256 and in
+ * the right ballpark given the touchpad's physical dimensions and estimate
+ * resolution per spec sheet, device active area dimensions are
+ * 101.6 x 60.1 mm.
+ */
+#define BYD_PAD_WIDTH          11264
+#define BYD_PAD_HEIGHT         6656
+#define BYD_PAD_RESOLUTION     111
 
-int byd_detect(struct psmouse *psmouse, bool set_properties)
+/*
+ * Given the above dimensions, relative packets velocity is in multiples of
+ * 1 unit / 11 milliseconds.  We use this dt to estimate distance traveled
+ */
+#define BYD_DT                 11
+/* Time in jiffies used to timeout various touch events (64 ms) */
+#define BYD_TOUCH_TIMEOUT      msecs_to_jiffies(64)
+
+/* BYD commands reverse engineered from windows driver */
+
+/*
+ * Swipe gesture from off-pad to on-pad
+ *  0 : disable
+ *  1 : enable
+ */
+#define BYD_CMD_SET_OFFSCREEN_SWIPE            0x10cc
+/*
+ * Tap and drag delay time
+ *  0 : disable
+ *  1 - 8 : least to most delay
+ */
+#define BYD_CMD_SET_TAP_DRAG_DELAY_TIME                0x10cf
+/*
+ * Physical buttons function mapping
+ *  0 : enable
+ *  4 : normal
+ *  5 : left button custom command
+ *  6 : right button custom command
+ *  8 : disable
+ */
+#define BYD_CMD_SET_PHYSICAL_BUTTONS           0x10d0
+/*
+ * Absolute mode (1 byte X/Y resolution)
+ *  0 : disable
+ *  2 : enable
+ */
+#define BYD_CMD_SET_ABSOLUTE_MODE              0x10d1
+/*
+ * Two finger scrolling
+ *  1 : vertical
+ *  2 : horizontal
+ *  3 : vertical + horizontal
+ *  4 : disable
+ */
+#define BYD_CMD_SET_TWO_FINGER_SCROLL          0x10d2
+/*
+ * Handedness
+ *  1 : right handed
+ *  2 : left handed
+ */
+#define BYD_CMD_SET_HANDEDNESS                 0x10d3
+/*
+ * Tap to click
+ *  1 : enable
+ *  2 : disable
+ */
+#define BYD_CMD_SET_TAP                                0x10d4
+/*
+ * Tap and drag
+ *  1 : tap and hold to drag
+ *  2 : tap and hold to drag + lock
+ *  3 : disable
+ */
+#define BYD_CMD_SET_TAP_DRAG                   0x10d5
+/*
+ * Touch sensitivity
+ *  1 - 7 : least to most sensitive
+ */
+#define BYD_CMD_SET_TOUCH_SENSITIVITY          0x10d6
+/*
+ * One finger scrolling
+ *  1 : vertical
+ *  2 : horizontal
+ *  3 : vertical + horizontal
+ *  4 : disable
+ */
+#define BYD_CMD_SET_ONE_FINGER_SCROLL          0x10d7
+/*
+ * One finger scrolling function
+ *  1 : free scrolling
+ *  2 : edge motion
+ *  3 : free scrolling + edge motion
+ *  4 : disable
+ */
+#define BYD_CMD_SET_ONE_FINGER_SCROLL_FUNC     0x10d8
+/*
+ * Sliding speed
+ *  1 - 5 : slowest to fastest
+ */
+#define BYD_CMD_SET_SLIDING_SPEED              0x10da
+/*
+ * Edge motion
+ *  1 : disable
+ *  2 : enable when dragging
+ *  3 : enable when dragging and pointing
+ */
+#define BYD_CMD_SET_EDGE_MOTION                        0x10db
+/*
+ * Left edge region size
+ *  0 - 7 : smallest to largest width
+ */
+#define BYD_CMD_SET_LEFT_EDGE_REGION           0x10dc
+/*
+ * Top edge region size
+ *  0 - 9 : smallest to largest height
+ */
+#define BYD_CMD_SET_TOP_EDGE_REGION            0x10dd
+/*
+ * Disregard palm press as clicks
+ *  1 - 6 : smallest to largest
+ */
+#define BYD_CMD_SET_PALM_CHECK                 0x10de
+/*
+ * Right edge region size
+ *  0 - 7 : smallest to largest width
+ */
+#define BYD_CMD_SET_RIGHT_EDGE_REGION          0x10df
+/*
+ * Bottom edge region size
+ *  0 - 9 : smallest to largest height
+ */
+#define BYD_CMD_SET_BOTTOM_EDGE_REGION         0x10e1
+/*
+ * Multitouch gestures
+ *  1 : enable
+ *  2 : disable
+ */
+#define BYD_CMD_SET_MULTITOUCH                 0x10e3
+/*
+ * Edge motion speed
+ *  0 : control with finger pressure
+ *  1 - 9 : slowest to fastest
+ */
+#define BYD_CMD_SET_EDGE_MOTION_SPEED          0x10e4
+/*
+ * Two finger scolling function
+ *  0 : free scrolling
+ *  1 : free scrolling (with momentum)
+ *  2 : edge motion
+ *  3 : free scrolling (with momentum) + edge motion
+ *  4 : disable
+ */
+#define BYD_CMD_SET_TWO_FINGER_SCROLL_FUNC     0x10e5
+
+/*
+ * The touchpad generates a mixture of absolute and relative packets, indicated
+ * by the the last byte of each packet being set to one of the following:
+ */
+#define BYD_PACKET_ABSOLUTE                    0xf8
+#define BYD_PACKET_RELATIVE                    0x00
+/* Multitouch gesture packets */
+#define BYD_PACKET_PINCH_IN                    0xd8
+#define BYD_PACKET_PINCH_OUT                   0x28
+#define BYD_PACKET_ROTATE_CLOCKWISE            0x29
+#define BYD_PACKET_ROTATE_ANTICLOCKWISE                0xd7
+#define BYD_PACKET_TWO_FINGER_SCROLL_RIGHT     0x2a
+#define BYD_PACKET_TWO_FINGER_SCROLL_DOWN      0x2b
+#define BYD_PACKET_TWO_FINGER_SCROLL_UP                0xd5
+#define BYD_PACKET_TWO_FINGER_SCROLL_LEFT      0xd6
+#define BYD_PACKET_THREE_FINGER_SWIPE_RIGHT    0x2c
+#define BYD_PACKET_THREE_FINGER_SWIPE_DOWN     0x2d
+#define BYD_PACKET_THREE_FINGER_SWIPE_UP       0xd3
+#define BYD_PACKET_THREE_FINGER_SWIPE_LEFT     0xd4
+#define BYD_PACKET_FOUR_FINGER_DOWN            0x33
+#define BYD_PACKET_FOUR_FINGER_UP              0xcd
+#define BYD_PACKET_REGION_SCROLL_RIGHT         0x35
+#define BYD_PACKET_REGION_SCROLL_DOWN          0x36
+#define BYD_PACKET_REGION_SCROLL_UP            0xca
+#define BYD_PACKET_REGION_SCROLL_LEFT          0xcb
+#define BYD_PACKET_RIGHT_CORNER_CLICK          0xd2
+#define BYD_PACKET_LEFT_CORNER_CLICK           0x2e
+#define BYD_PACKET_LEFT_AND_RIGHT_CORNER_CLICK 0x2f
+#define BYD_PACKET_ONTO_PAD_SWIPE_RIGHT                0x37
+#define BYD_PACKET_ONTO_PAD_SWIPE_DOWN         0x30
+#define BYD_PACKET_ONTO_PAD_SWIPE_UP           0xd0
+#define BYD_PACKET_ONTO_PAD_SWIPE_LEFT         0xc9
+
+struct byd_data {
+       struct timer_list timer;
+       s32 abs_x;
+       s32 abs_y;
+       typeof(jiffies) last_touch_time;
+       bool btn_left;
+       bool btn_right;
+       bool touch;
+};
+
+static void byd_report_input(struct psmouse *psmouse)
 {
-       struct ps2dev *ps2dev = &psmouse->ps2dev;
-       unsigned char param[4];
+       struct byd_data *priv = psmouse->private;
+       struct input_dev *dev = psmouse->dev;
 
-       param[0] = 0x03;
-       param[1] = 0x00;
-       param[2] = 0x00;
-       param[3] = 0x00;
+       input_report_key(dev, BTN_TOUCH, priv->touch);
+       input_report_key(dev, BTN_TOOL_FINGER, priv->touch);
 
-       if (ps2_command(ps2dev, param, PSMOUSE_CMD_SETRES))
-               return -1;
-       if (ps2_command(ps2dev, param, PSMOUSE_CMD_SETRES))
-               return -1;
-       if (ps2_command(ps2dev, param, PSMOUSE_CMD_SETRES))
-               return -1;
-       if (ps2_command(ps2dev, param, PSMOUSE_CMD_SETRES))
-               return -1;
-       if (ps2_command(ps2dev, param, PSMOUSE_CMD_GETINFO))
-               return -1;
+       input_report_abs(dev, ABS_X, priv->abs_x);
+       input_report_abs(dev, ABS_Y, priv->abs_y);
+       input_report_key(dev, BTN_LEFT, priv->btn_left);
+       input_report_key(dev, BTN_RIGHT, priv->btn_right);
 
-       if (param[1] != 0x03 || param[2] != 0x64)
-               return -ENODEV;
+       input_sync(dev);
+}
 
-       psmouse_dbg(psmouse, "BYD touchpad detected\n");
+static void byd_clear_touch(unsigned long data)
+{
+       struct psmouse *psmouse = (struct psmouse *)data;
+       struct byd_data *priv = psmouse->private;
 
-       if (set_properties) {
-               psmouse->vendor = "BYD";
-               psmouse->name = "TouchPad";
-       }
+       serio_pause_rx(psmouse->ps2dev.serio);
+       priv->touch = false;
 
-       return 0;
+       byd_report_input(psmouse);
+
+       serio_continue_rx(psmouse->ps2dev.serio);
+
+       /*
+        * Move cursor back to center of pad when we lose touch - this
+        * specifically improves user experience when moving cursor with one
+        * finger, and pressing a button with another.
+        */
+       priv->abs_x = BYD_PAD_WIDTH / 2;
+       priv->abs_y = BYD_PAD_HEIGHT / 2;
 }
 
 static psmouse_ret_t byd_process_byte(struct psmouse *psmouse)
 {
-       struct input_dev *dev = psmouse->dev;
+       struct byd_data *priv = psmouse->private;
        u8 *pkt = psmouse->packet;
 
        if (psmouse->pktcnt > 0 && !(pkt[0] & PS2_ALWAYS_1)) {
@@ -102,53 +284,34 @@ static psmouse_ret_t byd_process_byte(struct psmouse *psmouse)
 
        /* Otherwise, a full packet has been received */
        switch (pkt[3]) {
-       case 0: {
+       case BYD_PACKET_ABSOLUTE:
+               /* Only use absolute packets for the start of movement. */
+               if (!priv->touch) {
+                       /* needed to detect tap */
+                       typeof(jiffies) tap_time =
+                               priv->last_touch_time + BYD_TOUCH_TIMEOUT;
+                       priv->touch = time_after(jiffies, tap_time);
+
+                       /* init abs position */
+                       priv->abs_x = pkt[1] * (BYD_PAD_WIDTH / 256);
+                       priv->abs_y = (255 - pkt[2]) * (BYD_PAD_HEIGHT / 256);
+               }
+               break;
+       case BYD_PACKET_RELATIVE: {
                /* Standard packet */
                /* Sign-extend if a sign bit is set. */
-               unsigned int signx = pkt[0] & PS2_X_SIGN ? ~0xFF : 0;
-               unsigned int signy = pkt[0] & PS2_Y_SIGN ? ~0xFF : 0;
-               int dx = signx | (int) pkt[1];
-               int dy = signy | (int) pkt[2];
+               u32 signx = pkt[0] & PS2_X_SIGN ? ~0xFF : 0;
+               u32 signy = pkt[0] & PS2_Y_SIGN ? ~0xFF : 0;
+               s32 dx = signx | (int) pkt[1];
+               s32 dy = signy | (int) pkt[2];
 
-               input_report_rel(psmouse->dev, REL_X, dx);
-               input_report_rel(psmouse->dev, REL_Y, -dy);
+               /* Update position based on velocity */
+               priv->abs_x += dx * BYD_DT;
+               priv->abs_y -= dy * BYD_DT;
 
-               input_report_key(psmouse->dev, BTN_LEFT, pkt[0] & PS2_LEFT);
-               input_report_key(psmouse->dev, BTN_RIGHT, pkt[0] & PS2_RIGHT);
-               input_report_key(psmouse->dev, BTN_MIDDLE, pkt[0] & PS2_MIDDLE);
+               priv->touch = true;
                break;
        }
-
-       case BYD_SCROLLDOWN:
-       case BYD_2DOWN:
-               input_report_rel(dev, REL_WHEEL, -1);
-               break;
-
-       case BYD_SCROLLUP:
-       case BYD_2UP:
-               input_report_rel(dev, REL_WHEEL, 1);
-               break;
-
-       case BYD_SCROLLLEFT:
-       case BYD_2LEFT:
-               input_report_rel(dev, REL_HWHEEL, -1);
-               break;
-
-       case BYD_SCROLLRIGHT:
-       case BYD_2RIGHT:
-               input_report_rel(dev, REL_HWHEEL, 1);
-               break;
-
-       case BYD_ZOOMOUT:
-       case BYD_ZOOMIN:
-       case BYD_3UP:
-       case BYD_3DOWN:
-       case BYD_3LEFT:
-       case BYD_3RIGHT:
-       case BYD_4UP:
-       case BYD_4DOWN:
-               break;
-
        default:
                psmouse_warn(psmouse,
                             "Unrecognized Z: pkt = %02x %02x %02x %02x\n",
@@ -157,134 +320,76 @@ static psmouse_ret_t byd_process_byte(struct psmouse *psmouse)
                return PSMOUSE_BAD_DATA;
        }
 
-       input_sync(dev);
+       priv->btn_left = pkt[0] & PS2_LEFT;
+       priv->btn_right = pkt[0] & PS2_RIGHT;
 
-       return PSMOUSE_FULL_PACKET;
-}
+       byd_report_input(psmouse);
 
-/* Send a sequence of bytes, where each is ACKed before the next is sent. */
-static int byd_send_sequence(struct psmouse *psmouse, const u8 *seq, size_t len)
-{
-       unsigned int i;
-
-       for (i = 0; i < len; ++i) {
-               if (ps2_command(&psmouse->ps2dev, NULL, seq[i]))
-                       return -1;
+       /* Reset time since last touch. */
+       if (priv->touch) {
+               priv->last_touch_time = jiffies;
+               mod_timer(&priv->timer, jiffies + BYD_TOUCH_TIMEOUT);
        }
-       return 0;
-}
-
-/* Keep scrolling after fingers are removed. */
-#define SCROLL_INERTIAL                0x01
-#define SCROLL_NO_INERTIAL     0x02
-
-/* Clicking can be done by tapping or pressing. */
-#define CLICK_BOTH             0x01
-/* Clicking can only be done by pressing. */
-#define CLICK_PRESS_ONLY       0x02
 
-static int byd_enable(struct psmouse *psmouse)
-{
-       const u8 seq1[] = { 0xE2, 0x00, 0xE0, 0x02, 0xE0 };
-       const u8 seq2[] = {
-               0xD3, 0x01,
-               0xD0, 0x00,
-               0xD0, 0x04,
-               /* Whether clicking is done by tapping or pressing. */
-               0xD4, CLICK_PRESS_ONLY,
-               0xD5, 0x01,
-               0xD7, 0x03,
-               /* Vertical and horizontal one-finger scroll zone inertia. */
-               0xD8, SCROLL_INERTIAL,
-               0xDA, 0x05,
-               0xDB, 0x02,
-               0xE4, 0x05,
-               0xD6, 0x01,
-               0xDE, 0x04,
-               0xE3, 0x01,
-               0xCF, 0x00,
-               0xD2, 0x03,
-               /* Vertical and horizontal two-finger scrolling inertia. */
-               0xE5, SCROLL_INERTIAL,
-               0xD9, 0x02,
-               0xD9, 0x07,
-               0xDC, 0x03,
-               0xDD, 0x03,
-               0xDF, 0x03,
-               0xE1, 0x03,
-               0xD1, 0x00,
-               0xCE, 0x00,
-               0xCC, 0x00,
-               0xE0, 0x00,
-               0xE2, 0x01
-       };
-       u8 param[4];
-
-       if (byd_send_sequence(psmouse, seq1, ARRAY_SIZE(seq1)))
-               return -1;
-
-       /* Send a 0x01 command, which should return 4 bytes. */
-       if (ps2_command(&psmouse->ps2dev, param, 0x0401))
-               return -1;
-
-       if (byd_send_sequence(psmouse, seq2, ARRAY_SIZE(seq2)))
-               return -1;
-
-       return 0;
+       return PSMOUSE_FULL_PACKET;
 }
 
-/*
- * Send the set of PS/2 commands required to make it identify as an
- * intellimouse with 4-byte instead of 3-byte packets.
- */
-static int byd_send_intellimouse_sequence(struct psmouse *psmouse)
+static int byd_reset_touchpad(struct psmouse *psmouse)
 {
        struct ps2dev *ps2dev = &psmouse->ps2dev;
        u8 param[4];
-       int i;
+       size_t i;
+
        const struct {
                u16 command;
                u8 arg;
        } seq[] = {
-               { PSMOUSE_CMD_RESET_BAT, 0 },
-               { PSMOUSE_CMD_RESET_BAT, 0 },
-               { PSMOUSE_CMD_GETID, 0 },
-               { PSMOUSE_CMD_SETSCALE11, 0 },
-               { PSMOUSE_CMD_SETSCALE11, 0 },
-               { PSMOUSE_CMD_SETSCALE11, 0 },
-               { PSMOUSE_CMD_GETINFO, 0 },
-               { PSMOUSE_CMD_SETRES, 0x03 },
+               /*
+                * Intellimouse initialization sequence, to get 4-byte instead
+                * of 3-byte packets.
+                */
                { PSMOUSE_CMD_SETRATE, 0xC8 },
                { PSMOUSE_CMD_SETRATE, 0x64 },
                { PSMOUSE_CMD_SETRATE, 0x50 },
                { PSMOUSE_CMD_GETID, 0 },
-               { PSMOUSE_CMD_SETRATE, 0xC8 },
-               { PSMOUSE_CMD_SETRATE, 0xC8 },
-               { PSMOUSE_CMD_SETRATE, 0x50 },
-               { PSMOUSE_CMD_GETID, 0 },
-               { PSMOUSE_CMD_SETRATE, 0x64 },
-               { PSMOUSE_CMD_SETRES, 0x03 },
-               { PSMOUSE_CMD_ENABLE, 0 }
+               { PSMOUSE_CMD_ENABLE, 0 },
+               /*
+                * BYD-specific initialization, which enables absolute mode and
+                * (if desired), the touchpad's built-in gesture detection.
+                */
+               { 0x10E2, 0x00 },
+               { 0x10E0, 0x02 },
+               /* The touchpad should reply with 4 seemingly-random bytes */
+               { 0x14E0, 0x01 },
+               /* Pairs of parameters and values. */
+               { BYD_CMD_SET_HANDEDNESS, 0x01 },
+               { BYD_CMD_SET_PHYSICAL_BUTTONS, 0x04 },
+               { BYD_CMD_SET_TAP, 0x02 },
+               { BYD_CMD_SET_ONE_FINGER_SCROLL, 0x04 },
+               { BYD_CMD_SET_ONE_FINGER_SCROLL_FUNC, 0x04 },
+               { BYD_CMD_SET_EDGE_MOTION, 0x01 },
+               { BYD_CMD_SET_PALM_CHECK, 0x00 },
+               { BYD_CMD_SET_MULTITOUCH, 0x02 },
+               { BYD_CMD_SET_TWO_FINGER_SCROLL, 0x04 },
+               { BYD_CMD_SET_TWO_FINGER_SCROLL_FUNC, 0x04 },
+               { BYD_CMD_SET_LEFT_EDGE_REGION, 0x00 },
+               { BYD_CMD_SET_TOP_EDGE_REGION, 0x00 },
+               { BYD_CMD_SET_RIGHT_EDGE_REGION, 0x00 },
+               { BYD_CMD_SET_BOTTOM_EDGE_REGION, 0x00 },
+               { BYD_CMD_SET_ABSOLUTE_MODE, 0x02 },
+               /* Finalize initialization. */
+               { 0x10E0, 0x00 },
+               { 0x10E2, 0x01 },
        };
 
-       memset(param, 0, sizeof(param));
        for (i = 0; i < ARRAY_SIZE(seq); ++i) {
+               memset(param, 0, sizeof(param));
                param[0] = seq[i].arg;
                if (ps2_command(ps2dev, param, seq[i].command))
-                       return -1;
+                       return -EIO;
        }
 
-       return 0;
-}
-
-static int byd_reset_touchpad(struct psmouse *psmouse)
-{
-       if (byd_send_intellimouse_sequence(psmouse))
-               return -EIO;
-
-       if (byd_enable(psmouse))
-               return -EIO;
-
+       psmouse_set_state(psmouse, PSMOUSE_ACTIVATED);
        return 0;
 }
 
@@ -314,9 +419,50 @@ static int byd_reconnect(struct psmouse *psmouse)
        return 0;
 }
 
+static void byd_disconnect(struct psmouse *psmouse)
+{
+       struct byd_data *priv = psmouse->private;
+
+       if (priv) {
+               del_timer(&priv->timer);
+               kfree(psmouse->private);
+               psmouse->private = NULL;
+       }
+}
+
+int byd_detect(struct psmouse *psmouse, bool set_properties)
+{
+       struct ps2dev *ps2dev = &psmouse->ps2dev;
+       u8 param[4] = {0x03, 0x00, 0x00, 0x00};
+
+       if (ps2_command(ps2dev, param, PSMOUSE_CMD_SETRES))
+               return -1;
+       if (ps2_command(ps2dev, param, PSMOUSE_CMD_SETRES))
+               return -1;
+       if (ps2_command(ps2dev, param, PSMOUSE_CMD_SETRES))
+               return -1;
+       if (ps2_command(ps2dev, param, PSMOUSE_CMD_SETRES))
+               return -1;
+       if (ps2_command(ps2dev, param, PSMOUSE_CMD_GETINFO))
+               return -1;
+
+       if (param[1] != 0x03 || param[2] != 0x64)
+               return -ENODEV;
+
+       psmouse_dbg(psmouse, "BYD touchpad detected\n");
+
+       if (set_properties) {
+               psmouse->vendor = "BYD";
+               psmouse->name = "TouchPad";
+       }
+
+       return 0;
+}
+
 int byd_init(struct psmouse *psmouse)
 {
        struct input_dev *dev = psmouse->dev;
+       struct byd_data *priv;
 
        if (psmouse_reset(psmouse))
                return -EIO;
@@ -324,14 +470,39 @@ int byd_init(struct psmouse *psmouse)
        if (byd_reset_touchpad(psmouse))
                return -EIO;
 
+       priv = kzalloc(sizeof(*priv), GFP_KERNEL);
+       if (!priv)
+               return -ENOMEM;
+
+       memset(priv, 0, sizeof(*priv));
+       setup_timer(&priv->timer, byd_clear_touch, (unsigned long) psmouse);
+
+       psmouse->private = priv;
+       psmouse->disconnect = byd_disconnect;
        psmouse->reconnect = byd_reconnect;
        psmouse->protocol_handler = byd_process_byte;
        psmouse->pktsize = 4;
        psmouse->resync_time = 0;
 
-       __set_bit(BTN_MIDDLE, dev->keybit);
-       __set_bit(REL_WHEEL, dev->relbit);
-       __set_bit(REL_HWHEEL, dev->relbit);
+       __set_bit(INPUT_PROP_POINTER, dev->propbit);
+       /* Touchpad */
+       __set_bit(BTN_TOUCH, dev->keybit);
+       __set_bit(BTN_TOOL_FINGER, dev->keybit);
+       /* Buttons */
+       __set_bit(BTN_LEFT, dev->keybit);
+       __set_bit(BTN_RIGHT, dev->keybit);
+       __clear_bit(BTN_MIDDLE, dev->keybit);
+
+       /* Absolute position */
+       __set_bit(EV_ABS, dev->evbit);
+       input_set_abs_params(dev, ABS_X, 0, BYD_PAD_WIDTH, 0, 0);
+       input_set_abs_params(dev, ABS_Y, 0, BYD_PAD_HEIGHT, 0, 0);
+       input_abs_set_res(dev, ABS_X, BYD_PAD_RESOLUTION);
+       input_abs_set_res(dev, ABS_Y, BYD_PAD_RESOLUTION);
+       /* No relative support */
+       __clear_bit(EV_REL, dev->evbit);
+       __clear_bit(REL_X, dev->relbit);
+       __clear_bit(REL_Y, dev->relbit);
 
        return 0;
 }
index 39d1bec..5784e20 100644 (file)
@@ -846,7 +846,7 @@ static const struct psmouse_protocol psmouse_protocols[] = {
 #ifdef CONFIG_MOUSE_PS2_BYD
        {
                .type           = PSMOUSE_BYD,
-               .name           = "BydPS/2",
+               .name           = "BYDPS/2",
                .alias          = "byd",
                .detect         = byd_detect,
                .init           = byd_init,
index 6025eb4..a41d832 100644 (file)
@@ -862,8 +862,9 @@ static void synaptics_report_ext_buttons(struct psmouse *psmouse,
        if (!SYN_CAP_MULTI_BUTTON_NO(priv->ext_cap))
                return;
 
-       /* Bug in FW 8.1, buttons are reported only when ExtBit is 1 */
-       if (SYN_ID_FULL(priv->identity) == 0x801 &&
+       /* Bug in FW 8.1 & 8.2, buttons are reported only when ExtBit is 1 */
+       if ((SYN_ID_FULL(priv->identity) == 0x801 ||
+            SYN_ID_FULL(priv->identity) == 0x802) &&
            !((psmouse->packet[0] ^ psmouse->packet[3]) & 0x02))
                return;
 
index da38f0a..faa295e 100644 (file)
@@ -126,7 +126,7 @@ static void process_one_interrupt(struct rmi_driver_data *data,
                return;
 
        fh = to_rmi_function_handler(fn->dev.driver);
-       if (fn->irq_mask && fh->attention) {
+       if (fh->attention) {
                bitmap_and(data->fn_irq_bits, data->irq_status, fn->irq_mask,
                                data->irq_count);
                if (!bitmap_empty(data->fn_irq_bits, data->irq_count))
@@ -172,8 +172,7 @@ int rmi_process_interrupt_requests(struct rmi_device *rmi_dev)
         * use irq_chip.
         */
        list_for_each_entry(entry, &data->function_list, node)
-               if (entry->irq_mask)
-                       process_one_interrupt(data, entry);
+               process_one_interrupt(data, entry);
 
        if (data->input)
                input_sync(data->input);
index 8927297..fb5fb91 100644 (file)
@@ -1310,8 +1310,34 @@ static ssize_t mip4_sysfs_read_fw_version(struct device *dev,
 
 static DEVICE_ATTR(fw_version, S_IRUGO, mip4_sysfs_read_fw_version, NULL);
 
+static ssize_t mip4_sysfs_read_hw_version(struct device *dev,
+                                         struct device_attribute *attr,
+                                         char *buf)
+{
+       struct i2c_client *client = to_i2c_client(dev);
+       struct mip4_ts *ts = i2c_get_clientdata(client);
+       size_t count;
+
+       /* Take lock to prevent racing with firmware update */
+       mutex_lock(&ts->input->mutex);
+
+       /*
+        * product_name shows the name or version of the hardware
+        * paired with current firmware in the chip.
+        */
+       count = snprintf(buf, PAGE_SIZE, "%.*s\n",
+               (int)sizeof(ts->product_name), ts->product_name);
+
+       mutex_unlock(&ts->input->mutex);
+
+       return count;
+}
+
+static DEVICE_ATTR(hw_version, S_IRUGO, mip4_sysfs_read_hw_version, NULL);
+
 static struct attribute *mip4_attrs[] = {
        &dev_attr_fw_version.attr,
+       &dev_attr_hw_version.attr,
        &dev_attr_update_fw.attr,
        NULL,
 };
@@ -1512,6 +1538,6 @@ static struct i2c_driver mip4_driver = {
 module_i2c_driver(mip4_driver);
 
 MODULE_DESCRIPTION("MELFAS MIP4 Touchscreen");
-MODULE_VERSION("2016.03.03");
+MODULE_VERSION("2016.03.12");
 MODULE_AUTHOR("Sangwon Jee <jeesw@melfas.com>");
 MODULE_LICENSE("GPL");
index b6c4d03..880c40b 100644 (file)
@@ -197,28 +197,34 @@ static int sur40_command(struct sur40_state *dev,
 static int sur40_init(struct sur40_state *dev)
 {
        int result;
-       u8 buffer[24];
+       u8 *buffer;
+
+       buffer = kmalloc(24, GFP_KERNEL);
+       if (!buffer) {
+               result = -ENOMEM;
+               goto error;
+       }
 
        /* stupidly replay the original MS driver init sequence */
        result = sur40_command(dev, SUR40_GET_VERSION, 0x00, buffer, 12);
        if (result < 0)
-               return result;
+               goto error;
 
        result = sur40_command(dev, SUR40_GET_VERSION, 0x01, buffer, 12);
        if (result < 0)
-               return result;
+               goto error;
 
        result = sur40_command(dev, SUR40_GET_VERSION, 0x02, buffer, 12);
        if (result < 0)
-               return result;
+               goto error;
 
        result = sur40_command(dev, SUR40_UNKNOWN2,    0x00, buffer, 24);
        if (result < 0)
-               return result;
+               goto error;
 
        result = sur40_command(dev, SUR40_UNKNOWN1,    0x00, buffer,  5);
        if (result < 0)
-               return result;
+               goto error;
 
        result = sur40_command(dev, SUR40_GET_VERSION, 0x03, buffer, 12);
 
@@ -226,7 +232,8 @@ static int sur40_init(struct sur40_state *dev)
         * Discard the result buffer - no known data inside except
         * some version strings, maybe extract these sometime...
         */
-
+error:
+       kfree(buffer);
        return result;
 }