Skip to content

Commit

Permalink
HID: wacom: do not directly use input_mt_report_pointer_emulation
Browse files Browse the repository at this point in the history
input_mt_sync_frame() calls input_mt_report_pointer_emulation() and do
some internal steps required to keep in sync the state of the touch within
the various reports.

Given that we use input_mt_get_slot_by_key() in this driver, it is better to
use input_mt_sync_frame().

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Reviewed-by: Ping Cheng <pingc@wacom.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
  • Loading branch information
bentiss authored and Jiri Kosina committed Feb 18, 2015
1 parent 870fd0f commit 9a1c001
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions drivers/hid/wacom_wac.c
Original file line number Diff line number Diff line change
Expand Up @@ -1093,7 +1093,7 @@ static int wacom_24hdt_irq(struct wacom_wac *wacom)
}
}
}
input_mt_report_pointer_emulation(input, true);
input_mt_sync_frame(input);

wacom->num_contacts_left -= contacts_to_send;
if (wacom->num_contacts_left <= 0)
Expand Down Expand Up @@ -1144,7 +1144,7 @@ static int wacom_mt_touch(struct wacom_wac *wacom)
input_report_abs(input, ABS_MT_POSITION_Y, y);
}
}
input_mt_report_pointer_emulation(input, true);
input_mt_sync_frame(input);

wacom->num_contacts_left -= contacts_to_send;
if (wacom->num_contacts_left < 0)
Expand Down Expand Up @@ -1176,7 +1176,7 @@ static int wacom_tpc_mt_touch(struct wacom_wac *wacom)
contact_with_no_pen_down_count++;
}
}
input_mt_report_pointer_emulation(input, true);
input_mt_sync_frame(input);

/* keep touch state for pen event */
wacom->shared->touch_down = (contact_with_no_pen_down_count > 0);
Expand Down Expand Up @@ -1638,7 +1638,7 @@ static int wacom_bpt_touch(struct wacom_wac *wacom)
}
}

input_mt_report_pointer_emulation(input, true);
input_mt_sync_frame(input);

input_report_key(pad_input, BTN_LEFT, (data[1] & 0x08) != 0);
input_report_key(pad_input, BTN_FORWARD, (data[1] & 0x04) != 0);
Expand Down Expand Up @@ -1728,7 +1728,7 @@ static int wacom_bpt3_touch(struct wacom_wac *wacom)
wacom_bpt3_button_msg(wacom, data + offset);

}
input_mt_report_pointer_emulation(input, true);
input_mt_sync_frame(input);

return 1;
}
Expand Down

0 comments on commit 9a1c001

Please sign in to comment.