-----Original Message-----
From: Intel-gfx <[email protected]> On Behalf Of Rodrigo 
Vivi
Sent: Tuesday, September 24, 2024 1:36 PM
To: [email protected]; [email protected]
Cc: Deak, Imre <[email protected]>; Vivi, Rodrigo <[email protected]>
Subject: [PATCH 25/31] drm/xe/display: Move hpd_poll calls to later runtime 
stages
> 
> Align placement of HDP poll calls with i915.
> 
> Signed-off-by: Rodrigo Vivi <[email protected]>

LGTM.

Reviewed-by: Jonathan Cavitt <[email protected]>
-Jonathan Cavitt

> ---
>  drivers/gpu/drm/xe/display/xe_display.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/display/xe_display.c 
> b/drivers/gpu/drm/xe/display/xe_display.c
> index 4d1dd5294b89..c758874a9059 100644
> --- a/drivers/gpu/drm/xe/display/xe_display.c
> +++ b/drivers/gpu/drm/xe/display/xe_display.c
> @@ -437,8 +437,6 @@ void xe_display_pm_runtime_suspend(struct xe_device *xe)
>  
>       if (xe->d3cold.allowed)
>               xe_display_to_d3cold(xe);
> -
> -     intel_hpd_poll_enable(xe);
>  }
>  
>  void xe_display_pm_runtime_suspend_late(struct xe_device *xe)
> @@ -448,6 +446,8 @@ void xe_display_pm_runtime_suspend_late(struct xe_device 
> *xe)
>  
>       if (xe->d3cold.allowed)
>               intel_display_power_suspend_late(xe, false);
> +
> +     intel_hpd_poll_enable(xe);
>  }
>  
>  void xe_display_pm_runtime_resume_early(struct xe_device *xe)
> @@ -464,12 +464,11 @@ void xe_display_pm_runtime_resume(struct xe_device *xe)
>       if (!xe->info.probe_display)
>               return;
>  
> -     intel_hpd_poll_disable(xe);
> -
>       if (xe->d3cold.allowed)
>               xe_display_from_d3cold(xe);
>  
>       intel_hpd_init(xe);
> +     intel_hpd_poll_disable(xe);
>  }
>  
>  static void display_device_remove(struct drm_device *dev, void *arg)
> -- 
> 2.46.0
> 
> 

Reply via email to