drm/mediatek: Add pull-down MIPI operation in mtk_dsi_poweroff function
authorXinlei Lee <xinlei.lee@mediatek.com>
Fri, 20 May 2022 02:00:07 +0000 (10:00 +0800)
committerChun-Kuang Hu <chunkuang.hu@kernel.org>
Mon, 6 Jun 2022 23:36:45 +0000 (07:36 +0800)
In the dsi_enable function, mtk_dsi_rxtx_control is to
pull up the MIPI signal operation. Before dsi_disable,
MIPI should also be pulled down by writing a register
instead of disabling dsi.

If disable dsi without pulling the mipi signal low, the value of
the register will still maintain the setting of the mipi signal being
pulled high.
After resume, even if the mipi signal is not pulled high, it will still
be in the high state.

Fixes: 2e54c14e310f ("drm/mediatek: Add DSI sub driver")

Link: https://patchwork.kernel.org/project/linux-mediatek/patch/1653012007-11854-5-git-send-email-xinlei.lee@mediatek.com/
Signed-off-by: Jitao Shi <jitao.shi@mediatek.com>
Signed-off-by: Xinlei Lee <xinlei.lee@mediatek.com>
Reviewed-by: Rex-BC Chen <rex-bc.chen@mediatek.com>
Signed-off-by: Chun-Kuang Hu <chunkuang.hu@kernel.org>
drivers/gpu/drm/mediatek/mtk_dsi.c

index 907d07e..af2f123 100644 (file)
@@ -688,6 +688,8 @@ static void mtk_dsi_poweroff(struct mtk_dsi *dsi)
        mtk_dsi_reset_engine(dsi);
        mtk_dsi_lane0_ulp_mode_enter(dsi);
        mtk_dsi_clk_ulp_mode_enter(dsi);
+       /* set the lane number as 0 to pull down mipi */
+       writel(0, dsi->regs + DSI_TXRX_CTRL);
 
        mtk_dsi_disable(dsi);