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, 5 insertions(+), 6 deletions(-)

diff --git a/drivers/media/i2c/smiapp/smiapp-core.c 
b/drivers/media/i2c/smiapp/smiapp-core.c
index 76d7d204ec17..61de8cdccc4b 100644
--- a/drivers/media/i2c/smiapp/smiapp-core.c
+++ b/drivers/media/i2c/smiapp/smiapp-core.c
@@ -1674,13 +1674,12 @@ static void smiapp_propagate(struct v4l2_subdev *subdev,
                                sensor->binning_vertical = 1;
                        }
                }
-               /* Fall through */
-       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 +2061,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 +2715,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