media: nxp: imx8-isi: Remove 300ms sleep after enabling channel
authorJacopo Mondi <jacopo.mondi@ideasonboard.com>
Tue, 14 Feb 2023 17:35:39 +0000 (17:35 +0000)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Sun, 16 Apr 2023 06:38:01 +0000 (07:38 +0100)
The current implementation of the mxc_isi_channel_enable() function has
a hard 300ms sleep. This was copied from the NXP BSP, which has since
dropped the sleep in a commit that implied the delay was meant to wait
for the sensor to be "stable". As the sensor is started after the ISI,
the delay won't affect sensor operation. Drop it.

Signed-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c

index be12d0d..db538f3 100644 (file)
@@ -515,8 +515,6 @@ void mxc_isi_channel_enable(struct mxc_isi_pipe *pipe)
        mxc_isi_write(pipe, CHNL_CTRL, val);
 
        mutex_unlock(&pipe->lock);
        mxc_isi_write(pipe, CHNL_CTRL, val);
 
        mutex_unlock(&pipe->lock);
-
-       msleep(300);
 }
 
 void mxc_isi_channel_disable(struct mxc_isi_pipe *pipe)
 }
 
 void mxc_isi_channel_disable(struct mxc_isi_pipe *pipe)