media: staging: media: atomisp: remove unnecessary braces
authorDeepak R Varma <drv@mailo.com>
Wed, 28 Apr 2021 18:06:50 +0000 (20:06 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Wed, 19 May 2021 07:51:38 +0000 (09:51 +0200)
According to the coding style guidelines, if...else blocks with single
instructions do not need enclosing braces. This resolves checkpatch
WARNING / CHECK complaints.

Signed-off-by: Deepak R Varma <drv@mailo.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/atomisp/i2c/atomisp-ov2680.c

index c90730513438bd8af6f6da87040871ac772d92f7..f167781e258ac3a165f1c52b87b4679122ac1e13 100644 (file)
@@ -461,11 +461,11 @@ static int ov2680_v_flip(struct v4l2_subdev *sd, s32 value)
        ret = ov2680_read_reg(client, 1, OV2680_FLIP_REG, &val);
        if (ret)
                return ret;
-       if (value) {
+       if (value)
                val |= OV2680_FLIP_MIRROR_BIT_ENABLE;
-       } else {
+       else
                val &= ~OV2680_FLIP_MIRROR_BIT_ENABLE;
-       }
+
        ret = ov2680_write_reg(client, 1,
                               OV2680_FLIP_REG, val);
        if (ret)