On Tue, Jul 24, 2018 at 12:34 AM Dmitry Torokhov
[off-list ref] wrote:

On Fri, Jul 20, 2018 at 10:51:21PM +0100, Nick Dyer wrote:

quoted

From: Nick Dyer <redacted>

input_mt_report_slot_state() ignores the tool when the slot is closed.
Remove the tool type from these function calls, which has caused a bit of
confusion.

Hmm, maybe we could introduce MT_TOOL_NONE or MT_TOOL_INACTIVE and get
rid of the 3rd parameter? It will require a bit of macro trickery for a
release or 2...

I am not sure what would be the benefit of adding those new tools, if
the input_mt code discards them. Do you want to forward them to the
userspace with the release?
This reminds me the discussion we had recently with the touchscreens
releasing the slots with a MT_TOOL_PALM.

Anyway, better include Peter as he will be using this new MT_TOOL.

Cheers,
Benjamin

quoted

Signed-off-by: Nick Dyer <redacted>
---
 drivers/input/touchscreen/atmel_mxt_ts.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c
index d7023d261458..c31af790ef84 100644
--- a/drivers/input/touchscreen/atmel_mxt_ts.c
+++ b/drivers/input/touchscreen/atmel_mxt_ts.c

@@ -838,8 +838,7 @@ static void mxt_proc_t9_message(struct mxt_data *data, u8 *message)
               * have happened.
               */
              if (status & MXT_T9_RELEASE) {
-                     input_mt_report_slot_state(input_dev,
-                                                MT_TOOL_FINGER, 0);
+                     input_mt_report_slot_state(input_dev, 0, 0);
                      mxt_input_sync(data);
              }

@@ -855,7 +854,7 @@ static void mxt_proc_t9_message(struct mxt_data *data, u8 *message)
              input_report_abs(input_dev, ABS_MT_TOUCH_MAJOR, area);
      } else {
              /* Touch no longer active, close out slot */
-             input_mt_report_slot_state(input_dev, MT_TOOL_FINGER, 0);
+             input_mt_report_slot_state(input_dev, 0, 0);
      }

      data->update_input = true;
--

2.17.1

--
Dmitry