On Fri, Nov 02, 2018 at 07:21:11PM +0000, Fabrizio Castro wrote: Do not use "," but ";" to separate instructions. > Signed-off-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> > --- > drivers/watchdog/renesas_wdt.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/renesas_wdt.c b/drivers/watchdog/renesas_wdt.c > index 0d74c3e..b570962 100644 > --- a/drivers/watchdog/renesas_wdt.c > +++ b/drivers/watchdog/renesas_wdt.c > @@ -220,8 +220,8 @@ static int rwdt_probe(struct platform_device *pdev) > goto out_pm_disable; > } > > - priv->wdev.info = &rwdt_ident, > - priv->wdev.ops = &rwdt_ops, > + priv->wdev.info = &rwdt_ident; > + priv->wdev.ops = &rwdt_ops; > priv->wdev.parent = &pdev->dev; > priv->wdev.min_timeout = 1; > priv->wdev.max_timeout = DIV_BY_CLKS_PER_SEC(priv, 65536); > -- > 2.7.4 >