| Message ID | fa4451efd21e287f8fdf2f7f8495b070544209c0.1631699262.git.geert+renesas@glider.be (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | watchdog: rza_wdt: Use semicolons instead of commas | expand |
On 9/15/21 2:48 AM, Geert Uytterhoeven wrote: > This code works, but it is cleaner to use semicolons at the end of > statements instead of commas. > > Extracted from a big anonymous patch by Julia Lawall > <julia.lawall@inria.fr>. > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Reviewed-by: Guenter Roeck <linux@roeck-us.net> > --- > drivers/watchdog/rza_wdt.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/rza_wdt.c b/drivers/watchdog/rza_wdt.c > index 7b6c365f7cd36297..fe6c2ed35e04cc4b 100644 > --- a/drivers/watchdog/rza_wdt.c > +++ b/drivers/watchdog/rza_wdt.c > @@ -189,8 +189,8 @@ static int rza_wdt_probe(struct platform_device *pdev) > return -ENOENT; > } > > - priv->wdev.info = &rza_wdt_ident, > - priv->wdev.ops = &rza_wdt_ops, > + priv->wdev.info = &rza_wdt_ident; > + priv->wdev.ops = &rza_wdt_ops; > priv->wdev.parent = dev; > > priv->cks = (u8)(uintptr_t) of_device_get_match_data(dev); >
diff --git a/drivers/watchdog/rza_wdt.c b/drivers/watchdog/rza_wdt.c index 7b6c365f7cd36297..fe6c2ed35e04cc4b 100644 --- a/drivers/watchdog/rza_wdt.c +++ b/drivers/watchdog/rza_wdt.c @@ -189,8 +189,8 @@ static int rza_wdt_probe(struct platform_device *pdev) return -ENOENT; } - priv->wdev.info = &rza_wdt_ident, - priv->wdev.ops = &rza_wdt_ops, + priv->wdev.info = &rza_wdt_ident; + priv->wdev.ops = &rza_wdt_ops; priv->wdev.parent = dev; priv->cks = (u8)(uintptr_t) of_device_get_match_data(dev);
This code works, but it is cleaner to use semicolons at the end of statements instead of commas. Extracted from a big anonymous patch by Julia Lawall <julia.lawall@inria.fr>. Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> --- drivers/watchdog/rza_wdt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)