[RFC PATCH 0/5] pm_pme support on display hotplug

Vinod Govindapillai <[email protected]>
Newsgroups org.freedesktop.lists.intel-xe,org.freedesktop.lists.intel-gfx
Organization Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs Bertel Jungin Aukio 5, 02600 Espoo
Message-ID <[email protected]>
Initial RFC patches for the pm_pme support for display hotplugs
to see if these proposed changes are in the right direction. This
is not complete yet. I am still figuring out the need to keeping
HPD interrupts enabled for PME generation in suspended state

TODO list:

1. Restrict this to specific display versions or rely on the 
   pci_dev_run_wake()
2. How to check display hotplug generates PM_PME (Bspec 68857, 68867)
 	2.1 HPD -> PME generation work in D3. But it requires hpd
            interrupts and path enabled and unmasked as per the bspec.
            I am still figuring this out.
3. So far I have not been able to test this properly with HPDs. Some
   IGT tests worked successfully which do not involve HPDs. But 
   main thing to verify is that HPD can generate PME in the suspended
   state

Vinod Govindapillai (5):
  drm/xe/pm: add xe_pm_pme_supported() helper
  drm/i915/display: add pme_capable() to the parent interface
  drm/i915/xe: plug the pme_capability query for xe
  drm/i915/display: add intel_hpd_polling_enabled()
  drm/i915/display: skip HPD polling if PME is supported

 .../drm/i915/display/intel_display_driver.c   | 21 ++++++++++++++++---
 .../gpu/drm/i915/display/intel_display_rpm.c  |  7 +++++++
 .../gpu/drm/i915/display/intel_display_rpm.h  |  1 +
 drivers/gpu/drm/i915/display/intel_hotplug.c  | 11 ++++++++++
 drivers/gpu/drm/i915/display/intel_hotplug.h  |  1 +
 drivers/gpu/drm/xe/display/xe_display_rpm.c   |  6 ++++++
 drivers/gpu/drm/xe/xe_pm.c                    | 21 +++++++++++++++++++
 drivers/gpu/drm/xe/xe_pm.h                    |  1 +
 include/drm/intel/display_parent_interface.h  |  1 +
 9 files changed, 67 insertions(+), 3 deletions(-)

-- 
2.43.0
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.