Hi Guenter, > timeout_to_regval() returns a valid error code. Might as well use it. > > Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> > --- > drivers/watchdog/ts72xx_wdt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/watchdog/ts72xx_wdt.c b/drivers/watchdog/ts72xx_wdt.c > index 42913f1..a4f3ce5 100644 > --- a/drivers/watchdog/ts72xx_wdt.c > +++ b/drivers/watchdog/ts72xx_wdt.c > @@ -192,7 +192,7 @@ static int ts72xx_wdt_open(struct inode *inode, struct file *file) > dev_err(&wdt->pdev->dev, > "failed to convert timeout (%d) to register value\n", > timeout); > - return -EINVAL; > + return regval; > } > > if (mutex_lock_interruptible(&wdt->lock)) This patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
