* [PATCH -next 011/491] ARM/QUALCOMM SUPPORT: Use fallthrough; [not found] <cover.1583896344.git.joe@perches.com> @ 2020-03-11 4:51 ` Joe Perches 2020-03-11 5:23 ` Amit Kucheria 2020-03-13 12:04 ` Stanimir Varbanov 0 siblings, 2 replies; 3+ messages in thread From: Joe Perches @ 2020-03-11 4:51 UTC (permalink / raw) To: Stanimir Varbanov, Andy Gross, Bjorn Andersson, Ohad Ben-Cohen, Amit Kucheria Cc: Mauro Carvalho Chehab, Kishon Vijay Abraham I, Linus Walleij, Zhang Rui, Daniel Lezcano, linux-media, linux-arm-msm, linux-kernel, linux-gpio, linux-remoteproc, linux-pm Convert the various uses of fallthrough comments to fallthrough; Done via script Link: https://lore.kernel.org/lkml/b56602fcf79f849e733e7b521bb0e17895d390fa.1582230379.git.joe.com/ Signed-off-by: Joe Perches <joe@perches.com> --- drivers/media/platform/qcom/venus/vdec.c | 2 +- drivers/phy/qualcomm/phy-qcom-usb-hs.c | 2 +- drivers/pinctrl/qcom/pinctrl-spmi-gpio.c | 4 ++-- drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c | 2 +- drivers/rpmsg/qcom_glink_native.c | 4 ++-- drivers/soc/qcom/socinfo.c | 16 ++++++++-------- drivers/thermal/qcom/tsens-v0_1.c | 8 ++++---- drivers/thermal/qcom/tsens-v1.c | 4 ++-- 8 files changed, 21 insertions(+), 21 deletions(-) diff --git a/drivers/media/platform/qcom/venus/vdec.c b/drivers/media/platform/qcom/venus/vdec.c index f34920..9e0451 100644 --- a/drivers/media/platform/qcom/venus/vdec.c +++ b/drivers/media/platform/qcom/venus/vdec.c @@ -983,7 +983,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); - /* fallthrough */ + fallthrough; case VENUS_DEC_STATE_DRAIN: vdec_cancel_dst_buffers(inst); inst->codec_state = VENUS_DEC_STATE_STOPPED; diff --git a/drivers/phy/qualcomm/phy-qcom-usb-hs.c b/drivers/phy/qualcomm/phy-qcom-usb-hs.c index 610542..327df1a 100644 --- a/drivers/phy/qualcomm/phy-qcom-usb-hs.c +++ b/drivers/phy/qualcomm/phy-qcom-usb-hs.c @@ -53,7 +53,7 @@ static int qcom_usb_hs_phy_set_mode(struct phy *phy, case PHY_MODE_USB_OTG: case PHY_MODE_USB_HOST: val |= ULPI_INT_IDGRD; - /* fall through */ + fallthrough; case PHY_MODE_USB_DEVICE: val |= ULPI_INT_SESS_VALID; default: diff --git a/drivers/pinctrl/qcom/pinctrl-spmi-gpio.c b/drivers/pinctrl/qcom/pinctrl-spmi-gpio.c index fe0be8..3b0ab0e 100644 --- a/drivers/pinctrl/qcom/pinctrl-spmi-gpio.c +++ b/drivers/pinctrl/qcom/pinctrl-spmi-gpio.c @@ -793,13 +793,13 @@ static int pmic_gpio_populate(struct pmic_gpio_state *state, switch (subtype) { case PMIC_GPIO_SUBTYPE_GPIO_4CH: pad->have_buffer = true; - /* Fall through */ + fallthrough; case PMIC_GPIO_SUBTYPE_GPIOC_4CH: pad->num_sources = 4; break; case PMIC_GPIO_SUBTYPE_GPIO_8CH: pad->have_buffer = true; - /* Fall through */ + fallthrough; case PMIC_GPIO_SUBTYPE_GPIOC_8CH: pad->num_sources = 8; break; diff --git a/drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c b/drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c index 338a15..b5949f7 100644 --- a/drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c +++ b/drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c @@ -346,7 +346,7 @@ static int pm8xxx_pin_config_set(struct pinctrl_dev *pctldev, return -EINVAL; } pin->pull_up_strength = arg; - /* FALLTHROUGH */ + fallthrough; case PIN_CONFIG_BIAS_PULL_UP: pin->bias = pin->pull_up_strength; banks |= BIT(2); diff --git a/drivers/rpmsg/qcom_glink_native.c b/drivers/rpmsg/qcom_glink_native.c index 1995f5b..f40312 100644 --- a/drivers/rpmsg/qcom_glink_native.c +++ b/drivers/rpmsg/qcom_glink_native.c @@ -553,7 +553,7 @@ static void qcom_glink_receive_version(struct qcom_glink *glink, break; case GLINK_VERSION_1: glink->features &= features; - /* FALLTHROUGH */ + fallthrough; default: qcom_glink_send_version_ack(glink); break; @@ -584,7 +584,7 @@ static void qcom_glink_receive_version_ack(struct qcom_glink *glink, break; glink->features &= features; - /* FALLTHROUGH */ + fallthrough; default: qcom_glink_send_version(glink); break; diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c index 7864b7..8ae5646 100644 --- a/drivers/soc/qcom/socinfo.c +++ b/drivers/soc/qcom/socinfo.c @@ -325,7 +325,7 @@ static void socinfo_debugfs_init(struct qcom_socinfo *qcom_socinfo, debugfs_create_x32("raw_device_number", 0400, qcom_socinfo->dbg_root, &qcom_socinfo->info.raw_device_num); - /* Fall through */ + fallthrough; case SOCINFO_VERSION(0, 11): case SOCINFO_VERSION(0, 10): case SOCINFO_VERSION(0, 9): @@ -333,12 +333,12 @@ static void socinfo_debugfs_init(struct qcom_socinfo *qcom_socinfo, debugfs_create_u32("foundry_id", 0400, qcom_socinfo->dbg_root, &qcom_socinfo->info.foundry_id); - /* Fall through */ + fallthrough; case SOCINFO_VERSION(0, 8): case SOCINFO_VERSION(0, 7): DEBUGFS_ADD(info, pmic_model); DEBUGFS_ADD(info, pmic_die_rev); - /* Fall through */ + fallthrough; case SOCINFO_VERSION(0, 6): qcom_socinfo->info.hw_plat_subtype = __le32_to_cpu(info->hw_plat_subtype); @@ -346,7 +346,7 @@ static void socinfo_debugfs_init(struct qcom_socinfo *qcom_socinfo, debugfs_create_u32("hardware_platform_subtype", 0400, qcom_socinfo->dbg_root, &qcom_socinfo->info.hw_plat_subtype); - /* Fall through */ + fallthrough; case SOCINFO_VERSION(0, 5): qcom_socinfo->info.accessory_chip = __le32_to_cpu(info->accessory_chip); @@ -354,27 +354,27 @@ static void socinfo_debugfs_init(struct qcom_socinfo *qcom_socinfo, debugfs_create_u32("accessory_chip", 0400, qcom_socinfo->dbg_root, &qcom_socinfo->info.accessory_chip); - /* Fall through */ + fallthrough; case SOCINFO_VERSION(0, 4): qcom_socinfo->info.plat_ver = __le32_to_cpu(info->plat_ver); debugfs_create_u32("platform_version", 0400, qcom_socinfo->dbg_root, &qcom_socinfo->info.plat_ver); - /* Fall through */ + fallthrough; case SOCINFO_VERSION(0, 3): qcom_socinfo->info.hw_plat = __le32_to_cpu(info->hw_plat); debugfs_create_u32("hardware_platform", 0400, qcom_socinfo->dbg_root, &qcom_socinfo->info.hw_plat); - /* Fall through */ + fallthrough; case SOCINFO_VERSION(0, 2): qcom_socinfo->info.raw_ver = __le32_to_cpu(info->raw_ver); debugfs_create_u32("raw_version", 0400, qcom_socinfo->dbg_root, &qcom_socinfo->info.raw_ver); - /* Fall through */ + fallthrough; case SOCINFO_VERSION(0, 1): DEBUGFS_ADD(info, build_id); break; diff --git a/drivers/thermal/qcom/tsens-v0_1.c b/drivers/thermal/qcom/tsens-v0_1.c index 4b8dd6..893ce1 100644 --- a/drivers/thermal/qcom/tsens-v0_1.c +++ b/drivers/thermal/qcom/tsens-v0_1.c @@ -163,7 +163,7 @@ static int calibrate_8916(struct tsens_priv *priv) p2[4] = (qfprom_cdata[1] & MSM8916_S4_P2_MASK) >> MSM8916_S4_P2_SHIFT; for (i = 0; i < priv->num_sensors; i++) p2[i] = ((base1 + p2[i]) << 3); - /* Fall through */ + fallthrough; case ONE_PT_CALIB2: base0 = (qfprom_cdata[0] & MSM8916_BASE0_MASK); p1[0] = (qfprom_cdata[0] & MSM8916_S0_P1_MASK) >> MSM8916_S0_P1_SHIFT; @@ -228,7 +228,7 @@ static int calibrate_8974(struct tsens_priv *priv) p2[8] = (calib[5] & S8_P2_BKP_MASK) >> S8_P2_BKP_SHIFT; p2[9] = (calib[5] & S9_P2_BKP_MASK) >> S9_P2_BKP_SHIFT; p2[10] = (calib[5] & S10_P2_BKP_MASK) >> S10_P2_BKP_SHIFT; - /* Fall through */ + fallthrough; case ONE_PT_CALIB: case ONE_PT_CALIB2: base1 = bkp[0] & BASE1_MASK; @@ -263,7 +263,7 @@ static int calibrate_8974(struct tsens_priv *priv) p2[8] = (calib[4] & S8_P2_MASK) >> S8_P2_SHIFT; p2[9] = (calib[4] & S9_P2_MASK) >> S9_P2_SHIFT; p2[10] = (calib[4] & S10_P2_MASK) >> S10_P2_SHIFT; - /* Fall through */ + fallthrough; case ONE_PT_CALIB: case ONE_PT_CALIB2: base1 = calib[0] & BASE1_MASK; @@ -293,7 +293,7 @@ static int calibrate_8974(struct tsens_priv *priv) p2[i] <<= 2; p2[i] |= BIT_APPEND; } - /* Fall through */ + fallthrough; case ONE_PT_CALIB2: for (i = 0; i < priv->num_sensors; i++) { p1[i] += base1; diff --git a/drivers/thermal/qcom/tsens-v1.c b/drivers/thermal/qcom/tsens-v1.c index bd2ddb..d096d3 100644 --- a/drivers/thermal/qcom/tsens-v1.c +++ b/drivers/thermal/qcom/tsens-v1.c @@ -202,7 +202,7 @@ static int calibrate_v1(struct tsens_priv *priv) p2[9] = (qfprom_cdata[3] & S9_P2_MASK) >> S9_P2_SHIFT; for (i = 0; i < priv->num_sensors; i++) p2[i] = ((base1 + p2[i]) << 2); - /* Fall through */ + fallthrough; case ONE_PT_CALIB2: base0 = (qfprom_cdata[4] & BASE0_MASK) >> BASE0_SHIFT; p1[0] = (qfprom_cdata[0] & S0_P1_MASK) >> S0_P1_SHIFT; @@ -263,7 +263,7 @@ static int calibrate_8976(struct tsens_priv *priv) for (i = 0; i < priv->num_sensors; i++) p2[i] = ((base1 + p2[i]) << 2); - /* Fall through */ + fallthrough; case ONE_PT_CALIB2: base0 = qfprom_cdata[0] & MSM8976_BASE0_MASK; p1[0] = (qfprom_cdata[0] & MSM8976_S0_P1_MASK) >> MSM8976_S0_P1_SHIFT; -- 2.24.0 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH -next 011/491] ARM/QUALCOMM SUPPORT: Use fallthrough; 2020-03-11 4:51 ` [PATCH -next 011/491] ARM/QUALCOMM SUPPORT: Use fallthrough; Joe Perches @ 2020-03-11 5:23 ` Amit Kucheria 2020-03-13 12:04 ` Stanimir Varbanov 1 sibling, 0 replies; 3+ messages in thread From: Amit Kucheria @ 2020-03-11 5:23 UTC (permalink / raw) To: Joe Perches Cc: Stanimir Varbanov, Andy Gross, Bjorn Andersson, Ohad Ben-Cohen, Mauro Carvalho Chehab, Kishon Vijay Abraham I, Linus Walleij, Zhang Rui, Daniel Lezcano, linux-media, linux-arm-msm, LKML, linux-gpio, linux-remoteproc, Linux PM list On Wed, Mar 11, 2020 at 10:37 AM Joe Perches <joe@perches.com> wrote: > > Convert the various uses of fallthrough comments to fallthrough; > > Done via script > Link: https://lore.kernel.org/lkml/b56602fcf79f849e733e7b521bb0e17895d390fa.1582230379.git.joe.com/ Reviewed-by: Amit Kucheria <amit.kucheria@linaro.org> > Signed-off-by: Joe Perches <joe@perches.com> > --- > drivers/media/platform/qcom/venus/vdec.c | 2 +- > drivers/phy/qualcomm/phy-qcom-usb-hs.c | 2 +- > drivers/pinctrl/qcom/pinctrl-spmi-gpio.c | 4 ++-- > drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c | 2 +- > drivers/rpmsg/qcom_glink_native.c | 4 ++-- > drivers/soc/qcom/socinfo.c | 16 ++++++++-------- > drivers/thermal/qcom/tsens-v0_1.c | 8 ++++---- > drivers/thermal/qcom/tsens-v1.c | 4 ++-- > 8 files changed, 21 insertions(+), 21 deletions(-) > > diff --git a/drivers/media/platform/qcom/venus/vdec.c b/drivers/media/platform/qcom/venus/vdec.c > index f34920..9e0451 100644 > --- a/drivers/media/platform/qcom/venus/vdec.c > +++ b/drivers/media/platform/qcom/venus/vdec.c > @@ -983,7 +983,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); > - /* fallthrough */ > + fallthrough; > case VENUS_DEC_STATE_DRAIN: > vdec_cancel_dst_buffers(inst); > inst->codec_state = VENUS_DEC_STATE_STOPPED; > diff --git a/drivers/phy/qualcomm/phy-qcom-usb-hs.c b/drivers/phy/qualcomm/phy-qcom-usb-hs.c > index 610542..327df1a 100644 > --- a/drivers/phy/qualcomm/phy-qcom-usb-hs.c > +++ b/drivers/phy/qualcomm/phy-qcom-usb-hs.c > @@ -53,7 +53,7 @@ static int qcom_usb_hs_phy_set_mode(struct phy *phy, > case PHY_MODE_USB_OTG: > case PHY_MODE_USB_HOST: > val |= ULPI_INT_IDGRD; > - /* fall through */ > + fallthrough; > case PHY_MODE_USB_DEVICE: > val |= ULPI_INT_SESS_VALID; > default: > diff --git a/drivers/pinctrl/qcom/pinctrl-spmi-gpio.c b/drivers/pinctrl/qcom/pinctrl-spmi-gpio.c > index fe0be8..3b0ab0e 100644 > --- a/drivers/pinctrl/qcom/pinctrl-spmi-gpio.c > +++ b/drivers/pinctrl/qcom/pinctrl-spmi-gpio.c > @@ -793,13 +793,13 @@ static int pmic_gpio_populate(struct pmic_gpio_state *state, > switch (subtype) { > case PMIC_GPIO_SUBTYPE_GPIO_4CH: > pad->have_buffer = true; > - /* Fall through */ > + fallthrough; > case PMIC_GPIO_SUBTYPE_GPIOC_4CH: > pad->num_sources = 4; > break; > case PMIC_GPIO_SUBTYPE_GPIO_8CH: > pad->have_buffer = true; > - /* Fall through */ > + fallthrough; > case PMIC_GPIO_SUBTYPE_GPIOC_8CH: > pad->num_sources = 8; > break; > diff --git a/drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c b/drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c > index 338a15..b5949f7 100644 > --- a/drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c > +++ b/drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c > @@ -346,7 +346,7 @@ static int pm8xxx_pin_config_set(struct pinctrl_dev *pctldev, > return -EINVAL; > } > pin->pull_up_strength = arg; > - /* FALLTHROUGH */ > + fallthrough; > case PIN_CONFIG_BIAS_PULL_UP: > pin->bias = pin->pull_up_strength; > banks |= BIT(2); > diff --git a/drivers/rpmsg/qcom_glink_native.c b/drivers/rpmsg/qcom_glink_native.c > index 1995f5b..f40312 100644 > --- a/drivers/rpmsg/qcom_glink_native.c > +++ b/drivers/rpmsg/qcom_glink_native.c > @@ -553,7 +553,7 @@ static void qcom_glink_receive_version(struct qcom_glink *glink, > break; > case GLINK_VERSION_1: > glink->features &= features; > - /* FALLTHROUGH */ > + fallthrough; > default: > qcom_glink_send_version_ack(glink); > break; > @@ -584,7 +584,7 @@ static void qcom_glink_receive_version_ack(struct qcom_glink *glink, > break; > > glink->features &= features; > - /* FALLTHROUGH */ > + fallthrough; > default: > qcom_glink_send_version(glink); > break; > diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c > index 7864b7..8ae5646 100644 > --- a/drivers/soc/qcom/socinfo.c > +++ b/drivers/soc/qcom/socinfo.c > @@ -325,7 +325,7 @@ static void socinfo_debugfs_init(struct qcom_socinfo *qcom_socinfo, > debugfs_create_x32("raw_device_number", 0400, > qcom_socinfo->dbg_root, > &qcom_socinfo->info.raw_device_num); > - /* Fall through */ > + fallthrough; > case SOCINFO_VERSION(0, 11): > case SOCINFO_VERSION(0, 10): > case SOCINFO_VERSION(0, 9): > @@ -333,12 +333,12 @@ static void socinfo_debugfs_init(struct qcom_socinfo *qcom_socinfo, > > debugfs_create_u32("foundry_id", 0400, qcom_socinfo->dbg_root, > &qcom_socinfo->info.foundry_id); > - /* Fall through */ > + fallthrough; > case SOCINFO_VERSION(0, 8): > case SOCINFO_VERSION(0, 7): > DEBUGFS_ADD(info, pmic_model); > DEBUGFS_ADD(info, pmic_die_rev); > - /* Fall through */ > + fallthrough; > case SOCINFO_VERSION(0, 6): > qcom_socinfo->info.hw_plat_subtype = > __le32_to_cpu(info->hw_plat_subtype); > @@ -346,7 +346,7 @@ static void socinfo_debugfs_init(struct qcom_socinfo *qcom_socinfo, > debugfs_create_u32("hardware_platform_subtype", 0400, > qcom_socinfo->dbg_root, > &qcom_socinfo->info.hw_plat_subtype); > - /* Fall through */ > + fallthrough; > case SOCINFO_VERSION(0, 5): > qcom_socinfo->info.accessory_chip = > __le32_to_cpu(info->accessory_chip); > @@ -354,27 +354,27 @@ static void socinfo_debugfs_init(struct qcom_socinfo *qcom_socinfo, > debugfs_create_u32("accessory_chip", 0400, > qcom_socinfo->dbg_root, > &qcom_socinfo->info.accessory_chip); > - /* Fall through */ > + fallthrough; > case SOCINFO_VERSION(0, 4): > qcom_socinfo->info.plat_ver = __le32_to_cpu(info->plat_ver); > > debugfs_create_u32("platform_version", 0400, > qcom_socinfo->dbg_root, > &qcom_socinfo->info.plat_ver); > - /* Fall through */ > + fallthrough; > case SOCINFO_VERSION(0, 3): > qcom_socinfo->info.hw_plat = __le32_to_cpu(info->hw_plat); > > debugfs_create_u32("hardware_platform", 0400, > qcom_socinfo->dbg_root, > &qcom_socinfo->info.hw_plat); > - /* Fall through */ > + fallthrough; > case SOCINFO_VERSION(0, 2): > qcom_socinfo->info.raw_ver = __le32_to_cpu(info->raw_ver); > > debugfs_create_u32("raw_version", 0400, qcom_socinfo->dbg_root, > &qcom_socinfo->info.raw_ver); > - /* Fall through */ > + fallthrough; > case SOCINFO_VERSION(0, 1): > DEBUGFS_ADD(info, build_id); > break; > diff --git a/drivers/thermal/qcom/tsens-v0_1.c b/drivers/thermal/qcom/tsens-v0_1.c > index 4b8dd6..893ce1 100644 > --- a/drivers/thermal/qcom/tsens-v0_1.c > +++ b/drivers/thermal/qcom/tsens-v0_1.c > @@ -163,7 +163,7 @@ static int calibrate_8916(struct tsens_priv *priv) > p2[4] = (qfprom_cdata[1] & MSM8916_S4_P2_MASK) >> MSM8916_S4_P2_SHIFT; > for (i = 0; i < priv->num_sensors; i++) > p2[i] = ((base1 + p2[i]) << 3); > - /* Fall through */ > + fallthrough; > case ONE_PT_CALIB2: > base0 = (qfprom_cdata[0] & MSM8916_BASE0_MASK); > p1[0] = (qfprom_cdata[0] & MSM8916_S0_P1_MASK) >> MSM8916_S0_P1_SHIFT; > @@ -228,7 +228,7 @@ static int calibrate_8974(struct tsens_priv *priv) > p2[8] = (calib[5] & S8_P2_BKP_MASK) >> S8_P2_BKP_SHIFT; > p2[9] = (calib[5] & S9_P2_BKP_MASK) >> S9_P2_BKP_SHIFT; > p2[10] = (calib[5] & S10_P2_BKP_MASK) >> S10_P2_BKP_SHIFT; > - /* Fall through */ > + fallthrough; > case ONE_PT_CALIB: > case ONE_PT_CALIB2: > base1 = bkp[0] & BASE1_MASK; > @@ -263,7 +263,7 @@ static int calibrate_8974(struct tsens_priv *priv) > p2[8] = (calib[4] & S8_P2_MASK) >> S8_P2_SHIFT; > p2[9] = (calib[4] & S9_P2_MASK) >> S9_P2_SHIFT; > p2[10] = (calib[4] & S10_P2_MASK) >> S10_P2_SHIFT; > - /* Fall through */ > + fallthrough; > case ONE_PT_CALIB: > case ONE_PT_CALIB2: > base1 = calib[0] & BASE1_MASK; > @@ -293,7 +293,7 @@ static int calibrate_8974(struct tsens_priv *priv) > p2[i] <<= 2; > p2[i] |= BIT_APPEND; > } > - /* Fall through */ > + fallthrough; > case ONE_PT_CALIB2: > for (i = 0; i < priv->num_sensors; i++) { > p1[i] += base1; > diff --git a/drivers/thermal/qcom/tsens-v1.c b/drivers/thermal/qcom/tsens-v1.c > index bd2ddb..d096d3 100644 > --- a/drivers/thermal/qcom/tsens-v1.c > +++ b/drivers/thermal/qcom/tsens-v1.c > @@ -202,7 +202,7 @@ static int calibrate_v1(struct tsens_priv *priv) > p2[9] = (qfprom_cdata[3] & S9_P2_MASK) >> S9_P2_SHIFT; > for (i = 0; i < priv->num_sensors; i++) > p2[i] = ((base1 + p2[i]) << 2); > - /* Fall through */ > + fallthrough; > case ONE_PT_CALIB2: > base0 = (qfprom_cdata[4] & BASE0_MASK) >> BASE0_SHIFT; > p1[0] = (qfprom_cdata[0] & S0_P1_MASK) >> S0_P1_SHIFT; > @@ -263,7 +263,7 @@ static int calibrate_8976(struct tsens_priv *priv) > > for (i = 0; i < priv->num_sensors; i++) > p2[i] = ((base1 + p2[i]) << 2); > - /* Fall through */ > + fallthrough; > case ONE_PT_CALIB2: > base0 = qfprom_cdata[0] & MSM8976_BASE0_MASK; > p1[0] = (qfprom_cdata[0] & MSM8976_S0_P1_MASK) >> MSM8976_S0_P1_SHIFT; > -- > 2.24.0 > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH -next 011/491] ARM/QUALCOMM SUPPORT: Use fallthrough; 2020-03-11 4:51 ` [PATCH -next 011/491] ARM/QUALCOMM SUPPORT: Use fallthrough; Joe Perches 2020-03-11 5:23 ` Amit Kucheria @ 2020-03-13 12:04 ` Stanimir Varbanov 1 sibling, 0 replies; 3+ messages in thread From: Stanimir Varbanov @ 2020-03-13 12:04 UTC (permalink / raw) To: Joe Perches, Stanimir Varbanov, Andy Gross, Bjorn Andersson, Ohad Ben-Cohen, Amit Kucheria Cc: Mauro Carvalho Chehab, Kishon Vijay Abraham I, Linus Walleij, Zhang Rui, Daniel Lezcano, linux-media, linux-arm-msm, linux-kernel, linux-gpio, linux-remoteproc, linux-pm Hi, On 3/11/20 6:51 AM, Joe Perches wrote: > Convert the various uses of fallthrough comments to fallthrough; > > Done via script > Link: https://lore.kernel.org/lkml/b56602fcf79f849e733e7b521bb0e17895d390fa.1582230379.git.joe.com/ > > Signed-off-by: Joe Perches <joe@perches.com> > --- > drivers/media/platform/qcom/venus/vdec.c | 2 +- For the Venus bits: Acked-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> > drivers/phy/qualcomm/phy-qcom-usb-hs.c | 2 +- > drivers/pinctrl/qcom/pinctrl-spmi-gpio.c | 4 ++-- > drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c | 2 +- > drivers/rpmsg/qcom_glink_native.c | 4 ++-- > drivers/soc/qcom/socinfo.c | 16 ++++++++-------- > drivers/thermal/qcom/tsens-v0_1.c | 8 ++++---- > drivers/thermal/qcom/tsens-v1.c | 4 ++-- > 8 files changed, 21 insertions(+), 21 deletions(-) > -- regards, Stan ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-03-13 12:04 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <cover.1583896344.git.joe@perches.com>
2020-03-11 4:51 ` [PATCH -next 011/491] ARM/QUALCOMM SUPPORT: Use fallthrough; Joe Perches
2020-03-11 5:23 ` Amit Kucheria
2020-03-13 12:04 ` Stanimir Varbanov
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).