When the firmware identifies a contact as a palm the driver sets the tool type to MT_TOOL_PALM, but sets the slot state as active. Reporting the palm as active results in userspace input libraries considering the palm as a valid contact. Touchpads which previously were using hid-multitouch are now not suppressing palms when switching to the RMI4 driver. This change fixes palm rejection when using the RMI4 driver. Signed-off-by: Andrew Duggan <aduggan@xxxxxxxxxxxxx> Tested-by: Cameron Gutman <aicommander@xxxxxxxxx> --- drivers/input/rmi4/rmi_2d_sensor.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/input/rmi4/rmi_2d_sensor.c b/drivers/input/rmi4/rmi_2d_sensor.c index 8bb866c..8d1f295 100644 --- a/drivers/input/rmi4/rmi_2d_sensor.c +++ b/drivers/input/rmi4/rmi_2d_sensor.c @@ -80,7 +80,8 @@ void rmi_2d_sensor_abs_report(struct rmi_2d_sensor *sensor, input_mt_slot(input, slot); input_mt_report_slot_state(input, obj->mt_tool, - obj->type != RMI_2D_OBJECT_NONE); + (obj->type != RMI_2D_OBJECT_NONE) + && (obj->type != RMI_2D_OBJECT_PALM)); if (obj->type != RMI_2D_OBJECT_NONE) { obj->x = sensor->tracking_pos[slot].x; -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html