lkml.org 
[lkml]   [2021]   [Nov]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 0/6] Add touch-keys support to the Zinitix touch driver
Hi Linus,

Linus Walleij писал(а) 09.11.2021 09:45:
> Hi Nikita,
>
> On Wed, Oct 27, 2021 at 8:15 PM Nikita Travkin <nikita@trvn.ru> wrote:
>
>> This series adds support for the touch-keys that can be present on
>> some
>> touchscreen configurations, adds the compatible for bt532 and fixes a
>> small race condition bug in the driver probe function.
>>
>> I also pick up the series that converts the dt bindings to yaml
>> initially submitted by Linus Walleij in [1].
>> I made some minor changes to those patches:
>> - Fixed dt_schema_check error
>> - Adressed the review comments from Dmitry on the original series
>
> Thanks for picking this up!
>
> Have you notices some behaviour like surplus touch events
> (like many press/release events fall through to the UI)
> when using this driver? I think it might need some z fuzzing
> but I am not sure.
>

On my device (8 inch tablet with BT532) I saw no problems with touch
so far. However another person with a different tablet (10 inch with
ZT7554)
indeed says that they notice "multiplied" touches that make typing hard
so maybe that depends on controller model/firmware...

And speaking of that ZT7554: Seems like it's works with the driver
and I'd like to add the compatible for it in v2 but I'd also have to add
it
to the bindings. Looking at how you add all other similar names for BT*
there
does it make sense to add ZT* as well? Maybe you have some hints where
to look
for a list of the models?

I was planning to send a v2 with all the review fixes near the end of
the week
but I've noticed a yet another quirky issue with the touch controller:
At least on my device, for some reason enabling touchkeys changes the
way the
controller reports the finger touch events which breaks multi-touch...
Assuming that *not* enabling the touchkeys leads to calibration being
wrong
(controller assigns the touchkey sense lines to the touch area in that
case)
I now have to resolve this quirk as well...

Thanks,
Nikita

> Yours,
> Linus Walleij

\
 
 \ /
  Last update: 2021-11-09 16:24    [W:0.148 / U:0.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site