media: staging: atomisp-gc2235: don't fill an unused var
authorMauro Carvalho Chehab <mchehab@s-opensource.com>
Mon, 16 Apr 2018 16:37:07 +0000 (12:37 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Wed, 16 May 2018 14:28:59 +0000 (10:28 -0400)
The code with uses the dummy var is commented out. So,
coment out its definition/initialization.

Fix this warning:

  drivers/staging/media/atomisp/i2c/atomisp-gc2235.c: In function 'gc2235_get_intg_factor':
  drivers/staging/media/atomisp/i2c/atomisp-gc2235.c:249:26: warning: variable 'dummy' set but not used [-Wunused-but-set-variable]
    u16 reg_val, reg_val_h, dummy;
                            ^~~~~

Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/staging/media/atomisp/i2c/atomisp-gc2235.c

index 93f9c618f3d879ec14cd5ea4fef60713479c594e..4b6b6568b3cf352fc9dfe7e52849be324c0fc35d 100644 (file)
@@ -246,7 +246,7 @@ static int gc2235_get_intg_factor(struct i2c_client *client,
        struct v4l2_subdev *sd = i2c_get_clientdata(client);
        struct gc2235_device *dev = to_gc2235_sensor(sd);
        struct atomisp_sensor_mode_data *buf = &info->data;
-       u16 reg_val, reg_val_h, dummy;
+       u16 reg_val, reg_val_h;
        int ret;
 
        if (!info)
@@ -316,7 +316,9 @@ static int gc2235_get_intg_factor(struct i2c_client *client,
        if (ret)
                return ret;
 
-       dummy = (reg_val_h << 8) | reg_val;
+#if 0
+       u16 dummy = (reg_val_h << 8) | reg_val;
+#endif
 
        ret = gc2235_read_reg(client, GC2235_8BIT,
                                        GC2235_SH_DELAY_H, &reg_val_h);