When going to d3cold we surely don't need to flush commits,
nor handle fbdev. So, clean this up a bit.

Also move the opregion call to make the function more symmetric.

Cc: Imre Deak <imre.d...@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.v...@intel.com>
---
 drivers/gpu/drm/xe/display/xe_display.c | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_display.c 
b/drivers/gpu/drm/xe/display/xe_display.c
index daf9b9baa88a..3fd3e74f1310 100644
--- a/drivers/gpu/drm/xe/display/xe_display.c
+++ b/drivers/gpu/drm/xe/display/xe_display.c
@@ -310,9 +310,6 @@ static void xe_display_to_d3cold(struct xe_device *xe)
         * properly.
         */
        intel_power_domains_disable(xe);
-       intel_fbdev_set_suspend(&xe->drm, FBINFO_STATE_SUSPENDED, true);
-
-       xe_display_flush_cleanup_work(xe);
 
        intel_dp_mst_suspend(xe);
 
@@ -329,18 +326,15 @@ static void xe_display_from_d3cold(struct xe_device *xe)
 
        intel_dmc_resume(xe);
 
-       drm_mode_config_reset(&xe->drm);
-
        intel_display_driver_init_hw(xe);
+
+       intel_opregion_resume(display);
+
        intel_hpd_init(xe);
 
        /* MST sideband requires HPD interrupts enabled */
        intel_dp_mst_resume(xe);
 
-       intel_opregion_resume(display);
-
-       intel_fbdev_set_suspend(&xe->drm, FBINFO_STATE_RUNNING, false);
-
        intel_power_domains_enable(xe);
 }
 
-- 
2.46.0

Reply via email to