# \[PATCH\] touchscreen/goodix.c:unify format

- Subsystems: GOODIX TOUCHSCREEN, INPUT (KEYBOARD, MOUSE, JOYSTICK, TOUCHSCREEN) DRIVERS, THE REST
- Maintainers: Hans de Goede, Dmitry Torokhov, Linus Torvalds
- List: linux-input
- 2 messages, 2 authors
- Page: https://ratatoskr.run/linux-input/2021/05/1388922/t

## \[PATCH\] touchscreen/goodix.c:unify format

- From: mateng \<hidden\>
- Date: 2021-05-24 02:46:54 
- Message: https://ratatoskr.run/linux-input/2021/05/1388922.md

```text
From: mateng <redacted>

unify format

Signed-off-by: mateng <redacted>
---
 drivers/input/touchscreen/goodix.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c
index c682b02..ab12065 100644
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -210,7 +210,7 @@ static int goodix_check_cfg_16(struct goodix_ts_data *ts,
 		.matches = {
 			DMI_MATCH(DMI_SYS_VENDOR, "WinBook"),
 			DMI_MATCH(DMI_PRODUCT_NAME, "TW100")
-		}
+		},
 	},
 	{
 		.ident = "WinBook TW700",
@@ -230,7 +230,7 @@ static int goodix_check_cfg_16(struct goodix_ts_data *ts,
 		/* YB1-X91L/F and YB1-X90L/F */
 		.matches = {
 			DMI_MATCH(DMI_PRODUCT_NAME, "Lenovo YB1-X9")
-		}
+		},
 	},
 #endif
 	{}
-- 
1.9.1


```

## Re: \[PATCH\] touchscreen/goodix.c:unify format

- From: Bastien Nocera \<hadess@hadess.net\>
- Date: 2021-05-24 08:48:21 
- Message: https://ratatoskr.run/linux-input/2021/05/1389087.md

```text
On Mon, 2021-05-24 at 10:46 +0800, mateng wrote:
> From: mateng <redacted>
> 
> unify format

This doesn't "unify the format", it adds commas where none are needed.

NAK

> 
> Signed-off-by: mateng <redacted>
> ---
>  drivers/input/touchscreen/goodix.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/input/touchscreen/goodix.c
> b/drivers/input/touchscreen/goodix.c
> index c682b02..ab12065 100644
> --- a/drivers/input/touchscreen/goodix.c
> +++ b/drivers/input/touchscreen/goodix.c
> @@ -210,7 +210,7 @@ static int goodix_check_cfg_16(struct
> goodix_ts_data *ts,
>                 .matches = {
>                         DMI_MATCH(DMI_SYS_VENDOR, "WinBook"),
>                         DMI_MATCH(DMI_PRODUCT_NAME, "TW100")
> -               }
> +               },
>         },
>         {
>                 .ident = "WinBook TW700",
> @@ -230,7 +230,7 @@ static int goodix_check_cfg_16(struct
> goodix_ts_data *ts,
>                 /* YB1-X91L/F and YB1-X90L/F */
>                 .matches = {
>                         DMI_MATCH(DMI_PRODUCT_NAME, "Lenovo YB1-X9")
> -               }
> +               },
>         },
>  #endif
>         {}


```