Remove switch fall-through cases in the driver.

Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
---
 drivers/media/i2c/smiapp/smiapp-core.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/drivers/media/i2c/smiapp/smiapp-core.c 
b/drivers/media/i2c/smiapp/smiapp-core.c
index 76d7d204ec17..c6202f3a4015 100644
--- a/drivers/media/i2c/smiapp/smiapp-core.c
+++ b/drivers/media/i2c/smiapp/smiapp-core.c
@@ -1674,13 +1674,14 @@ static void smiapp_propagate(struct v4l2_subdev *subdev,
                                sensor->binning_vertical = 1;
                        }
                }
-               /* Fall through */
+               break;
        case V4L2_SEL_TGT_COMPOSE:
-               *crops[SMIAPP_PAD_SRC] = *comp;
                break;
        default:
-               BUG();
+               WARN_ON(1);
+               return;
        }
+       *crops[SMIAPP_PAD_SRC] = *comp;
 }
 
 static const struct smiapp_csi_data_format
@@ -2062,7 +2063,7 @@ static int __smiapp_sel_supported(struct v4l2_subdev 
*subdev,
                    && sensor->limits[SMIAPP_LIMIT_SCALING_CAPABILITY]
                    != SMIAPP_SCALING_CAPABILITY_NONE)
                        return 0;
-               /* Fall through */
+               return -EINVAL;
        default:
                return -EINVAL;
        }
@@ -2716,7 +2717,7 @@ static struct smiapp_hwconfig *smiapp_get_hwconfig(struct 
device *dev)
                case 180:
                        hwcfg->module_board_orient =
                                SMIAPP_MODULE_BOARD_ORIENT_180;
-                       /* Fall through */
+                       break;
                case 0:
                        break;
                default:
-- 
2.20.1

Reply via email to