mbox series

[0/2] Add support for Goodix GT9286 chip

Message ID 20210109135512.149032-1-angelogioacchino.delregno@somainline.org (mailing list archive)
Headers show
Series Add support for Goodix GT9286 chip | expand

Message

AngeloGioacchino Del Regno Jan. 9, 2021, 1:55 p.m. UTC 
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(+)

Comments

Bastien Nocera Jan. 9, 2021, 2:37 p.m. UTC | #1 
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(+)
>
Dmitry Torokhov Jan. 10, 2021, 6:17 a.m. UTC | #2 
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.