media: ov5640: Remove redundant register setup
authorMaxime Ripard <maxime.ripard@bootlin.com>
Mon, 3 Dec 2018 08:44:20 +0000 (03:44 -0500)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Wed, 5 Dec 2018 11:47:05 +0000 (06:47 -0500)
The MIPI divider is also cleared as part of the clock setup sequence, so we
can remove that code.

Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
Tested-by: Adam Ford <aford173@gmail.com> #imx6dq
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/i2c/ov5640.c

index 7c18120..e36b049 100644 (file)
@@ -1404,16 +1404,6 @@ static int ov5640_set_stream_dvp(struct ov5640_dev *sensor, bool on)
 
        if (on) {
                /*
-                * reset MIPI PCLK/SERCLK divider
-                *
-                * SC PLL CONTRL1 0
-                * - [3..0]:    MIPI PCLK/SERCLK divider
-                */
-               ret = ov5640_mod_reg(sensor, OV5640_REG_SC_PLL_CTRL1, 0x0f, 0);
-               if (ret)
-                       return ret;
-
-               /*
                 * configure parallel port control lines polarity
                 *
                 * POLARITY CTRL0