On Mon, Oct 19, 2020 at 10:53:42AM -0700, trix@xxxxxxxxxx wrote: > From: Tom Rix <trix@xxxxxxxxxx> > > A break is not needed if it is preceded by a return > > Signed-off-by: Tom Rix <trix@xxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> > --- > drivers/watchdog/geodewdt.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/watchdog/geodewdt.c b/drivers/watchdog/geodewdt.c > index 83418924e30a..0b699c783d57 100644 > --- a/drivers/watchdog/geodewdt.c > +++ b/drivers/watchdog/geodewdt.c > @@ -150,8 +150,6 @@ static long geodewdt_ioctl(struct file *file, unsigned int cmd, > case WDIOC_GETSUPPORT: > return copy_to_user(argp, &ident, > sizeof(ident)) ? -EFAULT : 0; > - break; > - > case WDIOC_GETSTATUS: > case WDIOC_GETBOOTSTATUS: > return put_user(0, p); > -- > 2.18.1 >
