drm/mediatek: Add wait_event_timeout when disabling plane
authorJason-JH Lin <jason-jh.lin@mediatek.com>
Tue, 24 Jun 2025 11:31:41 +0000 (19:31 +0800)
committerChun-Kuang Hu <chunkuang.hu@kernel.org>
Thu, 17 Jul 2025 23:18:53 +0000 (23:18 +0000)
commitd208261e9f7c66960587b10473081dc1cecbe50b
treecaac5d6c0b98517e32c024ea5f7c544a4442e568
parent19272b37aa4f83ca52bdf9c16d5d81bdd1354494
drm/mediatek: Add wait_event_timeout when disabling plane

Our hardware registers are set through GCE, not by the CPU.
DRM might assume the hardware is disabled immediately after calling
atomic_disable() of drm_plane, but it is only truly disabled after the
GCE IRQ is triggered.

Additionally, the cursor plane in DRM uses async_commit, so DRM will
not wait for vblank and will free the buffer immediately after calling
atomic_disable().

To prevent the framebuffer from being freed before the layer disable
settings are configured into the hardware, which can cause an IOMMU
fault error, a wait_event_timeout has been added to wait for the
ddp_cmdq_cb() callback,indicating that the GCE IRQ has been triggered.

Fixes: 2f965be7f900 ("drm/mediatek: apply CMDQ control flow")
Signed-off-by: Jason-JH Lin <jason-jh.lin@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: CK Hu <ck.hu@mediatek.com>
Link: https://patchwork.kernel.org/project/linux-mediatek/patch/20250624113223.443274-1-jason-jh.lin@mediatek.com/
Signed-off-by: Chun-Kuang Hu <chunkuang.hu@kernel.org>
drivers/gpu/drm/mediatek/mtk_crtc.c
drivers/gpu/drm/mediatek/mtk_crtc.h
drivers/gpu/drm/mediatek/mtk_plane.c