From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4A192C433B4 for ; Fri, 16 Apr 2021 07:16:55 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 15E0060238 for ; Fri, 16 Apr 2021 07:16:55 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234681AbhDPHRR (ORCPT ); Fri, 16 Apr 2021 03:17:17 -0400 Received: from mail.astralinux.ru ([217.74.38.119]:57484 "EHLO mail.astralinux.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230466AbhDPHRR (ORCPT ); Fri, 16 Apr 2021 03:17:17 -0400 Received: from localhost (localhost [127.0.0.1]) by mail.astralinux.ru (Postfix) with ESMTP id 230C935E7571; Fri, 16 Apr 2021 10:16:31 +0300 (MSK) Received: from mail.astralinux.ru ([127.0.0.1]) by localhost (rbta-msk-vsrv-mail01.astralinux.ru [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id EzC_lXdhUdom; Fri, 16 Apr 2021 10:16:30 +0300 (MSK) Received: from localhost (localhost [127.0.0.1]) by mail.astralinux.ru (Postfix) with ESMTP id 5442835E7573; Fri, 16 Apr 2021 10:16:30 +0300 (MSK) X-Virus-Scanned: amavisd-new at astralinux.ru Received: from mail.astralinux.ru ([127.0.0.1]) by localhost (rbta-msk-vsrv-mail01.astralinux.ru [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id a_wrFsdTO1HC; Fri, 16 Apr 2021 10:16:30 +0300 (MSK) Received: from [192.168.32.67] (unknown [192.168.32.67]) by mail.astralinux.ru (Postfix) with ESMTPSA id 25EDD35E7570; Fri, 16 Apr 2021 10:16:30 +0300 (MSK) Subject: Re: UPD: hid-multitouch: goodix: hovering works as touch To: Hans de Goede , Vitaly Minko , linux-input@vger.kernel.org Cc: Benjamin Tissoires , Daniele Gobbetti References: <7b7af64e-8e70-3a9c-eee9-010574adc241@astralinux.ru> <8e25e7c4-53bd-98d5-35da-cb596f84e9c4@minko.me> <62b7301b-a3a1-d49c-2a60-5640984a404a@redhat.com> From: Dmitry Mastykin Message-ID: <7b988eea-62b5-e799-ae5e-d7fa3ebcb9a1@astralinux.ru> Date: Fri, 16 Apr 2021 10:16:29 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: <62b7301b-a3a1-d49c-2a60-5640984a404a@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org Hi Hans, Of course I would like to put a pen patch to upstream, and ready to work for that. Have you taken a look to it? It's seems to be like a workaround. And it's difficult to say if pen button part of the patch will work fine with all gt7385p firmwares (sequence of tip, barrel and eraser is very tricky, but is it stable from version to version?) Should I rebase my patch, provide with comments and send the series to linux-input? Do you have any wishes how I could improve it before? Thank you very much! Dmitry Mastykin On 4/15/21 7:02 PM, Hans de Goede wrote: > Hi, > > On 4/15/21 3:33 PM, Vitaly Minko wrote: >> Hello Dmitry, >> >> >> I have tested your patches on the kernel version 5.10.27. I had to add one define for I2C_DEVICE_ID_GOODIX_0113. Everything else was applied successfully. >> >> The digitizer works fine now. So the original issue is solved. Both pen buttons also work as expected (right and middle mouse buttons). >> >> All tests performed on One-Netbook A1. >> >> Thanks a lot for your efforts. Could you please send me a private email with your credentials for sending a tip (I would like to buy you a coffee). > > Although it is nice that these patches fix things for you, I wonder if there > is any plan to get the fixes for this upstream, so that once distros jump > to a newer kernel this will just work out of the box for all Linux users ? > > Regards, > > Hans > > > > >> 15.04.2021 10:13, Dmitry Mastykin wrote: >>> Hi Vitaly, >>> >>> Yes, we made a quirk for separate non-multitouch processing of the HID_DG_PEN application for Goodix gt7385p (I2C_DEVICE_ID_GOODIX_0113 == 0x0113). It works well, BTN_TOOL_PEN and BTN_TOUCH events have right order. >>> I attach patches for kernel 5.4 >>> Buttons support was very tricky, because the panel tries to imitate eraser usage with combination of tip's and eraser's bits off and on, seems that Windows driver needs this. In second patch we add state-machines for pen buttons to simulate right and middle click. It works also well with this panel/pen, but I'm not sure if it works with other devices. >>> >>> Kind regards, >>> Dmitry Mastykin >>> >>> >>> On 4/14/21 6:35 PM, Vitaly Minko wrote: >>>> Hello Dmitry, >>>> >>>> >>>> I have faced with exactly the same problem. Have you managed to solve the issue? >>>> >>>> If yes, could you please share the patch? >>>> >>>> Thanks in advance! >>>> >>>> >>>> Best regards, >>>> >>>> Vitaly >>>> >> >