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 6bd4280..4cde779 100644 --- a/drivers/media/video/mt9m032.c +++ b/drivers/media/video/mt9m032.c @@ -226,7 +226,7 @@ static int mt9m032_setup_pll(struct mt9m032 *sensor) struct mt9m032_platform_data* pdata = sensor->pdata; u16 reg_pll1; unsigned int pre_div; - int res, ret; + int ret; /* TODO: also support other pre-div values */ if (pdata->pll_pre_div != 6) { @@ -253,9 +253,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