On Thu, 8 Jan 2015, Ping Cheng wrote:
> Signed-off-by: Ping Cheng <pi...@wacom.com>
Could you please provide changelog? I.e. why is this change
needed/desired?
Thanks.
> ---
> drivers/hid/wacom_wac.c | 50
> +++++++++++++++++++++++++------------------------
> 1 file changed, 26 insertions(+), 24 deletions(-)
>
> diff --git a/drivers/hid/wacom_wac.c b/drivers/hid/wacom_wac.c
> index 596a6fb5..4ccf622 100644
> --- a/drivers/hid/wacom_wac.c
> +++ b/drivers/hid/wacom_wac.c
> @@ -534,9 +534,24 @@ static int wacom_intuos_inout(struct wacom_wac *wacom)
> return 1;
> }
>
> + /*
> + * don't report events for invalid data
> + */
> /* older I4 styli don't work with new Cintiqs */
> - if (!((wacom->id[idx] >> 20) & 0x01) &&
> - (features->type == WACOM_21UX2))
> + if ((!((wacom->id[idx] >> 20) & 0x01) &&
> + (features->type == WACOM_21UX2)) ||
> + /* Only large Intuos support Lense Cursor */
> + (wacom->tool[idx] == BTN_TOOL_LENS &&
> + (features->type == INTUOS3 ||
> + features->type == INTUOS3S ||
> + features->type == INTUOS4 ||
> + features->type == INTUOS4S ||
> + features->type == INTUOS5 ||
> + features->type == INTUOS5S ||
> + features->type == INTUOSPM ||
> + features->type == INTUOSPS)) ||
> + /* Cintiq doesn't send data when RDY bit isn't set */
> + (features->type == CINTIQ && !(data[1] & 0x40)))
> return 1;
>
> /* Range Report */
> @@ -553,6 +568,10 @@ static int wacom_intuos_inout(struct wacom_wac *wacom)
> if (features->quirks & WACOM_QUIRK_MULTI_INPUT)
> wacom->shared->stylus_in_proximity = false;
>
> + /* don't report exit if we don't know the ID */
> + if (!wacom->id[idx])
> + return 1;
> +
> /*
> * Reset all states otherwise we lose the initial states
> * when in-prox next time
> @@ -585,6 +604,11 @@ static int wacom_intuos_inout(struct wacom_wac *wacom)
> wacom->id[idx] = 0;
> return 2;
> }
> +
> + /* don't report other events if we don't know the ID */
> + if (!wacom->id[idx])
> + return 1;
> +
> return 0;
> }
>
> @@ -842,28 +866,6 @@ static int wacom_intuos_irq(struct wacom_wac *wacom)
> if (result)
> return result - 1;
>
> - /* don't proceed if we don't know the ID */
> - if (!wacom->id[idx])
> - return 0;
> -
> - /* Only large Intuos support Lense Cursor */
> - if (wacom->tool[idx] == BTN_TOOL_LENS &&
> - (features->type == INTUOS3 ||
> - features->type == INTUOS3S ||
> - features->type == INTUOS4 ||
> - features->type == INTUOS4S ||
> - features->type == INTUOS5 ||
> - features->type == INTUOS5S ||
> - features->type == INTUOSPM ||
> - features->type == INTUOSPS)) {
> -
> - return 0;
> - }
> -
> - /* Cintiq doesn't send data when RDY bit isn't set */
> - if (features->type == CINTIQ && !(data[1] & 0x40))
> - return 0;
> -
> if (features->type >= INTUOS3S) {
> input_report_abs(input, ABS_X, (data[2] << 9) | (data[3] << 1)
> | ((data[9] >> 1) & 1));
> input_report_abs(input, ABS_Y, (data[4] << 9) | (data[5] << 1)
> | (data[9] & 1));
> --
> 1.9.1
>
--
Jiri Kosina
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
