2020-03-02 10:19:19

by Yannick FERTRE

Subject: [PATCH v1 0/2] goodix: support of gt9147

Add support of GT9147 for Goodix touchscreen.
The chip data on GT9147 is similar to GT912, like
- config data register has 0x8047 address
- config data register max len is 240
- config data checksum has 8-bit


Yannick Fertre (2):
dt-bindings: touchscreen: goodix: support of gt9147
Input: goodix - support gt9147 touchpanel

Documentation/devicetree/bindings/input/touchscreen/goodix.yaml | 1 +
drivers/input/touchscreen/goodix.c | 2 ++
2 files changed, 3 insertions(+)

--
2.7.4

2020-03-02 10:19:31

by Yannick FERTRE

Subject: [PATCH v1 2/2] Input: goodix - support gt9147 touchpanel

Add support for it by adding compatible and supported chip data
(default settings used).
The chip data on GT9147 is similar to GT912, like
- config data register has 0x8047 address
- config data register max len is 240
- config data checksum has 8-bit

Signed-off-by: Yannick Fertre <[email protected]>
---
drivers/input/touchscreen/goodix.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c
index 0403102..8851a35 100644
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -251,6 +251,7 @@ static const struct goodix_chip_data *goodix_get_chip_data(u16 id)
return &gt911_chip_data;

case 912:
+ case 9147:
case 967:
return &gt967_chip_data;

@@ -1054,6 +1055,7 @@ static const struct of_device_id goodix_of_match[] = {
{ .compatible = "goodix,gt9271" },
{ .compatible = "goodix,gt928" },
{ .compatible = "goodix,gt967" },
+ { .compatible = "goodix,gt9147",},
{ }
};
MODULE_DEVICE_TABLE(of, goodix_of_match);
--
2.7.4

2020-03-02 11:00:42

by Bastien Nocera

Subject: Re: [PATCH v1 2/2] Input: goodix - support gt9147 touchpanel

On Mon, 2020-03-02 at 11:18 +0100, Yannick Fertre wrote:
> Add support for it by adding compatible and supported chip data
> (default settings used).
> The chip data on GT9147 is similar to GT912, like
> - config data register has 0x8047 address
> - config data register max len is 240
> - config data checksum has 8-bit
>
> Signed-off-by: Yannick Fertre <[email protected]>


Both patches:
Reviewed-by: Bastien Nocera <[email protected]>

But you'll likely need to send another update on top of the "Add
support for Goodix GT917S touch controller" patchset, which makes some
changes to goodix_get_chip_data().