Re: [PATCH] iio: accel: cros_ec_accel_legacy: Mark expected switch fall-throughs

2018-08-19 Thread Jonathan Cameron
On Sat, 18 Aug 2018 17:34:40 +0200 Marcus Folkesson wrote: > Hi Gutavo, > > Sorry for the delay. > > On Wed, Aug 15, 2018 at 12:50:10PM -0500, Gustavo A. R. Silva wrote: > > Hi Marcus, > > > > On 8/15/18 12:27 PM, Marcus Folkesson wrote: > > > Hi, > > > > > > On Wed, Aug 15, 2018 at 11:38:5

Re: [PATCH] iio: accel: cros_ec_accel_legacy: Mark expected switch fall-throughs

2018-08-18 Thread Marcus Folkesson
Hi Gutavo, Sorry for the delay. On Wed, Aug 15, 2018 at 12:50:10PM -0500, Gustavo A. R. Silva wrote: > Hi Marcus, > > On 8/15/18 12:27 PM, Marcus Folkesson wrote: > > Hi, > > > > On Wed, Aug 15, 2018 at 11:38:52AM -0500, Gustavo A. R. Silva wrote: > >> In preparation to enabling -Wimplicit-fall

Re: [PATCH] iio: accel: cros_ec_accel_legacy: Mark expected switch fall-throughs

2018-08-15 Thread Gustavo A. R. Silva
Hi Marcus, On 8/15/18 12:27 PM, Marcus Folkesson wrote: > Hi, > > On Wed, Aug 15, 2018 at 11:38:52AM -0500, Gustavo A. R. Silva wrote: >> In preparation to enabling -Wimplicit-fallthrough, mark switch cases >> where we are expecting to fall through. >> >> Addresses-Coverity-ID: 1462408 ("Missing

Re: [PATCH] iio: accel: cros_ec_accel_legacy: Mark expected switch fall-throughs

2018-08-15 Thread Marcus Folkesson
Hi, On Wed, Aug 15, 2018 at 11:38:52AM -0500, Gustavo A. R. Silva wrote: > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. > > Addresses-Coverity-ID: 1462408 ("Missing break in switch") > Signed-off-by: Gustavo A. R. Silva > --- > d

[PATCH] iio: accel: cros_ec_accel_legacy: Mark expected switch fall-throughs

2018-08-15 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 1462408 ("Missing break in switch") Signed-off-by: Gustavo A. R. Silva --- drivers/iio/accel/cros_ec_accel_legacy.c | 2 ++ 1 file changed, 2 insertions(+) diff --