Re: [PATCH RFC 0/4] input: touchscreen: add initial support for Goodix Berlin touchscreen IC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 06/06/2023 17:31, Hans de Goede wrote:
Hi Neil,

On 6/6/23 16:31, Neil Armstrong wrote:
These touchscreen ICs support SPI, I2C and I3C interface, up to
10 finger touch, stylus and gestures events.

This initial driver is derived from the Goodix goodix_ts_berlin
available at [1] and [2] and only supports the GT9916 IC
present on the Qualcomm SM8550 MTP & QRD touch panel.

The current implementation only supports BerlinD, aka GT9916.

Support for advanced features like:
- Firmware & config update
- Stylus events
- Gestures events
- Previous revisions support (BerlinA or BerlinB)
is not included in current version.

The current support will work with currently flashed firmware
and config, and bail out if firmware or config aren't flashed yet.

What I'm missing here / in the commit msg of
"input: touchscreen: add core support for Goodix Berlin Touchscreen IC"

is an explanation why this is a new driver instead of adding
support to the existing goodix.c code.

I assume you have good reasons for this, but it would be good
if you can write the reasons for this down.

Sure, should I write it down here and/or update the commit message in a new revision ?

Anyway, here's the reasons:
- globally the event handling "looks like" the current goodix.c, but again the offsets
are again different and none of the register address are the same, and unlike the current
support all registers are provided by the "ic_info" structure
- while with the current code it *could* be possible to merge it, with a lot of changes,
the firmware management looks really different, and it would be really hard to merge.

But I may be wrong, and may be misleaded by the goodix driver structure (even if it
went through a really heavy cleaning process).

Globally it seems they tried to match the "event handling" process of the previous
generations, but the firmware interface is completely different.

Neil


Regards,

Hans




[1] https://github.com/goodix/goodix_ts_berlin
[2] https://git.codelinaro.org/clo/la/platform/vendor/opensource/touch-drivers

Signed-off-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx>
---
Neil Armstrong (4):
       dt-bindings: input: document Goodix Berlin Touchscreen IC
       input: touchscreen: add core support for Goodix Berlin Touchscreen IC
       input: touchscreen: add I2C support for Goodix Berlin Touchscreen IC
       input: touchscreen: add SPI support for Goodix Berlin Touchscreen IC

  .../bindings/input/touchscreen/goodix-berlin.yaml  |  81 ++
  drivers/input/touchscreen/Kconfig                  |  33 +
  drivers/input/touchscreen/Makefile                 |   3 +
  drivers/input/touchscreen/goodix_berlin.h          | 228 +++++
  drivers/input/touchscreen/goodix_berlin_core.c     | 935 +++++++++++++++++++++
  drivers/input/touchscreen/goodix_berlin_i2c.c      |  76 ++
  drivers/input/touchscreen/goodix_berlin_spi.c      | 183 ++++
  7 files changed, 1539 insertions(+)
---
base-commit: 6db29e14f4fb7bce9eb5290288e71b05c2b0d118
change-id: 20230606-topic-goodix-berlin-upstream-initial-ba97e8ec8f4c

Best regards,





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux