On Sat, Nov 21, 2020 at 10:49:51AM -0800, Guenter Roeck wrote: > On Fri, Nov 20, 2020 at 12:32:51PM -0600, Gustavo A. R. Silva wrote: > > In preparation to enable -Wimplicit-fallthrough for Clang, fix a warning > > by explicitly adding a fallthrough pseudo-keyword instead of letting the > > code fall through to the next case. > > > > Link: https://github.com/KSPP/linux/issues/115 > > Signed-off-by: Gustavo A. R. Silva <gustavo...@kernel.org> > > --- > > drivers/watchdog/machzwd.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/watchdog/machzwd.c b/drivers/watchdog/machzwd.c > > index 743377c5b173..73f2221f6222 100644 > > --- a/drivers/watchdog/machzwd.c > > +++ b/drivers/watchdog/machzwd.c > > @@ -174,6 +174,7 @@ static inline void zf_set_timer(unsigned short new, > > unsigned char n) > > fallthrough; > > case WD2: > > zf_writeb(COUNTER_2, new > 0xff ? 0xff : new); > > + fallthrough; > > fallthrough to return ? Oh well, this is an old style driver anyway, > so I guess who cares.
:) > Acked-by: Guenter Roeck <li...@roeck-us.net> Thanks, Guenter. -- Gustavo