From: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> --- drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c b/drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c index be12d0d2f42f..db538f3d88ec 100644 --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c @@ -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); - - msleep(300); } void mxc_isi_channel_disable(struct mxc_isi_pipe *pipe) -- Regards, Laurent Pinchart
