[PATCH] media: cleanup fall-through comments
Greg KH
gregkh at linuxfoundation.org
Tue Aug 7 16:43:51 UTC 2018
On Tue, Aug 07, 2018 at 09:33:03AM -0700, Nick Desaulniers wrote:
> On Tue, Aug 7, 2018 at 5:07 AM Mauro Carvalho Chehab
> <mchehab+samsung at kernel.org> wrote:
> >
> > As Ian pointed out, adding a '-' to the fallthrough seems to meet
> > the regex requirements at level 3 of the warning, at least when
> > the comment fits into a single line.
> >
> > So, replace by a single line the comments that were broken into
> > multiple lines just to make gcc -Wimplicit-fallthrough=3 happy.
> >
> > Suggested-by: Ian Arkver <ian.arkver.dev at gmail.com>
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung at kernel.org>
> > ---
> > drivers/media/dvb-frontends/drx39xyj/drxj.c | 3 +--
> > drivers/media/dvb-frontends/drxd_hard.c | 6 ++----
> > drivers/media/dvb-frontends/drxk_hard.c | 18 ++++++------------
> > drivers/staging/media/imx/imx-media-csi.c | 3 +--
> > 4 files changed, 10 insertions(+), 20 deletions(-)
> >
> > diff --git a/drivers/media/dvb-frontends/drx39xyj/drxj.c b/drivers/media/dvb-frontends/drx39xyj/drxj.c
> > index 2ddb7d218ace..2948d12d7c14 100644
> > --- a/drivers/media/dvb-frontends/drx39xyj/drxj.c
> > +++ b/drivers/media/dvb-frontends/drx39xyj/drxj.c
> > @@ -2841,8 +2841,7 @@ ctrl_set_cfg_mpeg_output(struct drx_demod_instance *demod, struct drx_cfg_mpeg_o
> > /* coef = 188/204 */
> > max_bit_rate =
> > (ext_attr->curr_symbol_rate / 8) * nr_bits * 188;
> > - /* pass through as b/c Annex A/c need following settings */
> > - /* fall-through */
> > + /* fall-through - as b/c Annex A/C need following settings */
> > case DRX_STANDARD_ITU_B:
> > rc = drxj_dap_write_reg16(dev_addr, FEC_OC_FCT_USAGE__A, FEC_OC_FCT_USAGE__PRE, 0);
> > if (rc != 0) {
> > diff --git a/drivers/media/dvb-frontends/drxd_hard.c b/drivers/media/dvb-frontends/drxd_hard.c
> > index 11fc259e4383..684d428efb0d 100644
> > --- a/drivers/media/dvb-frontends/drxd_hard.c
> > +++ b/drivers/media/dvb-frontends/drxd_hard.c
> > @@ -1970,8 +1970,7 @@ static int DRX_Start(struct drxd_state *state, s32 off)
> > switch (p->transmission_mode) {
> > default: /* Not set, detect it automatically */
> > operationMode |= SC_RA_RAM_OP_AUTO_MODE__M;
> > - /* try first guess DRX_FFTMODE_8K */
> > - /* fall through */
> > + /* fall through - try first guess DRX_FFTMODE_8K */
> > case TRANSMISSION_MODE_8K:
> > transmissionParams |= SC_RA_RAM_OP_PARAM_MODE_8K;
> > if (state->type_A) {
> > @@ -2144,8 +2143,7 @@ static int DRX_Start(struct drxd_state *state, s32 off)
> > switch (p->modulation) {
> > default:
> > operationMode |= SC_RA_RAM_OP_AUTO_CONST__M;
> > - /* try first guess DRX_CONSTELLATION_QAM64 */
> > - /* fall through */
> > + /* fall through - try first guess DRX_CONSTELLATION_QAM64 */
> > case QAM_64:
> > transmissionParams |= SC_RA_RAM_OP_PARAM_CONST_QAM64;
> > if (state->type_A) {
> > diff --git a/drivers/media/dvb-frontends/drxk_hard.c b/drivers/media/dvb-frontends/drxk_hard.c
> > index ac10781d3550..f1886945a7bc 100644
> > --- a/drivers/media/dvb-frontends/drxk_hard.c
> > +++ b/drivers/media/dvb-frontends/drxk_hard.c
> > @@ -3270,13 +3270,11 @@ static int dvbt_sc_command(struct drxk_state *state,
> > case OFDM_SC_RA_RAM_CMD_SET_PREF_PARAM:
> > case OFDM_SC_RA_RAM_CMD_PROGRAM_PARAM:
> > status |= write16(state, OFDM_SC_RA_RAM_PARAM1__A, param1);
> > - /* All commands using 1 parameters */
> > - /* fall through */
> > + /* fall through - All commands using 1 parameters */
> > case OFDM_SC_RA_RAM_CMD_SET_ECHO_TIMING:
> > case OFDM_SC_RA_RAM_CMD_USER_IO:
> > status |= write16(state, OFDM_SC_RA_RAM_PARAM0__A, param0);
> > - /* All commands using 0 parameters */
> > - /* fall through */
> > + /* fall through - All commands using 0 parameters */
> > case OFDM_SC_RA_RAM_CMD_GET_OP_PARAM:
> > case OFDM_SC_RA_RAM_CMD_NULL:
> > /* Write command */
> > @@ -3784,8 +3782,7 @@ static int set_dvbt(struct drxk_state *state, u16 intermediate_freqk_hz,
> > case TRANSMISSION_MODE_AUTO:
> > default:
> > operation_mode |= OFDM_SC_RA_RAM_OP_AUTO_MODE__M;
> > - /* try first guess DRX_FFTMODE_8K */
> > - /* fall through */
> > + /* fall through - try first guess DRX_FFTMODE_8K */
> > case TRANSMISSION_MODE_8K:
> > transmission_params |= OFDM_SC_RA_RAM_OP_PARAM_MODE_8K;
> > break;
> > @@ -3799,8 +3796,7 @@ static int set_dvbt(struct drxk_state *state, u16 intermediate_freqk_hz,
> > default:
> > case GUARD_INTERVAL_AUTO:
> > operation_mode |= OFDM_SC_RA_RAM_OP_AUTO_GUARD__M;
> > - /* try first guess DRX_GUARD_1DIV4 */
> > - /* fall through */
> > + /* fall through - try first guess DRX_GUARD_1DIV4 */
> > case GUARD_INTERVAL_1_4:
> > transmission_params |= OFDM_SC_RA_RAM_OP_PARAM_GUARD_4;
> > break;
> > @@ -3841,8 +3837,7 @@ static int set_dvbt(struct drxk_state *state, u16 intermediate_freqk_hz,
> > case QAM_AUTO:
> > default:
> > operation_mode |= OFDM_SC_RA_RAM_OP_AUTO_CONST__M;
> > - /* try first guess DRX_CONSTELLATION_QAM64 */
> > - /* fall through */
> > + /* fall through - try first guess DRX_CONSTELLATION_QAM64 */
> > case QAM_64:
> > transmission_params |= OFDM_SC_RA_RAM_OP_PARAM_CONST_QAM64;
> > break;
> > @@ -3885,8 +3880,7 @@ static int set_dvbt(struct drxk_state *state, u16 intermediate_freqk_hz,
> > case FEC_AUTO:
> > default:
> > operation_mode |= OFDM_SC_RA_RAM_OP_AUTO_RATE__M;
> > - /* try first guess DRX_CODERATE_2DIV3 */
> > - /* fall through */
> > + /* fall through - try first guess DRX_CODERATE_2DIV3 */
> > case FEC_2_3:
> > transmission_params |= OFDM_SC_RA_RAM_OP_PARAM_RATE_2_3;
> > break;
> > diff --git a/drivers/staging/media/imx/imx-media-csi.c b/drivers/staging/media/imx/imx-media-csi.c
> > index b7ffd231c64b..cd2c291e1e94 100644
> > --- a/drivers/staging/media/imx/imx-media-csi.c
> > +++ b/drivers/staging/media/imx/imx-media-csi.c
> > @@ -460,8 +460,7 @@ static int csi_idmac_setup_channel(struct csi_priv *priv)
> > passthrough_cycles = incc->cycles;
> > break;
> > }
> > - /* for non-passthrough RGB565 (CSI-2 bus) */
> > - /* Falls through */
> > + /* fallthrough - non-passthrough RGB565 (CSI-2 bus) */
> > default:
> > burst_size = (image.pix.width & 0xf) ? 8 : 16;
> > passthrough_bits = 16;
> > --
> > 2.17.1
> >
>
> Can we use the compiler attribute:
>
> __attribute__((fallthrough))
If so, you have a lot of the kernel to fix up now, as this has been
implemented already all over the kernel with comments. The past few
months loads of patches have been accepted for this very issue.
I don't care either way, as long as we are consistent :)
thanks,
greg k-h
More information about the devel
mailing list