diff mbox series

watchdog: lenovo_se10_wdt: Convert comma to semicolon

Message ID 20240716030725.400400-1-nichen@iscas.ac.cn
State New
Headers show
Series watchdog: lenovo_se10_wdt: Convert comma to semicolon | expand

Commit Message

Chen Ni July 16, 2024, 3:07 a.m. UTC
Replace a comma between expression statements by a semicolon.

Fixes: 1f6602c8ed1e ("watchdog: lenovo_se10_wdt: Watchdog driver for Lenovo SE10 platform")
Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
---
 drivers/watchdog/lenovo_se10_wdt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Mark Pearson July 16, 2024, 2:23 p.m. UTC | #1
Hi Chen,

Thank you for fixing this - I should have caught this originally :( 
I'm slightly bemused that it worked at all.

On Tue, Jul 16, 2024, at 1:29 AM, Guenter Roeck wrote:
> On 7/15/24 20:07, Chen Ni wrote:
>> Replace a comma between expression statements by a semicolon.
>> 
>> Fixes: 1f6602c8ed1e ("watchdog: lenovo_se10_wdt: Watchdog driver for Lenovo SE10 platform")
>> Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
>
> Reviewed-by: Guenter Roeck <linux@roeck-us.net>

Reviewed-by: Mark Pearson <mpearson-lenovo@squebb.ca>
diff mbox series

Patch

diff --git a/drivers/watchdog/lenovo_se10_wdt.c b/drivers/watchdog/lenovo_se10_wdt.c
index 139ff0e8220f..cd0500e5080b 100644
--- a/drivers/watchdog/lenovo_se10_wdt.c
+++ b/drivers/watchdog/lenovo_se10_wdt.c
@@ -196,8 +196,8 @@  static int se10_wdt_probe(struct platform_device *pdev)
 	watchdog_set_drvdata(&priv->wdd, priv);
 
 	priv->wdd.parent = dev;
-	priv->wdd.info = &wdt_info,
-	priv->wdd.ops = &se10_wdt_ops,
+	priv->wdd.info = &wdt_info;
+	priv->wdd.ops = &se10_wdt_ops;
 	priv->wdd.timeout = WATCHDOG_TIMEOUT; /* Set default timeout */
 	priv->wdd.min_timeout = MIN_TIMEOUT;
 	priv->wdd.max_timeout = MAX_TIMEOUT;