drivers/watchdog/lenovo_se10_wdt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
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(-)
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;
--
2.25.1
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>
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>
© 2016 - 2025 Red Hat, Inc.