mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-03-27 18:06:53 +08:00
drm/i915: Update connector atomic state before crtc sanitize-disabling
During HW state readout/sanitization an up-to-date connector atomic state will be required by a follow-up patch, which can disable CRTCs with an encoder (and calling the correct encoder hooks happens via the connector atomic state encoder pointer). So update the connector state already before the CRTC sanitize/disable step. For now this doesn't make a difference, since intel_modeset_update_connector_atomic_state() will update/enable the atomic state only for connectors that have an enabled encoder/CRTC. Such CRTCs/encoders will not be affected by intel_sanitize_crtc(). v2: Add comment about why the connector state needs to be up-to-date. Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Signed-off-by: Imre Deak <imre.deak@intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20230510103131.1618266-5-imre.deak@intel.com
This commit is contained in:
@@ -701,6 +701,12 @@ void intel_modeset_setup_hw_state(struct drm_i915_private *i915,
|
||||
for_each_intel_encoder(&i915->drm, encoder)
|
||||
intel_sanitize_encoder(encoder);
|
||||
|
||||
/*
|
||||
* Sanitizing CRTCs needs their connector atomic state to be
|
||||
* up-to-date, so ensure that already here.
|
||||
*/
|
||||
intel_modeset_update_connector_atomic_state(i915);
|
||||
|
||||
for_each_intel_crtc(&i915->drm, crtc) {
|
||||
struct intel_crtc_state *crtc_state =
|
||||
to_intel_crtc_state(crtc->base.state);
|
||||
@@ -709,8 +715,6 @@ void intel_modeset_setup_hw_state(struct drm_i915_private *i915,
|
||||
intel_crtc_state_dump(crtc_state, NULL, "setup_hw_state");
|
||||
}
|
||||
|
||||
intel_modeset_update_connector_atomic_state(i915);
|
||||
|
||||
intel_dpll_sanitize_state(i915);
|
||||
|
||||
intel_wm_get_hw_state(i915);
|
||||
|
||||
Reference in New Issue
Block a user