Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input
Pull input updates from Dmitry Torokhov: "A few drivers were updated with device tree bindings and others got a few small cleanups and fixes." Fix trivial conflict in drivers/input/keyboard/omap-keypad.c due to changes clashing with a whitespace cleanup. * 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input: (28 commits) Input: wacom - mark Intuos5 pad as in-prox when touching buttons Input: synaptics - adjust threshold for treating position values as negative Input: hgpk - use %*ph to dump small buffer Input: gpio_keys_polled - fix dt pdata->nbuttons Input: Add KD[GS]KBDIACRUC ioctls to the compatible list Input: omap-keypad - fixed formatting Input: tegra - move platform data header Input: wacom - add support for EMR on Cintiq 24HD touch Input: s3c2410_ts - make s3c_ts_pmops const Input: samsung-keypad - use of_get_child_count() helper Input: samsung-keypad - use of_match_ptr() Input: uinput - fix formatting Input: uinput - specify exact bit sizes on userspace APIs Input: uinput - mark failed submission requests as free Input: uinput - fix race that can block nonblocking read Input: uinput - return -EINVAL when read buffer size is too small Input: uinput - take event lock when fetching events from buffer Input: get rid of MATCH_BIT() macro Input: rotary-encoder - add DT bindings Input: rotary-encoder - constify platform data pointers ...
This commit is contained in:
@@ -334,11 +334,8 @@ static bool hgpk_is_byte_valid(struct psmouse *psmouse, unsigned char *packet)
|
||||
|
||||
if (!valid)
|
||||
psmouse_dbg(psmouse,
|
||||
"bad data, mode %d (%d) %02x %02x %02x %02x %02x %02x\n",
|
||||
priv->mode, pktcnt,
|
||||
psmouse->packet[0], psmouse->packet[1],
|
||||
psmouse->packet[2], psmouse->packet[3],
|
||||
psmouse->packet[4], psmouse->packet[5]);
|
||||
"bad data, mode %d (%d) %*ph\n",
|
||||
priv->mode, pktcnt, 6, psmouse->packet);
|
||||
|
||||
return valid;
|
||||
}
|
||||
@@ -1030,7 +1027,7 @@ static enum hgpk_model_t hgpk_get_model(struct psmouse *psmouse)
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
psmouse_dbg(psmouse, "ID: %02x %02x %02x\n", param[0], param[1], param[2]);
|
||||
psmouse_dbg(psmouse, "ID: %*ph\n", 3, param);
|
||||
|
||||
/* HGPK signature: 0x67, 0x00, 0x<model> */
|
||||
if (param[0] != 0x67 || param[1] != 0x00)
|
||||
|
@@ -53,14 +53,19 @@
|
||||
#define ABS_POS_BITS 13
|
||||
|
||||
/*
|
||||
* Any position values from the hardware above the following limits are
|
||||
* treated as "wrapped around negative" values that have been truncated to
|
||||
* the 13-bit reporting range of the hardware. These are just reasonable
|
||||
* guesses and can be adjusted if hardware is found that operates outside
|
||||
* of these parameters.
|
||||
* These values should represent the absolute maximum value that will
|
||||
* be reported for a positive position value. Some Synaptics firmware
|
||||
* uses this value to indicate a finger near the edge of the touchpad
|
||||
* whose precise position cannot be determined.
|
||||
*
|
||||
* At least one touchpad is known to report positions in excess of this
|
||||
* value which are actually negative values truncated to the 13-bit
|
||||
* reporting range. These values have never been observed to be lower
|
||||
* than 8184 (i.e. -8), so we treat all values greater than 8176 as
|
||||
* negative and any other value as positive.
|
||||
*/
|
||||
#define X_MAX_POSITIVE (((1 << ABS_POS_BITS) + XMAX) / 2)
|
||||
#define Y_MAX_POSITIVE (((1 << ABS_POS_BITS) + YMAX) / 2)
|
||||
#define X_MAX_POSITIVE 8176
|
||||
#define Y_MAX_POSITIVE 8176
|
||||
|
||||
/*****************************************************************************
|
||||
* Stuff we need even when we do not want native Synaptics support
|
||||
@@ -604,11 +609,21 @@ static int synaptics_parse_hw_state(const unsigned char buf[],
|
||||
hw->right = (buf[0] & 0x02) ? 1 : 0;
|
||||
}
|
||||
|
||||
/* Convert wrap-around values to negative */
|
||||
/*
|
||||
* Convert wrap-around values to negative. (X|Y)_MAX_POSITIVE
|
||||
* is used by some firmware to indicate a finger at the edge of
|
||||
* the touchpad whose precise position cannot be determined, so
|
||||
* convert these values to the maximum axis value.
|
||||
*/
|
||||
if (hw->x > X_MAX_POSITIVE)
|
||||
hw->x -= 1 << ABS_POS_BITS;
|
||||
else if (hw->x == X_MAX_POSITIVE)
|
||||
hw->x = XMAX;
|
||||
|
||||
if (hw->y > Y_MAX_POSITIVE)
|
||||
hw->y -= 1 << ABS_POS_BITS;
|
||||
else if (hw->y == Y_MAX_POSITIVE)
|
||||
hw->y = YMAX;
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
Reference in New Issue
Block a user