V4L/DVB (12536): soc-camera: remove .gain and .exposure struct soc_camera_device members

This makes the soc-camera interface for V4L2 subdevices thinner yet. Handle
gain and exposure internally in each driver just like all other controls.

Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

1 files changed, 21 insertions, 8 deletions

diff --git a/drivers/media/video/mt9m111.c b/drivers/media/video/mt9m111.c
index 186902f9be2e..90da699601ea 100644
--- a/drivers/media/video/mt9m111.c
+++ b/drivers/media/video/mt9m111.c
@@ -153,6 +153,7 @@ struct mt9m111 {
         enum mt9m111_context context;
         struct v4l2_rect rect;
         u32 pixfmt;
+        unsigned int gain;
         unsigned char autoexposure;
         unsigned char datawidth;
         unsigned int powered:1;
@@ -513,7 +514,8 @@ static int mt9m111_set_pixfmt(struct i2c_client *client, u32 pixfmt)
                 ret = mt9m111_setfmt_yuv(client);
                 break;
         default:
-                dev_err(&client->dev, "Pixel format not handled : %x\n", pixfmt);
+                dev_err(&client->dev, "Pixel format not handled : %x\n",
+                        pixfmt);
                 ret = -EINVAL;
         }
 
@@ -536,9 +538,9 @@ static int mt9m111_s_fmt(struct v4l2_subdev *sd, struct v4l2_format *f)
         };
         int ret;
 
-        dev_dbg(&client->dev, "%s fmt=%x left=%d, top=%d, width=%d, height=%d\n",
-                __func__, pix->pixelformat, rect.left, rect.top, rect.width,
-                rect.height);
+        dev_dbg(&client->dev,
+                "%s fmt=%x left=%d, top=%d, width=%d, height=%d\n", __func__,
+                pix->pixelformat, rect.left, rect.top, rect.width, rect.height);
 
         ret = mt9m111_make_rect(client, &rect);
         if (!ret)
@@ -672,8 +674,10 @@ static const struct v4l2_queryctrl mt9m111_controls[] = {
 };
 
 static int mt9m111_resume(struct soc_camera_device *icd);
+static int mt9m111_suspend(struct soc_camera_device *icd, pm_message_t state);
 
 static struct soc_camera_ops mt9m111_ops = {
+        .suspend                = mt9m111_suspend,
         .resume                 = mt9m111_resume,
         .query_bus_param        = mt9m111_query_bus_param,
         .set_bus_param          = mt9m111_set_bus_param,
@@ -714,13 +718,13 @@ static int mt9m111_get_global_gain(struct i2c_client *client)
 
 static int mt9m111_set_global_gain(struct i2c_client *client, int gain)
 {
-        struct soc_camera_device *icd = client->dev.platform_data;
+        struct mt9m111 *mt9m111 = to_mt9m111(client);
         u16 val;
 
         if (gain > 63 * 2 * 2)
                 return -EINVAL;
 
-        icd->gain = gain;
+        mt9m111->gain = gain;
         if ((gain >= 64 * 2) && (gain < 63 * 2 * 2))
                 val = (1 << 10) | (1 << 9) | (gain / 4);
         else if ((gain >= 64) && (gain < 64 * 2))
@@ -844,17 +848,26 @@ static int mt9m111_s_ctrl(struct v4l2_subdev *sd, struct v4l2_control *ctrl)
         return ret;
 }
 
+static int mt9m111_suspend(struct soc_camera_device *icd, pm_message_t state)
+{
+        struct i2c_client *client = to_i2c_client(to_soc_camera_control(icd));
+        struct mt9m111 *mt9m111 = to_mt9m111(client);
+
+        mt9m111->gain = mt9m111_get_global_gain(client);
+
+        return 0;
+}
+
 static int mt9m111_restore_state(struct i2c_client *client)
 {
         struct mt9m111 *mt9m111 = to_mt9m111(client);
-        struct soc_camera_device *icd = client->dev.platform_data;
 
         mt9m111_set_context(client, mt9m111->context);
         mt9m111_set_pixfmt(client, mt9m111->pixfmt);
         mt9m111_setup_rect(client, &mt9m111->rect);
         mt9m111_set_flip(client, mt9m111->hflip, MT9M111_RMB_MIRROR_COLS);
         mt9m111_set_flip(client, mt9m111->vflip, MT9M111_RMB_MIRROR_ROWS);
-        mt9m111_set_global_gain(client, icd->gain);
+        mt9m111_set_global_gain(client, mt9m111->gain);
         mt9m111_set_autoexposure(client, mt9m111->autoexposure);
         mt9m111_set_autowhitebalance(client, mt9m111->autowhitebalance);
         return 0;