Hello Guenter,
On Fri, Aug 07, 2015 at 07:56:51AM -0700, Guenter Roeck wrote:
> > enabled = in_be32(&wd_base->swcrr) & SWCRR_SWEN;
> > if (!enabled && wdt_type->hw_enabled) {
> > pr_info("could not be enabled in software\n");
> >- ret = -ENOSYS;
> >- goto err_unmap;
> >+ return -ENOSYS;
> > }
>
> checkpatch says:
>
> WARNING: ENOSYS means 'invalid syscall nr' and nothing else
> #65: FILE: drivers/watchdog/mpc8xxx_wdt.c:166:
> + return -ENOSYS;
>
> Maybe just use -ENODEV.
That would be a separate patch tough because I'm just changing the code
flow and ENOSYS was there already before.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
