Re: [PATCH 1/3] drm/xe: Fix build by setting the correct parameter to intel_dmc_suspend()

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Wed, Sep 11, 2024 at 01:02:16PM -0700, José Roberto de Souza wrote:
> intel_dmc_suspend() was a intel_display no a xe_device.
> 
> Fixes: 501d799a47e2 ("drm/xe: Wire up device shutdown handler")
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
> ---
>  drivers/gpu/drm/xe/display/xe_display.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
> index 3551a9c8eb755..e65e7d70f9890 100644
> --- a/drivers/gpu/drm/xe/display/xe_display.c
> +++ b/drivers/gpu/drm/xe/display/xe_display.c
> @@ -377,7 +377,7 @@ void xe_display_pm_shutdown(struct xe_device *xe)
>  
>  	intel_opregion_suspend(display, PCI_D3cold);
>  
> -	intel_dmc_suspend(xe);
> +	intel_dmc_suspend(display);

It's a drm-build conflict that needs to be solved at drm-tip rebuild...

I'm looking into it right now... Thanks for the heads up

>  }
>  
>  void xe_display_pm_runtime_suspend(struct xe_device *xe)
> -- 
> 2.46.0
>