Re: [PATCH] drm/xe: Fix merge fails related to display runtime PM

[Daniel Vetter <daniel.vetter@ffwll.ch>]

On Mon, Sep 02, 2024 at 01:20:02PM +0200, Maarten Lankhorst wrote:
> The most recent merge commits introduced some fails to drm/drm-next,
> I've noticed these when looking at the xe patches.
> 
> Solve it!
> 
> Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>

I dropped the first hunk, since that was broken in drm-tip too. So
separate bugfix that should go in with proper review and Fixes: tag and
all that imo.

Otherwise with this drm-next now matches what was in drm-tip.

Thanks for the quick fix!

Cheers, Sima
> ---
>  drivers/gpu/drm/xe/display/xe_display.c | 2 --
>  drivers/gpu/drm/xe/xe_pm.c              | 4 +---
>  2 files changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
> index 78a884ddd499a..710b1e2170c13 100644
> --- a/drivers/gpu/drm/xe/display/xe_display.c
> +++ b/drivers/gpu/drm/xe/display/xe_display.c
> @@ -341,8 +341,6 @@ void xe_display_pm_suspend(struct xe_device *xe, bool runtime)
>  
>  	xe_display_flush_cleanup_work(xe);
>  
> -	xe_display_flush_cleanup_work(xe);
> -
>  	intel_dp_mst_suspend(xe);
>  
>  	intel_hpd_cancel_work(xe);
> diff --git a/drivers/gpu/drm/xe/xe_pm.c b/drivers/gpu/drm/xe/xe_pm.c
> index 2e2accd76fb2b..39e0a8774a104 100644
> --- a/drivers/gpu/drm/xe/xe_pm.c
> +++ b/drivers/gpu/drm/xe/xe_pm.c
> @@ -392,8 +392,6 @@ int xe_pm_runtime_suspend(struct xe_device *xe)
>  	xe_display_pm_runtime_suspend(xe);
>  
>  	if (xe->d3cold.allowed) {
> -		xe_display_pm_suspend(xe, true);
> -
>  		err = xe_bo_evict_all(xe);
>  		if (err)
>  			goto out;
> @@ -411,7 +409,7 @@ int xe_pm_runtime_suspend(struct xe_device *xe)
>  		xe_display_pm_suspend_late(xe);
>  out:
>  	if (err)
> -		xe_display_pm_resume(xe, true);
> +		xe_display_pm_runtime_resume(xe);
>  	xe_rpm_lockmap_release(xe);
>  	xe_pm_write_callback_task(xe, NULL);
>  	return err;
> -- 
> 2.45.2
> 

-- 
Simona Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch