drm/panel-edp: use put_sync in unprepare
authorHsin-Yi Wang <hsinyi@chromium.org>
Wed, 20 Dec 2023 22:13:11 +0000 (14:13 -0800)
committerDouglas Anderson <dianders@chromium.org>
Mon, 8 Jan 2024 21:30:24 +0000 (13:30 -0800)
Some edp panel requires T10 (Delay from end of valid video data transmitted
by the Source device to power-off) less than 500ms. Using autosuspend with
delay set as 1000 violates this requirement.

Use put_sync_suspend in unprepare to meet the spec. For other cases (such
as getting EDID), it still uses autosuspend.

Suggested-by: Douglas Anderson <dianders@chromium.org>
Fixes: 3235b0f20a0a ("drm/panel: panel-simple: Use runtime pm to avoid excessive unprepare / prepare")
Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Signed-off-by: Douglas Anderson <dianders@chromium.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20231220221418.2610185-1-hsinyi@chromium.org
drivers/gpu/drm/panel/panel-edp.c

index cd05c76..7d556b1 100644 (file)
@@ -429,8 +429,7 @@ static int panel_edp_unprepare(struct drm_panel *panel)
        if (!p->prepared)
                return 0;
 
-       pm_runtime_mark_last_busy(panel->dev);
-       ret = pm_runtime_put_autosuspend(panel->dev);
+       ret = pm_runtime_put_sync_suspend(panel->dev);
        if (ret < 0)
                return ret;
        p->prepared = false;