diff options
author | Imre Deak <imre.deak@intel.com> | 2025-09-03 15:21:52 +0300 |
---|---|---|
committer | Imre Deak <imre.deak@intel.com> | 2025-09-05 14:20:55 +0300 |
commit | 2ce575d6a1be4ef03c804db0f668527af426472c (patch) | |
tree | afee5d7bee929e7460ff9a97b06f63a5f1dd2928 /rust/helpers/security.c | |
parent | d074a40b8828fe29c2bab59d0c4a26c940af7e88 (diff) |
drm/i915/display: Remove power state verification before HW readout
During system resume the display power state verification will print the
"power well x state mismatch (refcount 0/enabled 1)"
error message from the early resume sequence for a power well left
enabled by BIOS. This power well was probably left enabled by BIOS
inadvertently, since BIOS versions on current platforms do not leave any
display output enabled while resuming from an Sx power state, hence the
enabled display power well is unused. In theory however it is possible
that BIOS leaves a display output enabled, in that case the enabled
power well shouldn't be reported as an error.
According to the above, remove the display power state verification from
the early resume phase to avoid incorrectly reporting an enabled power
well without a power reference as an error.
Note: The refcount for any enabled and used power well (i.e. used for an
enabled display output) will be acquired following the early resume
sequence, after the HW state for display outputs (encoder/crtc etc.) is
read out. Any power well enabled but not used (hence not holding a
reference) will be disabled after the HW state readout. The display
power state will be verified afterwards in intel_power_domains_enable().
Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/6012
Reviewed-by: Mika Kahola <mika.kahola@intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
Link: https://lore.kernel.org/r/20250903122152.2526050-1-imre.deak@intel.com
Diffstat (limited to 'rust/helpers/security.c')
0 files changed, 0 insertions, 0 deletions