[PATCH 0/2] Add support for Goodix GT9286 chip

[PATCH 0/2] Add support for Goodix GT9286 chip

[AngeloGioacchino Del Regno]

Add support for the GT9286 chip, tested on F(x)Tec Pro1 (MSM8998).

AngeloGioacchino Del Regno (2):
  input: goodix: Add support for Goodix GT9286 chip
  dt-bindings: ts: goodix: Add binding for GT9286 IC

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

-- 
2.29.2

[PATCH 1/2] input: goodix: Add support for Goodix GT9286 chip

[AngeloGioacchino Del Regno]

The Goodix GT9286 is a capacitive touch sensor IC based on GT1x.

This chip can be found on a number of smartphones, including the
F(x)tec Pro 1 and the Elephone U.

Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
---
 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 02c75ea385e0..becc5af21801 100644
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -157,6 +157,7 @@ static const struct goodix_chip_id goodix_chip_ids[] = {
 	{ .id = "5663", .data = &gt1x_chip_data },
 	{ .id = "5688", .data = &gt1x_chip_data },
 	{ .id = "917S", .data = &gt1x_chip_data },
+	{ .id = "9286", .data = &gt1x_chip_data },
 
 	{ .id = "911", .data = &gt911_chip_data },
 	{ .id = "9271", .data = &gt911_chip_data },
@@ -1433,6 +1434,7 @@ static const struct of_device_id goodix_of_match[] = {
 	{ .compatible = "goodix,gt927" },
 	{ .compatible = "goodix,gt9271" },
 	{ .compatible = "goodix,gt928" },
+	{ .compatible = "goodix,gt9286" },
 	{ .compatible = "goodix,gt967" },
 	{ }
 };
-- 
2.29.2

[PATCH 2/2] dt-bindings: ts: goodix: Add binding for GT9286 IC

[AngeloGioacchino Del Regno]

Support for this chip was added to the goodix driver: add the
DT binding for it.

Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
---
 Documentation/devicetree/bindings/input/touchscreen/goodix.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/input/touchscreen/goodix.yaml b/Documentation/devicetree/bindings/input/touchscreen/goodix.yaml
index da5b0d87e16d..93f2ce3130ae 100644
--- a/Documentation/devicetree/bindings/input/touchscreen/goodix.yaml
+++ b/Documentation/devicetree/bindings/input/touchscreen/goodix.yaml
@@ -26,6 +26,7 @@ properties:
       - goodix,gt927
       - goodix,gt9271
       - goodix,gt928
+      - goodix,gt9286
       - goodix,gt967
 
   reg:
-- 
2.29.2

Re: [PATCH 0/2] Add support for Goodix GT9286 chip

[Bastien Nocera]

On Sat, 2021-01-09 at 14:55 +0100, AngeloGioacchino Del Regno wrote:
> Add support for the GT9286 chip, tested on F(x)Tec Pro1 (MSM8998).

Can you please add this test information to the commit message for the
goodix.c patch?

Feel free to add my:
Reviewed-by: Bastien Nocera <hadess@hadess.net>
to both patches when you send a v2.

Cheers

> 
> AngeloGioacchino Del Regno (2):
>   input: goodix: Add support for Goodix GT9286 chip
>   dt-bindings: ts: goodix: Add binding for GT9286 IC
> 
>  Documentation/devicetree/bindings/input/touchscreen/goodix.yaml | 1
> +
>  drivers/input/touchscreen/goodix.c                              | 2
> ++
>  2 files changed, 3 insertions(+)
> 

Re: [PATCH 0/2] Add support for Goodix GT9286 chip

[Dmitry Torokhov]

On Sat, Jan 09, 2021 at 03:37:40PM +0100, Bastien Nocera wrote:
> On Sat, 2021-01-09 at 14:55 +0100, AngeloGioacchino Del Regno wrote:
> > Add support for the GT9286 chip, tested on F(x)Tec Pro1 (MSM8998).
> 
> Can you please add this test information to the commit message for the
> goodix.c patch?
> 
> Feel free to add my:
> Reviewed-by: Bastien Nocera <hadess@hadess.net>
> to both patches when you send a v2.

I added this to the patch by hand, there is no need to submit v2.

Thanks.

-- 
Dmitry

Re: [PATCH 1/2] input: goodix: Add support for Goodix GT9286 chip

[Dmitry Torokhov]

On Sat, Jan 09, 2021 at 02:55:11PM +0100, AngeloGioacchino Del Regno wrote:
> The Goodix GT9286 is a capacitive touch sensor IC based on GT1x.
> 
> This chip can be found on a number of smartphones, including the
> F(x)tec Pro 1 and the Elephone U.
> 
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>

Applied, thank you.

-- 
Dmitry

Re: [PATCH 2/2] dt-bindings: ts: goodix: Add binding for GT9286 IC

[Dmitry Torokhov]

On Sat, Jan 09, 2021 at 02:55:12PM +0100, AngeloGioacchino Del Regno wrote:
> Support for this chip was added to the goodix driver: add the
> DT binding for it.
> 
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>

Applied, thank you.

-- 
Dmitry