[media] m5mols: Do not reset the configured pixel format when unexpected
authorSylwester Nawrocki <s.nawrocki@samsung.com>
Tue, 15 Nov 2011 18:55:17 +0000 (15:55 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Fri, 30 Dec 2011 16:38:54 +0000 (14:38 -0200)
Initialize default pixel format in driver probe() rather than in
s_power handler. This also prevents resetting the configuration
applied before the device was powered on.

Acked-by: HeungJun Kim <riverful.kim@samsung.com>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/m5mols/m5mols_core.c

index ea594cf..12f223f 100644 (file)
@@ -864,13 +864,6 @@ static int m5mols_s_power(struct v4l2_subdev *sd, int on)
                ret = m5mols_sensor_power(info, true);
                if (!ret)
                        ret = m5mols_fw_start(sd);
-               if (ret)
-                       return ret;
-
-               info->ffmt[M5MOLS_RESTYPE_MONITOR] =
-                       m5mols_default_ffmt[M5MOLS_RESTYPE_MONITOR];
-               info->ffmt[M5MOLS_RESTYPE_CAPTURE] =
-                       m5mols_default_ffmt[M5MOLS_RESTYPE_CAPTURE];
                return ret;
        }
 
@@ -1007,6 +1000,8 @@ static int __devinit m5mols_probe(struct i2c_client *client,
                goto out_me;
        }
        info->res_type = M5MOLS_RESTYPE_MONITOR;
+       info->ffmt[0] = m5mols_default_ffmt[0];
+       info->ffmt[1] = m5mols_default_ffmt[1];
 
        ret = m5mols_sensor_power(info, true);
        if (ret)