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.steven...@raspberrypi.org>
---
 drivers/media/i2c/tc358743.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/media/i2c/tc358743.c b/drivers/media/i2c/tc358743.c
index 06bfdca..2f5763d 100644
--- a/drivers/media/i2c/tc358743.c
+++ b/drivers/media/i2c/tc358743.c
@@ -1905,6 +1905,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)
@@ -1919,7 +1921,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);
-- 
2.7.4

Reply via email to