There's not need to read register MT9M032_READ_MODE1 when setting up the PLL. Remove the read call. Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> --- drivers/media/video/mt9m032.c | 5 +---- 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/drivers/media/video/mt9m032.c b/drivers/media/video/mt9m032.c index b636ad4..8109bf1 100644 --- a/drivers/media/video/mt9m032.c +++ b/drivers/media/video/mt9m032.c @@ -223,7 +223,7 @@ static int mt9m032_setup_pll(struct mt9m032 *sensor) unsigned int pre_div; unsigned int pll_out_div; unsigned int pll_mul; - int res, ret; + int ret; pre_div = 6; @@ -244,9 +244,6 @@ static int mt9m032_setup_pll(struct mt9m032 *sensor) /* more reserved, Continuous */ /* Master Mode */ if (!ret) - res = mt9m032_read_reg(client, MT9M032_READ_MODE1); - - if (!ret) ret = mt9m032_write_reg(client, MT9M032_FORMATTER1, 0x111e); /* Set 14-bit mode, select 7 divider */ -- 1.7.3.4 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html