Re: [PATCH 0/6] Add touch-keys support to the Zinitix touch driver

From: Nikita Travkin
Date: Tue Nov 09 2021 - 10:23:15 EST


Hi Linus,

Linus Walleij писал(а) 09.11.2021 09:45:
Hi Nikita,

On Wed, Oct 27, 2021 at 8:15 PM Nikita Travkin <nikita@xxxxxxx> 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