Message ID | 20240716030725.400400-1-nichen@iscas.ac.cn (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | watchdog: lenovo_se10_wdt: Convert comma to semicolon | expand |
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>
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;
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(-)