| Message ID | 20240903012620.204247-1-nichen@iscas.ac.cn (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v2] watchdog: iTCO_wdt: Convert comma to semicolon | expand |
On 9/2/24 18:26, Chen Ni wrote: > Replace a comma between expression statements by a semicolon. > > Signed-off-by: Chen Ni <nichen@iscas.ac.cn> > --- > Changelog: > > v1 -> v2: > > 1. Drop Fixes tag. Please don't do that. Not only was v2 unnecessary, with it you unnecessarily dropped not only one but _two_ Reviewed-by: tags. Guenter > --- > drivers/watchdog/iTCO_wdt.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/iTCO_wdt.c b/drivers/watchdog/iTCO_wdt.c > index 264857d314da..35b358bcf94c 100644 > --- a/drivers/watchdog/iTCO_wdt.c > +++ b/drivers/watchdog/iTCO_wdt.c > @@ -563,8 +563,8 @@ static int iTCO_wdt_probe(struct platform_device *pdev) > } > > ident.firmware_version = p->iTCO_version; > - p->wddev.info = &ident, > - p->wddev.ops = &iTCO_wdt_ops, > + p->wddev.info = &ident; > + p->wddev.ops = &iTCO_wdt_ops; > p->wddev.bootstatus = 0; > p->wddev.timeout = WATCHDOG_TIMEOUT; > watchdog_set_nowayout(&p->wddev, nowayout);
diff --git a/drivers/watchdog/iTCO_wdt.c b/drivers/watchdog/iTCO_wdt.c index 264857d314da..35b358bcf94c 100644 --- a/drivers/watchdog/iTCO_wdt.c +++ b/drivers/watchdog/iTCO_wdt.c @@ -563,8 +563,8 @@ static int iTCO_wdt_probe(struct platform_device *pdev) } ident.firmware_version = p->iTCO_version; - p->wddev.info = &ident, - p->wddev.ops = &iTCO_wdt_ops, + p->wddev.info = &ident; + p->wddev.ops = &iTCO_wdt_ops; p->wddev.bootstatus = 0; p->wddev.timeout = WATCHDOG_TIMEOUT; watchdog_set_nowayout(&p->wddev, nowayout);
Replace a comma between expression statements by a semicolon. Signed-off-by: Chen Ni <nichen@iscas.ac.cn> --- Changelog: v1 -> v2: 1. Drop Fixes tag. --- drivers/watchdog/iTCO_wdt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)