diff mbox series

touchscreen/goodix.c:remove no needed commas

Message ID 20210524093354.1356-1-ayowoe@163.com (mailing list archive)
State New
Headers show
Series touchscreen/goodix.c:remove no needed commas | expand

Commit Message

mateng May 24, 2021, 9:33 a.m. UTC
From: mateng <mateng@yulong.com>

remove no needed commas

Signed-off-by: mateng <mateng@yulong.com>
---
 drivers/input/touchscreen/goodix.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Bastien Nocera May 24, 2021, 10:09 a.m. UTC | #1
On Mon, 2021-05-24 at 17:33 +0800, mateng wrote:
> From: mateng <mateng@yulong.com>
> 
> remove no needed commas

There's really no reason to be shuffling around commas. This isn't
helpful in the slightest. NAK.
diff mbox series

Patch

diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c
index c682b02..35febde 100644
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -190,8 +190,8 @@  static int goodix_check_cfg_16(struct goodix_ts_data *ts,
 			/* tPAD is too generic, also match on bios date */
 			DMI_MATCH(DMI_BOARD_VENDOR, "TECLAST"),
 			DMI_MATCH(DMI_BOARD_NAME, "tPAD"),
-			DMI_MATCH(DMI_BIOS_DATE, "12/19/2014"),
-		},
+			DMI_MATCH(DMI_BIOS_DATE, "12/19/2014")
+		}
 	},
 	{
 		.ident = "Teclast X98 Pro",
@@ -202,8 +202,8 @@  static int goodix_check_cfg_16(struct goodix_ts_data *ts,
 			 * (sometimes)...
 			 */
 			DMI_MATCH(DMI_BOARD_VENDOR, "TECLAST"),
-			DMI_MATCH(DMI_BIOS_DATE, "10/28/2015"),
-		},
+			DMI_MATCH(DMI_BIOS_DATE, "10/28/2015")
+		}
 	},
 	{
 		.ident = "WinBook TW100",
@@ -217,7 +217,7 @@  static int goodix_check_cfg_16(struct goodix_ts_data *ts,
 		.matches = {
 			DMI_MATCH(DMI_SYS_VENDOR, "WinBook"),
 			DMI_MATCH(DMI_PRODUCT_NAME, "TW700")
-		},
+		}
 	},
 #endif
 	{}
@@ -246,7 +246,7 @@  static int goodix_check_cfg_16(struct goodix_ts_data *ts,
 		.matches = {
 			DMI_MATCH(DMI_SYS_VENDOR, "Cube"),
 			DMI_MATCH(DMI_PRODUCT_NAME, "I15-TC")
-		},
+		}
 	},
 #endif
 	{}