Re: [RFC PATCH 2/5] drm/i915/display: add pme_capable() to the parent interface

[Jani Nikula <jani.nikula@linux.intel.com>]

On Mon, 17 Aug 2026, Vinod Govindapillai <vinod.govindapillai@intel.com> wrote:
> Add a provision to query the optional pme_capability to the parent
> interface so that it could be called independently from xe or i915
> based on the implementation.

But do we need this? Display has all the information to call
pci_dev_run_wake() directly, without going through the parent interface.

The only difference is this is not being set up for i915.

BR,
Jani.

>
> Assisted-by: GitHub_Copilot:claude-opus-5
> Signed-off-by: Vinod Govindapillai <vinod.govindapillai@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_display_rpm.c | 7 +++++++
>  drivers/gpu/drm/i915/display/intel_display_rpm.h | 1 +
>  include/drm/intel/display_parent_interface.h     | 1 +
>  3 files changed, 9 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_display_rpm.c b/drivers/gpu/drm/i915/display/intel_display_rpm.c
> index 0a331f89b4db..73d85384474e 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_rpm.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_rpm.c
> @@ -46,6 +46,13 @@ bool intel_display_rpm_suspended(struct intel_display *display)
>  	return display->parent->rpm->suspended(display->drm);
>  }
>  
> +bool intel_display_rpm_pme_capable(struct intel_display *display)
> +{
> +	const struct intel_display_rpm_interface *rpm = display->parent->rpm;
> +
> +	return rpm->pme_capable && rpm->pme_capable(display->drm);
> +}
> +
>  void assert_display_rpm_held(struct intel_display *display)
>  {
>  	display->parent->rpm->assert_held(display->drm);
> diff --git a/drivers/gpu/drm/i915/display/intel_display_rpm.h b/drivers/gpu/drm/i915/display/intel_display_rpm.h
> index 6ef48515f84b..2d4071fb9282 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_rpm.h
> +++ b/drivers/gpu/drm/i915/display/intel_display_rpm.h
> @@ -21,6 +21,7 @@ void intel_display_rpm_put(struct intel_display *display, struct ref_tracker *wa
>  
>  /* Only for special cases. */
>  bool intel_display_rpm_suspended(struct intel_display *display);
> +bool intel_display_rpm_pme_capable(struct intel_display *display);
>  
>  void assert_display_rpm_held(struct intel_display *display);
>  void intel_display_rpm_assert_block(struct intel_display *display);
> diff --git a/include/drm/intel/display_parent_interface.h b/include/drm/intel/display_parent_interface.h
> index d7d06dcaa699..563de403b102 100644
> --- a/include/drm/intel/display_parent_interface.h
> +++ b/include/drm/intel/display_parent_interface.h
> @@ -196,6 +196,7 @@ struct intel_display_rpm_interface {
>  	void (*put_unchecked)(const struct drm_device *drm);
>  
>  	bool (*suspended)(const struct drm_device *drm);
> +	bool (*pme_capable)(const struct drm_device *drm); /* Optional */
>  	void (*assert_held)(const struct drm_device *drm);
>  	void (*assert_block)(const struct drm_device *drm);
>  	void (*assert_unblock)(const struct drm_device *drm);

-- 
Jani Nikula, Intel