[media] tc358743: Setup default mbus_fmt before registering
authorDave Stevenson <dave.stevenson@raspberrypi.org>
Fri, 2 Jun 2017 12:18:13 +0000 (09:18 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Wed, 7 Jun 2017 12:30:54 +0000 (09:30 -0300)
Previously the mbus_fmt_code was set after the device was
registered. If a connected sub-device called tc358743_get_fmt
prior to that point it would get an invalid code back.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.org>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/i2c/tc358743.c

index 6ba65c783161e2668aa50d885dff7fe2c3d5adf6..7d20cac9711c71d88f9b73d76cfce9bda1d62afc 100644 (file)
@@ -1910,6 +1910,8 @@ static int tc358743_probe(struct i2c_client *client,
        if (err < 0)
                goto err_hdl;
 
+       state->mbus_fmt_code = MEDIA_BUS_FMT_RGB888_1X24;
+
        sd->dev = &client->dev;
        err = v4l2_async_register_subdev(sd);
        if (err < 0)
@@ -1924,7 +1926,6 @@ static int tc358743_probe(struct i2c_client *client,
 
        tc358743_s_dv_timings(sd, &default_timing);
 
-       state->mbus_fmt_code = MEDIA_BUS_FMT_RGB888_1X24;
        tc358743_set_csi_color_space(sd);
 
        tc358743_init_interrupts(sd);