media: atomisp-mt9m114: remove dead data
authorMauro Carvalho Chehab <mchehab@s-opensource.com>
Mon, 16 Apr 2018 16:37:11 +0000 (12:37 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Wed, 16 May 2018 14:32:44 +0000 (10:32 -0400)
It seems that, originally, the logic would allow selecting between
fine and coarse integration. However, only coarse seems to be
implemented.

Get rid of this warning:

  drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c: In function 'mt9m114_s_exposure':
  drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c:1003:6: warning: variable 'exposure_local' set but not used [-Wunused-but-set-variable]
    u16 exposure_local[3];
        ^~~~~~~~~~~~~~

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-mt9m114.c

index 44db9f9f1fc5487f3428adf57a3ec535c09a71d9..454a5c31a2067586b16382b02c5a7cb0ea6c6fac 100644 (file)
@@ -995,12 +995,10 @@ static long mt9m114_s_exposure(struct v4l2_subdev *sd,
        struct mt9m114_device *dev = to_mt9m114_sensor(sd);
        int ret = 0;
        unsigned int coarse_integration = 0;
-       unsigned int fine_integration = 0;
        unsigned int FLines = 0;
        unsigned int FrameLengthLines = 0; /* ExposureTime.FrameLengthLines; */
        unsigned int AnalogGain, DigitalGain;
        u32 AnalogGainToWrite = 0;
-       u16 exposure_local[3];
 
        dev_dbg(&client->dev, "%s(0x%X 0x%X 0x%X)\n", __func__,
                    exposure->integration_time[0], exposure->gain[0],
@@ -1032,10 +1030,7 @@ static long mt9m114_s_exposure(struct v4l2_subdev *sd,
                return -EINVAL;
        }
 
-       /* set coarse/fine integration */
-       exposure_local[0] = REG_EXPO_COARSE;
-       exposure_local[1] = (u16)coarse_integration;
-       exposure_local[2] = (u16)fine_integration;
+       /* set coarse integration */
        /* 3A provide real exposure time.
                should not translate to any value here. */
        ret = mt9m114_write_reg(client, MISENSOR_16BIT,