media: Use fallthrough pseudo-keyword
authorGustavo A. R. Silva <gustavoars@kernel.org>
Fri, 24 Jul 2020 22:10:14 +0000 (00:10 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Sat, 29 Aug 2020 06:35:27 +0000 (08:35 +0200)
Replace the existing /* fall through */ comments and its variants with
the new pseudo-keyword macro fallthrough[1]. Also, remove unnecessary
fall-through markings when it is the case.

[1] https://www.kernel.org/doc/html/v5.7/process/deprecated.html?highlight=fallthrough#implicit-switch-case-fall-through

Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
55 files changed:
drivers/media/cec/core/cec-adap.c
drivers/media/cec/core/cec-pin.c
drivers/media/cec/usb/pulse8/pulse8-cec.c
drivers/media/common/videobuf2/videobuf2-v4l2.c
drivers/media/dvb-frontends/af9013.c
drivers/media/dvb-frontends/lg2160.c
drivers/media/dvb-frontends/m88ds3103.c
drivers/media/i2c/adv7180.c
drivers/media/i2c/adv7511-v4l2.c
drivers/media/i2c/msp3400-kthreads.c
drivers/media/i2c/ov5640.c
drivers/media/i2c/ov6650.c
drivers/media/i2c/ov9640.c
drivers/media/i2c/s5c73m3/s5c73m3-ctrls.c
drivers/media/i2c/smiapp/smiapp-core.c
drivers/media/i2c/tda1997x.c
drivers/media/i2c/tvp5150.c
drivers/media/pci/bt8xx/bttv-driver.c
drivers/media/pci/cx88/cx88-cards.c
drivers/media/pci/cx88/cx88-video.c
drivers/media/pci/saa7134/saa7134-cards.c
drivers/media/pci/solo6x10/solo6x10-core.c
drivers/media/pci/solo6x10/solo6x10-i2c.c
drivers/media/platform/coda/coda-bit.c
drivers/media/platform/coda/coda-common.c
drivers/media/platform/exynos4-is/fimc-capture.c
drivers/media/platform/exynos4-is/fimc-reg.c
drivers/media/platform/exynos4-is/media-dev.c
drivers/media/platform/marvell-ccic/mcam-core.c
drivers/media/platform/omap3isp/ispvideo.c
drivers/media/platform/qcom/venus/vdec.c
drivers/media/platform/renesas-ceu.c
drivers/media/radio/si4713/si4713.c
drivers/media/rc/iguanair.c
drivers/media/test-drivers/vicodec/vicodec-core.c
drivers/media/test-drivers/vivid/vivid-vbi-gen.c
drivers/media/tuners/fc0011.c
drivers/media/tuners/tda18271-fe.c
drivers/media/usb/dvb-usb-v2/af9015.c
drivers/media/usb/dvb-usb-v2/gl861.c
drivers/media/usb/dvb-usb-v2/lmedm04.c
drivers/media/usb/dvb-usb-v2/mxl111sf-gpio.c
drivers/media/usb/em28xx/em28xx-audio.c
drivers/media/usb/go7007/go7007-driver.c
drivers/media/usb/gspca/mr97310a.c
drivers/media/usb/gspca/nw80x.c
drivers/media/usb/gspca/ov519.c
drivers/media/usb/gspca/sn9c20x.c
drivers/media/usb/gspca/sunplus.c
drivers/media/usb/gspca/xirlink_cit.c
drivers/media/usb/gspca/zc3xx.c
drivers/media/usb/pwc/pwc-v4l.c
drivers/media/usb/siano/smsusb.c
drivers/media/usb/tm6000/tm6000-alsa.c
drivers/media/usb/uvc/uvc_video.c

index 4efe801..80fd516 100644 (file)
@@ -1928,7 +1928,7 @@ static int cec_receive_notify(struct cec_adapter *adap, struct cec_msg *msg,
                 */
                if (!adap->passthrough && from_unregistered)
                        return 0;
-               /* Fall through */
+               fallthrough;
        case CEC_MSG_GIVE_DEVICE_VENDOR_ID:
        case CEC_MSG_GIVE_FEATURES:
        case CEC_MSG_GIVE_PHYSICAL_ADDR:
index 660fe11..f006bd8 100644 (file)
@@ -417,7 +417,7 @@ static void cec_pin_tx_states(struct cec_pin *pin, ktime_t ts)
                        wake_up_interruptible(&pin->kthread_waitq);
                        break;
                }
-               /* fall through */
+               fallthrough;
        case CEC_ST_TX_DATA_BIT_0_HIGH:
        case CEC_ST_TX_DATA_BIT_0_HIGH_SHORT:
        case CEC_ST_TX_DATA_BIT_0_HIGH_LONG:
@@ -445,7 +445,7 @@ static void cec_pin_tx_states(struct cec_pin *pin, ktime_t ts)
                        wake_up_interruptible(&pin->kthread_waitq);
                        break;
                }
-               /* fall through */
+               fallthrough;
        case CEC_ST_TX_DATA_BIT_HIGH_CUSTOM:
                if (tx_last_bit(pin)) {
                        /* Error Injection: just stop sending after this bit */
@@ -459,7 +459,7 @@ static void cec_pin_tx_states(struct cec_pin *pin, ktime_t ts)
                        break;
                }
                pin->tx_bit++;
-               /* fall through */
+               fallthrough;
        case CEC_ST_TX_START_BIT_HIGH:
        case CEC_ST_TX_START_BIT_HIGH_SHORT:
        case CEC_ST_TX_START_BIT_HIGH_LONG:
index beae6aa..e4d8446 100644 (file)
@@ -389,7 +389,7 @@ static irqreturn_t pulse8_interrupt(struct serio *serio, unsigned char data,
                                pulse8->new_rx_msg[0] = pulse8->buf[1];
                                break;
                        }
-                       /* fall through */
+                       fallthrough;
                case MSGCODE_FRAME_DATA:
                        if (pulse8->new_rx_msg_len < CEC_MAX_MSG_SIZE)
                                pulse8->new_rx_msg[pulse8->new_rx_msg_len++] =
index 2068305..f4197c2 100644 (file)
@@ -601,7 +601,7 @@ static void __fill_v4l2_buffer(struct vb2_buffer *vb, void *pb)
                break;
        case VB2_BUF_STATE_ERROR:
                b->flags |= V4L2_BUF_FLAG_ERROR;
-               /* fall through */
+               fallthrough;
        case VB2_BUF_STATE_DONE:
                b->flags |= V4L2_BUF_FLAG_DONE;
                break;
index 7281899..7d7c341 100644 (file)
@@ -597,7 +597,7 @@ static int af9013_read_status(struct dvb_frontend *fe, enum fe_status *status)
                        state->strength_en = 2;
                        break;
                }
-               /* Fall through */
+               fallthrough;
        case 1:
                if (time_is_after_jiffies(state->strength_jiffies + msecs_to_jiffies(2000)))
                        break;
index 10c152f..f343066 100644 (file)
@@ -1408,7 +1408,7 @@ struct dvb_frontend *lg2160_attach(const struct lg2160_config *config,
        switch (config->lg_chip) {
        default:
                lg_warn("invalid chip requested, defaulting to LG2160");
-               /* fall-thru */
+               fallthrough;
        case LG2160:
                memcpy(&state->frontend.ops, &lg2160_ops,
                       sizeof(struct dvb_frontend_ops));
index f204e71..ad6d9d5 100644 (file)
@@ -906,7 +906,7 @@ static int m88ds3103_set_frontend(struct dvb_frontend *fe)
                        if (ret)
                                goto err;
                }
-               /* fall through */
+               fallthrough;
        default:
                u16tmp = DIV_ROUND_UP(target_mclk, dev->cfg->ts_clk);
                u8tmp1 = u16tmp / 2 - 1;
index e8744ef..4498d14 100644 (file)
@@ -726,7 +726,7 @@ static int adv7180_set_pad_format(struct v4l2_subdev *sd,
        case V4L2_FIELD_NONE:
                if (state->chip_info->flags & ADV7180_FLAG_I2P)
                        break;
-               /* fall through */
+               fallthrough;
        default:
                format->format.field = V4L2_FIELD_ALTERNATE;
                break;
index 62763ec..a3161d7 100644 (file)
@@ -470,7 +470,7 @@ static int adv7511_g_register(struct v4l2_subdev *sd, struct v4l2_dbg_register *
                        reg->val = adv7511_cec_read(sd, reg->reg & 0xff);
                        break;
                }
-               /* fall through */
+               fallthrough;
        default:
                v4l2_info(sd, "Register %03llx not supported\n", reg->reg);
                adv7511_inv_register(sd);
@@ -492,7 +492,7 @@ static int adv7511_s_register(struct v4l2_subdev *sd, const struct v4l2_dbg_regi
                        adv7511_cec_write(sd, reg->reg & 0xff, reg->val & 0xff);
                        break;
                }
-               /* fall through */
+               fallthrough;
        default:
                v4l2_info(sd, "Register %03llx not supported\n", reg->reg);
                adv7511_inv_register(sd);
index d3b0d1c..52e506f 100644 (file)
@@ -646,7 +646,7 @@ restart:
                        break;
                case 0: /* 4.5 */
                        state->detected_std = V4L2_STD_MN;
-                       /* fall-through */
+                       fallthrough;
                default:
 no_second:
                        state->second = msp3400c_carrier_detect_main[max1].cdo;
index 2fe4a7a..ccdabc7 100644 (file)
@@ -3010,7 +3010,7 @@ static int ov5640_probe(struct i2c_client *client)
                switch (rotation) {
                case 180:
                        sensor->upside_down = true;
-                       /* fall through */
+                       fallthrough;
                case 0:
                        break;
                default:
index 48493af..d73f9f5 100644 (file)
@@ -685,7 +685,7 @@ static int ov6650_set_fmt(struct v4l2_subdev *sd,
        switch (mf->code) {
        case MEDIA_BUS_FMT_Y10_1X10:
                mf->code = MEDIA_BUS_FMT_Y8_1X8;
-               /* fall through */
+               fallthrough;
        case MEDIA_BUS_FMT_Y8_1X8:
        case MEDIA_BUS_FMT_YVYU8_2X8:
        case MEDIA_BUS_FMT_YUYV8_2X8:
@@ -694,7 +694,7 @@ static int ov6650_set_fmt(struct v4l2_subdev *sd,
                break;
        default:
                mf->code = MEDIA_BUS_FMT_SBGGR8_1X8;
-               /* fall through */
+               fallthrough;
        case MEDIA_BUS_FMT_SBGGR8_1X8:
                break;
        }
index fbd5d7b..e2a2524 100644 (file)
@@ -538,7 +538,7 @@ static int ov9640_set_fmt(struct v4l2_subdev *sd,
                break;
        default:
                mf->code = MEDIA_BUS_FMT_UYVY8_2X8;
-               /* fall through */
+               fallthrough;
        case MEDIA_BUS_FMT_UYVY8_2X8:
                mf->colorspace = V4L2_COLORSPACE_JPEG;
                break;
index 71cf68a..141ad0b 100644 (file)
@@ -46,7 +46,7 @@ static int s5c73m3_get_af_status(struct s5c73m3 *state, struct v4l2_ctrl *ctrl)
                break;
        default:
                v4l2_info(&state->sensor_sd, "Unknown AF status %#x\n", reg);
-               /* Fall through */
+               fallthrough;
        case REG_CAF_STATUS_UNFOCUSED:
        case REG_AF_STATUS_UNFOCUSED:
        case REG_AF_STATUS_INVALID:
index 8a9c7de..6fc0680 100644 (file)
@@ -1721,7 +1721,7 @@ static void smiapp_propagate(struct v4l2_subdev *subdev,
                                sensor->binning_vertical = 1;
                        }
                }
-               /* Fall through */
+               fallthrough;
        case V4L2_SEL_TGT_COMPOSE:
                *crops[SMIAPP_PAD_SRC] = *comp;
                break;
@@ -2120,7 +2120,7 @@ static int __smiapp_sel_supported(struct v4l2_subdev *subdev,
                    && SMIA_LIM(sensor, SCALING_CAPABILITY)
                    != SMIAPP_SCALING_CAPABILITY_NONE)
                        return 0;
-               /* Fall through */
+               fallthrough;
        default:
                return -EINVAL;
        }
@@ -2795,7 +2795,7 @@ static struct smiapp_hwconfig *smiapp_get_hwconfig(struct device *dev)
                case 180:
                        hwcfg->module_board_orient =
                                SMIAPP_MODULE_BOARD_ORIENT_180;
-                       /* Fall through */
+                       fallthrough;
                case 0:
                        break;
                default:
index 483cc19..a09bf0a 100644 (file)
@@ -2588,7 +2588,7 @@ static int tda1997x_probe(struct i2c_client *client,
                        case 36:
                                mbus_codes[i++] = MEDIA_BUS_FMT_RGB121212_1X36;
                                mbus_codes[i++] = MEDIA_BUS_FMT_YUV12_1X36;
-                               /* fall-through */
+                               fallthrough;
                        case 24:
                                mbus_codes[i++] = MEDIA_BUS_FMT_UYVY12_1X24;
                                break;
@@ -2617,10 +2617,10 @@ static int tda1997x_probe(struct i2c_client *client,
                                mbus_codes[i++] = MEDIA_BUS_FMT_RGB888_1X24;
                                mbus_codes[i++] = MEDIA_BUS_FMT_YUV8_1X24;
                                mbus_codes[i++] = MEDIA_BUS_FMT_UYVY12_1X24;
-                               /* fall through */
+                               fallthrough;
                        case 20:
                                mbus_codes[i++] = MEDIA_BUS_FMT_UYVY10_1X20;
-                               /* fall through */
+                               fallthrough;
                        case 16:
                                mbus_codes[i++] = MEDIA_BUS_FMT_UYVY8_1X16;
                                break;
@@ -2633,10 +2633,10 @@ static int tda1997x_probe(struct i2c_client *client,
                        case 16:
                        case 12:
                                mbus_codes[i++] = MEDIA_BUS_FMT_UYVY12_2X12;
-                               /* fall through */
+                               fallthrough;
                        case 10:
                                mbus_codes[i++] = MEDIA_BUS_FMT_UYVY10_2X10;
-                               /* fall through */
+                               fallthrough;
                        case 8:
                                mbus_codes[i++] = MEDIA_BUS_FMT_UYVY8_2X8;
                                break;
index 1c20509..7d94012 100644 (file)
@@ -293,7 +293,7 @@ static void tvp5150_selmux(struct v4l2_subdev *sd)
        switch (decoder->input) {
        case TVP5150_COMPOSITE1:
                input |= 2;
-               /* fall through */
+               fallthrough;
        case TVP5150_COMPOSITE0:
                break;
        case TVP5150_SVIDEO:
index 9144f79..8c61d29 100644 (file)
@@ -2332,7 +2332,7 @@ static int bttv_try_fmt_vid_cap(struct file *file, void *priv,
                        field = V4L2_FIELD_SEQ_TB;
                        break;
                }
-               /* fall through */
+               fallthrough;
        default: /* FIELD_ANY case */
                height2 = btv->crop[!!fh->do_crop].rect.height >> 1;
                field = (f->fmt.pix.height > height2)
index 9fa3886..8e224fc 100644 (file)
@@ -3499,7 +3499,7 @@ static void cx88_card_setup(struct cx88_core *core)
                cx_clear(MO_GP0_IO, 0x00000040);
                msleep(1000);
                cx_set(MO_GP0_IO, 0x00004040);
-               /* FALLTHROUGH */
+               fallthrough;
        case CX88_BOARD_DVICO_FUSIONHDTV_DVB_T1:
        case CX88_BOARD_DVICO_FUSIONHDTV_DVB_T_PLUS:
        case CX88_BOARD_DVICO_FUSIONHDTV_DVB_T_HYBRID:
index e7fd751..8cffdac 100644 (file)
@@ -1385,7 +1385,7 @@ static int cx8800_initdev(struct pci_dev *pci_dev,
                request_module("rtc-isl1208");
                core->i2c_rtc = i2c_new_client_device(&core->i2c_adap, &rtc_info);
        }
-               /* fall-through */
+               fallthrough;
        case CX88_BOARD_DVICO_FUSIONHDTV_5_PCI_NANO:
                request_module("ir-kbd-i2c");
        }
index c1937c3..ce449c9 100644 (file)
@@ -7812,7 +7812,7 @@ int saa7134_board_init2(struct saa7134_dev *dev)
                                dev->name, saa7134_boards[dev->board].name);
                        break;
                }
-               /* fall-through */
+               fallthrough;
        case SAA7134_BOARD_VIDEOMATE_DVBT_300:
        case SAA7134_BOARD_ASUS_EUROPA2_HYBRID:
        case SAA7134_BOARD_ASUS_EUROPA_HYBRID:
@@ -7870,7 +7870,7 @@ int saa7134_board_init2(struct saa7134_dev *dev)
                break;
        case SAA7134_BOARD_HAUPPAUGE_HVR1110:
                hauppauge_eeprom(dev, dev->eedata+0x80);
-               /* fall-through */
+               fallthrough;
        case SAA7134_BOARD_PINNACLE_PCTV_310i:
        case SAA7134_BOARD_KWORLD_DVBT_210:
        case SAA7134_BOARD_TEVION_DVBT_220RF:
index c6e0090..d497afc 100644 (file)
@@ -503,7 +503,7 @@ static int solo_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id)
        default:
                dev_warn(&pdev->dev, "Invalid chip_id 0x%02x, assuming 4 ch\n",
                         chip_id);
-               /* fall through */
+               fallthrough;
        case 5:
                solo_dev->nr_chans = 4;
                solo_dev->nr_ext = 1;
index f86f12f..7db785e 100644 (file)
@@ -183,7 +183,7 @@ int solo_i2c_isr(struct solo_dev *solo_dev)
                }
 
                solo_dev->i2c_state = IIC_STATE_WRITE;
-               /* fall through */
+               fallthrough;
        case IIC_STATE_WRITE:
                ret = solo_i2c_handle_write(solo_dev);
                break;
index b021604..bf75927 100644 (file)
@@ -1101,7 +1101,7 @@ static int coda_start_encoding(struct coda_ctx *ctx)
                break;
        case CODA_960:
                coda_write(dev, 0, CODA9_GDI_WPROT_RGN_EN);
-               /* fallthrough */
+               fallthrough;
        case CODA_HX4:
        case CODA_7541:
                coda_write(dev, CODA7_STREAM_BUF_DYNALLOC_EN |
@@ -1141,7 +1141,7 @@ static int coda_start_encoding(struct coda_ctx *ctx)
                                 CODA7_PICHEIGHT_MASK) << CODA_PICHEIGHT_OFFSET;
                        break;
                }
-               /* fallthrough */
+               fallthrough;
        case CODA_960:
                value = (q_data_src->rect.width & CODA7_PICWIDTH_MASK)
                        << CODA7_PICWIDTH_OFFSET;
index 3ab3d97..823a4d7 100644 (file)
@@ -808,7 +808,7 @@ static int coda_s_fmt(struct coda_ctx *ctx, struct v4l2_format *f,
                        ctx->tiled_map_type = GDI_TILED_FRAME_MB_RASTER_MAP;
                        break;
                }
-               /* else fall through */
+               fallthrough;
        case V4L2_PIX_FMT_YUV420:
        case V4L2_PIX_FMT_YVU420:
        case V4L2_PIX_FMT_YUV422P:
@@ -1015,7 +1015,7 @@ static int coda_g_selection(struct file *file, void *fh,
        case V4L2_SEL_TGT_CROP_DEFAULT:
        case V4L2_SEL_TGT_CROP_BOUNDS:
                rsel = &r;
-               /* fallthrough */
+               fallthrough;
        case V4L2_SEL_TGT_CROP:
                if (s->type != V4L2_BUF_TYPE_VIDEO_OUTPUT ||
                    ctx->inst_type == CODA_INST_DECODER)
@@ -1024,7 +1024,7 @@ static int coda_g_selection(struct file *file, void *fh,
        case V4L2_SEL_TGT_COMPOSE_BOUNDS:
        case V4L2_SEL_TGT_COMPOSE_PADDED:
                rsel = &r;
-               /* fallthrough */
+               fallthrough;
        case V4L2_SEL_TGT_COMPOSE:
        case V4L2_SEL_TGT_COMPOSE_DEFAULT:
                if (s->type != V4L2_BUF_TYPE_VIDEO_CAPTURE ||
@@ -1074,7 +1074,7 @@ static int coda_s_selection(struct file *file, void *fh,
 
                        return 0;
                }
-               /* else fall through */
+               fallthrough;
        case V4L2_SEL_TGT_NATIVE_SIZE:
        case V4L2_SEL_TGT_COMPOSE:
                return coda_g_selection(file, fh, s);
@@ -2628,7 +2628,7 @@ static int coda_open(struct file *file)
                 */
                if (enable_bwb || ctx->inst_type == CODA_INST_ENCODER)
                        ctx->frame_mem_ctrl = CODA9_FRAME_ENABLE_BWB;
-               /* fallthrough */
+               fallthrough;
        case CODA_HX4:
        case CODA_7541:
                ctx->reg_idx = 0;
index e7a4b06..4fbf550 100644 (file)
@@ -1279,7 +1279,7 @@ static int fimc_cap_g_selection(struct file *file, void *fh,
        case V4L2_SEL_TGT_COMPOSE_DEFAULT:
        case V4L2_SEL_TGT_COMPOSE_BOUNDS:
                f = &ctx->d_frame;
-               /* fall through */
+               fallthrough;
        case V4L2_SEL_TGT_CROP_BOUNDS:
        case V4L2_SEL_TGT_CROP_DEFAULT:
                s->r.left = 0;
@@ -1290,7 +1290,7 @@ static int fimc_cap_g_selection(struct file *file, void *fh,
 
        case V4L2_SEL_TGT_COMPOSE:
                f = &ctx->d_frame;
-               /* fall through */
+               fallthrough;
        case V4L2_SEL_TGT_CROP:
                s->r.left = f->offs_h;
                s->r.top = f->offs_v;
@@ -1601,7 +1601,7 @@ static int fimc_subdev_get_selection(struct v4l2_subdev *sd,
        switch (sel->target) {
        case V4L2_SEL_TGT_COMPOSE_BOUNDS:
                f = &ctx->d_frame;
-               /* fall through */
+               fallthrough;
        case V4L2_SEL_TGT_CROP_BOUNDS:
                r->width = f->o_width;
                r->height = f->o_height;
index 5ce2bde..c9e9ed7 100644 (file)
@@ -710,7 +710,7 @@ int fimc_hw_set_camera_type(struct fimc_dev *fimc,
                break;
        case FIMC_BUS_TYPE_LCD_WRITEBACK_A:
                cfg |= FIMC_REG_CIGCTRL_CAMIF_SELWB;
-               /* fall through */
+               fallthrough;
        case FIMC_BUS_TYPE_ISP_WRITEBACK:
                if (fimc->variant->has_isp_wb)
                        cfg |= FIMC_REG_CIGCTRL_CAMIF_SELWB;
index 6df2796..ae74d8f 100644 (file)
@@ -93,7 +93,7 @@ static void fimc_pipeline_prepare(struct fimc_pipeline *p,
                switch (sd->grp_id) {
                case GRP_ID_SENSOR:
                        sensor = sd;
-                       /* fall through */
+                       fallthrough;
                case GRP_ID_FIMC_IS_SENSOR:
                        p->subdevs[IDX_SENSOR] = sd;
                        break;
index f919cb4..c012fd2 100644 (file)
@@ -389,7 +389,7 @@ static int mcam_alloc_dma_bufs(struct mcam_camera *cam, int loadtime)
                dma_free_coherent(cam->dev, cam->dma_buf_size,
                                cam->dma_bufs[0], cam->dma_handles[0]);
                cam->nbufs = 0;
-               /* fall-through */
+               fallthrough;
        case 0:
                cam_err(cam, "Insufficient DMA buffers, cannot operate\n");
                return -ENOMEM;
index 1ac9aef..8811d6d 100644 (file)
@@ -703,7 +703,7 @@ isp_video_set_format(struct file *file, void *fh, struct v4l2_format *format)
                 * requested.
                 */
                format->fmt.pix.field = V4L2_FIELD_INTERLACED_TB;
-               /* Fall-through */
+               fallthrough;
        case V4L2_FIELD_INTERLACED_TB:
        case V4L2_FIELD_INTERLACED_BT:
                /* Interlaced orders are only supported at the CCDC output. */
index fbefc16..a88367a 100644 (file)
@@ -1072,7 +1072,7 @@ static int vdec_stop_capture(struct venus_inst *inst)
        switch (inst->codec_state) {
        case VENUS_DEC_STATE_DECODING:
                ret = hfi_session_flush(inst, HFI_FLUSH_ALL, true);
-               /* fallthrough */
+               fallthrough;
        case VENUS_DEC_STATE_DRAIN:
                vdec_cancel_dst_buffers(inst);
                inst->codec_state = VENUS_DEC_STATE_STOPPED;
index f7d71a6..4a633ad 100644 (file)
@@ -405,7 +405,7 @@ static int ceu_hw_config(struct ceu_device *ceudev)
        /* Non-swapped planar image capture mode. */
        case V4L2_PIX_FMT_NV16:
                cdocr   |= CEU_CDOCR_NO_DOWSAMPLE;
-               /* fall-through */
+               fallthrough;
        case V4L2_PIX_FMT_NV12:
                if (mbus_fmt->swapped)
                        camcr = mbus_fmt->fmt_order_swap;
@@ -419,7 +419,7 @@ static int ceu_hw_config(struct ceu_device *ceudev)
        /* Swapped planar image capture mode. */
        case V4L2_PIX_FMT_NV61:
                cdocr   |= CEU_CDOCR_NO_DOWSAMPLE;
-               /* fall-through */
+               fallthrough;
        case V4L2_PIX_FMT_NV21:
                if (mbus_fmt->swapped)
                        camcr = mbus_fmt->fmt_order;
index 7f3aee4..6afa7c3 100644 (file)
@@ -1157,7 +1157,7 @@ static int si4713_s_ctrl(struct v4l2_ctrl *ctrl)
                         * V4L2_CID_TUNE_POWER_LEVEL. */
                        if (force)
                                break;
-                       /* fall through */
+                       fallthrough;
                case V4L2_CID_TUNE_POWER_LEVEL:
                        ret = si4713_tx_tune_power(sdev,
                                sdev->tune_pwr_level->val, sdev->tune_ant_cap->val);
index 566c281..b5b993b 100644 (file)
@@ -101,7 +101,7 @@ static void process_ir_data(struct iguanair *ir, unsigned len)
                        break;
                case CMD_TX_OVERFLOW:
                        ir->tx_overflow = true;
-                       /* fall through */
+                       fallthrough;
                case CMD_RECEIVER_OFF:
                case CMD_RECEIVER_ON:
                case CMD_SEND:
index e4a8d3a..3281caf 100644 (file)
@@ -1310,7 +1310,7 @@ static int vicodec_subscribe_event(struct v4l2_fh *fh,
        case V4L2_EVENT_SOURCE_CHANGE:
                if (ctx->is_enc)
                        return -EINVAL;
-               /* fall through */
+               fallthrough;
        case V4L2_EVENT_EOS:
                if (ctx->is_stateless)
                        return -EINVAL;
index acc9844..a141369 100644 (file)
@@ -298,7 +298,7 @@ void vivid_vbi_gen_sliced(struct vivid_vbi_gen_data *vbi,
        switch (frame) {
        case 0:
                vivid_vbi_gen_set_time_of_day(vbi->time_of_day_packet);
-               /* fall through */
+               fallthrough;
        case 1 ... 7:
                data1->data[0] = vbi->time_of_day_packet[frame * 2];
                data1->data[1] = vbi->time_of_day_packet[frame * 2 + 1];
index b7b5b33..eaa3bbc 100644 (file)
@@ -250,7 +250,7 @@ static int fc0011_set_params(struct dvb_frontend *fe)
                dev_warn(&priv->i2c->dev, "Unsupported bandwidth %u kHz. Using 6000 kHz.\n",
                         bandwidth);
                bandwidth = 6000;
-               /* fallthrough */
+               fallthrough;
        case 6000:
                regs[FC11_REG_VCOSEL] |= FC11_VCOSEL_BW6M;
                break;
index 471aaf7..f0371d0 100644 (file)
@@ -948,7 +948,7 @@ static int tda18271_set_params(struct dvb_frontend *fe)
                break;
        case SYS_DVBC_ANNEX_B:
                bw = 6000000;
-               /* fall through */
+               fallthrough;
        case SYS_DVBC_ANNEX_A:
        case SYS_DVBC_ANNEX_C:
                if (bw <= 6000000) {
index c427b90..c70b3ce 100644 (file)
@@ -43,7 +43,7 @@ static int af9015_ctrl_msg(struct dvb_usb_device *d, struct req_t *req)
        case READ_I2C:
                write = 0;
                state->buf[2] |= 0x01; /* set I2C direction */
-               /* fall through */
+               fallthrough;
        case WRITE_I2C:
                state->buf[0] = READ_WRITE_I2C;
                break;
index b7ca236..0c43425 100644 (file)
@@ -41,7 +41,7 @@ static int gl861_ctrl_msg(struct dvb_usb_device *d, u8 request, u16 value,
        switch (request) {
        case CMD_WRITE:
                memcpy(ctx->buf, data, size);
-               /* Fall through */
+               fallthrough;
        case CMD_WRITE_SHORT:
                pipe = usb_sndctrlpipe(d->udev, 0);
                requesttype = USB_TYPE_VENDOR | USB_DIR_OUT;
index 8a3c0ee..5a7a952 100644 (file)
@@ -687,7 +687,7 @@ static const char *lme_firmware_switch(struct dvb_usb_device *d, int cold)
                                cold = 0;
                                break;
                        }
-                       /* fall through */
+                       fallthrough;
                case TUNER_LG:
                        fw_lme = fw_lg;
                        ret = request_firmware(&fw, fw_lme, &udev->dev);
@@ -710,7 +710,7 @@ static const char *lme_firmware_switch(struct dvb_usb_device *d, int cold)
                                cold = 0;
                                break;
                        }
-                       /* fall through */
+                       fallthrough;
                case TUNER_LG:
                        fw_lme = fw_c_lg;
                        ret = request_firmware(&fw, fw_lme, &udev->dev);
@@ -718,7 +718,7 @@ static const char *lme_firmware_switch(struct dvb_usb_device *d, int cold)
                                st->dvb_usb_lme2510_firmware = TUNER_LG;
                                break;
                        }
-                       /* fall through */
+                       fallthrough;
                case TUNER_S0194:
                        fw_lme = fw_c_s0194;
                        ret = request_firmware(&fw, fw_lme, &udev->dev);
@@ -1018,7 +1018,7 @@ static int dm04_lme2510_frontend_attach(struct dvb_usb_adapter *adap)
                        }
                        break;
                }
-               /* fall through */
+               fallthrough;
        case 0x22f0:
                st->i2c_gate = 5;
                adap->fe[0] = dvb_attach(m88rs2000_attach,
index 0b7dda9..ef489c5 100644 (file)
@@ -632,7 +632,7 @@ int mxl111sf_set_gpio(struct mxl111sf_state *state, int gpio, int val)
        default:
                mxl_printk(KERN_ERR,
                           "gpio_port_expander undefined, assuming PCA9534");
-               /* fall-thru */
+               fallthrough;
        case mxl111sf_PCA9534:
                return pca9534_set_gpio(state, gpio, val);
        case mxl111sf_gpio_hw:
@@ -693,7 +693,7 @@ int mxl111sf_init_port_expander(struct mxl111sf_state *state)
        default:
                mxl_printk(KERN_ERR,
                           "gpio_port_expander undefined, assuming PCA9534");
-               /* fall-thru */
+               fallthrough;
        case mxl111sf_PCA9534:
                return pca9534_init_port_expander(state);
        case mxl111sf_gpio_hw:
index 6833b5b..dc968fd 100644 (file)
@@ -362,13 +362,13 @@ static int snd_em28xx_capture_trigger(struct snd_pcm_substream *substream,
                return -ENODEV;
 
        switch (cmd) {
-       case SNDRV_PCM_TRIGGER_PAUSE_RELEASE: /* fall through */
-       case SNDRV_PCM_TRIGGER_RESUME: /* fall through */
+       case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
+       case SNDRV_PCM_TRIGGER_RESUME:
        case SNDRV_PCM_TRIGGER_START:
                atomic_set(&dev->adev.stream_started, 1);
                break;
-       case SNDRV_PCM_TRIGGER_PAUSE_PUSH: /* fall through */
-       case SNDRV_PCM_TRIGGER_SUSPEND: /* fall through */
+       case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
+       case SNDRV_PCM_TRIGGER_SUSPEND:
        case SNDRV_PCM_TRIGGER_STOP:
                atomic_set(&dev->adev.stream_started, 0);
                break;
index 153a0c3..f1767be 100644 (file)
@@ -643,7 +643,7 @@ void go7007_parse_video_stream(struct go7007 *go, u8 *buf, int length)
                        case 0xD8:
                                if (go->format == V4L2_PIX_FMT_MJPEG)
                                        vb = frame_boundary(go, vb);
-                               /* fall through */
+                               fallthrough;
                        default:
                                store_byte(vb, 0xFF);
                                store_byte(vb, buf[i]);
index 464aa61..3553788 100644 (file)
@@ -510,7 +510,7 @@ static int start_cif_cam(struct gspca_dev *gspca_dev)
        switch (gspca_dev->pixfmt.width) {
        case 160:
                data[9] |= 0x04;  /* reg 8, 2:1 scale down from 320 */
-               /* fall through */
+               fallthrough;
        case 320:
        default:
                data[3] = 0x28;                    /* reg 2, H size/8 */
@@ -520,7 +520,7 @@ static int start_cif_cam(struct gspca_dev *gspca_dev)
                break;
        case 176:
                data[9] |= 0x04;  /* reg 8, 2:1 scale down from 352 */
-               /* fall through */
+               fallthrough;
        case 352:
                data[3] = 0x2c;                    /* reg 2, H size/8 */
                data[4] = 0x48;                    /* reg 3, V size/4 */
@@ -607,10 +607,10 @@ static int start_vga_cam(struct gspca_dev *gspca_dev)
        switch (gspca_dev->pixfmt.width) {
        case 160:
                data[9] |= 0x0c;  /* reg 8, 4:1 scale down */
-               /* fall through */
+               fallthrough;
        case 320:
                data[9] |= 0x04;  /* reg 8, 2:1 scale down */
-               /* fall through */
+               fallthrough;
        case 640:
        default:
                data[3] = 0x50;  /* reg 2, H size/8 */
@@ -627,7 +627,7 @@ static int start_vga_cam(struct gspca_dev *gspca_dev)
 
        case 176:
                data[9] |= 0x04;  /* reg 8, 2:1 scale down */
-               /* fall through */
+               fallthrough;
        case 352:
                data[3] = 0x2c;  /* reg 2, H size */
                data[4] = 0x48;  /* reg 3, V size */
index 880f569..0f5f246 100644 (file)
@@ -2019,7 +2019,7 @@ static int sd_init_controls(struct gspca_dev *gspca_dev)
                        V4L2_CID_AUTOGAIN, 0, 1, 1, 1);
                gspca_dev->gain = v4l2_ctrl_new_std(hdl, &sd_ctrl_ops,
                        V4L2_CID_GAIN, 0, 253, 1, 128);
-               /* fall through */
+               fallthrough;
        case Cvideopro:
        case DvcV6:
        case Kritter:
index 0afe70a..cd6776c 100644 (file)
@@ -2004,7 +2004,7 @@ static void reg_w(struct sd *sd, u16 index, u16 value)
                break;
        case BRIDGE_OVFX2:
                req = 0x0a;
-               /* fall through */
+               fallthrough;
        case BRIDGE_W9968CF:
                gspca_dbg(gspca_dev, D_USBO, "SET %02x %04x %04x\n",
                          req, value, index);
@@ -3528,7 +3528,7 @@ static void ov511_mode_init_regs(struct sd *sd)
        case SEN_OV76BE:
                if (sd->gspca_dev.pixfmt.width == 320)
                        interlaced = 1;
-               /* Fall through */
+               fallthrough;
        case SEN_OV6630:
        case SEN_OV7610:
        case SEN_OV7670:
@@ -3541,7 +3541,7 @@ static void ov511_mode_init_regs(struct sd *sd)
                                break;
                        }
                        /* For 640x480 case */
-                       /* fall through */
+                       fallthrough;
                default:
 /*             case 20: */
 /*             case 15: */
index 2a6d0a1..bfd194c 100644 (file)
@@ -1637,7 +1637,7 @@ static int sd_config(struct gspca_dev *gspca_dev,
                break;
        case SENSOR_HV7131R:
                sd->i2c_intf = 0x81;                    /* i2c 400 Kb/s */
-               /* fall through */
+               fallthrough;
        default:
                cam->cam_mode = vga_mode;
                cam->nmodes = ARRAY_SIZE(vga_mode);
index f4a4222..ace3da4 100644 (file)
@@ -551,7 +551,7 @@ static void init_ctl_reg(struct gspca_dev *gspca_dev)
        case BRIDGE_SPCA504:
        case BRIDGE_SPCA504C:
                pollreg = 0;
-               /* fall through */
+               fallthrough;
        default:
 /*     case BRIDGE_SPCA533: */
 /*     case BRIDGE_SPCA504B: */
@@ -634,7 +634,7 @@ static int sd_init(struct gspca_dev *gspca_dev)
                reg_w_riv(gspca_dev, 0x00, 0x2000, 0x00);
                reg_w_riv(gspca_dev, 0x00, 0x2301, 0x13);
                reg_w_riv(gspca_dev, 0x00, 0x2306, 0x00);
-               /* fall through */
+               fallthrough;
        case BRIDGE_SPCA533:
                spca504B_PollingDataReady(gspca_dev);
                spca50x_GetFirmware(gspca_dev);
index c579b10..cc87c24 100644 (file)
@@ -1409,7 +1409,7 @@ static int cit_restart_stream(struct gspca_dev *gspca_dev)
        case CIT_MODEL0:
        case CIT_MODEL1:
                cit_write_reg(gspca_dev, 0x0001, 0x0114);
-               /* Fall through */
+               fallthrough;
        case CIT_MODEL2:
        case CIT_MODEL4:
                cit_write_reg(gspca_dev, 0x00c0, 0x010c); /* Go! */
@@ -2725,7 +2725,7 @@ static void sd_stop0(struct gspca_dev *gspca_dev)
                break;
        case CIT_MODEL2:
                v4l2_ctrl_grab(sd->lighting, false);
-               /* Fall through! */
+               fallthrough;
        case CIT_MODEL4:
                cit_model2_Packet1(gspca_dev, 0x0030, 0x0004);
 
index 15a2449..aa285d5 100644 (file)
@@ -6766,7 +6766,7 @@ static int sd_start(struct gspca_dev *gspca_dev)
        case SENSOR_HV7131R:
        case SENSOR_TAS5130C:
                reg_r(gspca_dev, 0x0008);
-               /* fall through */
+               fallthrough;
        case SENSOR_PO2030:
                reg_w(gspca_dev, 0x03, 0x0008);
                break;
@@ -6815,7 +6815,7 @@ static int sd_start(struct gspca_dev *gspca_dev)
        case SENSOR_TAS5130C:
                reg_w(gspca_dev, 0x09, 0x01ad); /* (from win traces) */
                reg_w(gspca_dev, 0x15, 0x01ae);
-               /* fall through */
+               fallthrough;
        case SENSOR_PAS202B:
        case SENSOR_PO2030:
 /*             reg_w(gspca_dev, 0x40, ZC3XX_R117_GGAIN); in win traces */
index 2f135d5..71b719d 100644 (file)
@@ -554,7 +554,7 @@ static int pwc_g_volatile_ctrl(struct v4l2_ctrl *ctrl)
                if (!DEVICE_USE_CODEC3(pdev->type))
                        break;
                /* For CODEC3 where autogain also controls expo */
-               /* fall through */
+               fallthrough;
        case V4L2_CID_EXPOSURE_AUTO:
                if (pdev->exposure_valid && time_before(jiffies,
                                pdev->last_exposure_update + HZ / 4)) {
index 9ba3a2a..df4c5dc 100644 (file)
@@ -430,7 +430,7 @@ static int smsusb_init_device(struct usb_interface *intf, int board_id)
                break;
        case SMS_UNKNOWN_TYPE:
                pr_err("Unspecified sms device type!\n");
-               /* fall-thru */
+               fallthrough;
        default:
                dev->buffer_size = USB2_BUFFER_SIZE;
                dev->response_alignment = align;
index c26a0ff..3a2df36 100644 (file)
@@ -272,13 +272,13 @@ static int snd_tm6000_card_trigger(struct snd_pcm_substream *substream, int cmd)
        int err = 0;
 
        switch (cmd) {
-       case SNDRV_PCM_TRIGGER_PAUSE_RELEASE: /* fall through */
-       case SNDRV_PCM_TRIGGER_RESUME: /* fall through */
+       case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
+       case SNDRV_PCM_TRIGGER_RESUME:
        case SNDRV_PCM_TRIGGER_START:
                atomic_set(&core->stream_started, 1);
                break;
-       case SNDRV_PCM_TRIGGER_PAUSE_PUSH: /* fall through */
-       case SNDRV_PCM_TRIGGER_SUSPEND: /* fall through */
+       case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
+       case SNDRV_PCM_TRIGGER_SUSPEND:
        case SNDRV_PCM_TRIGGER_STOP:
                atomic_set(&core->stream_started, 0);
                break;
index a65d535..6249e26 100644 (file)
@@ -1509,11 +1509,11 @@ static void uvc_video_complete(struct urb *urb)
        default:
                uvc_printk(KERN_WARNING, "Non-zero status (%d) in video "
                        "completion handler.\n", urb->status);
-               /* fall through */
+               fallthrough;
        case -ENOENT:           /* usb_poison_urb() called. */
                if (stream->frozen)
                        return;
-               /* fall through */
+               fallthrough;
        case -ECONNRESET:       /* usb_unlink_urb() called. */
        case -ESHUTDOWN:        /* The endpoint is being disabled. */
                uvc_queue_cancel(queue, urb->status == -ESHUTDOWN);