drm/i915: Don't preserve dpll_hw_state for slave crtc in Bigjoiner
authorStanislav Lisovskiy <stanislav.lisovskiy@intel.com>
Wed, 28 Jun 2023 14:10:17 +0000 (17:10 +0300)
committerStanislav Lisovskiy <stanislav.lisovskiy@intel.com>
Tue, 4 Jul 2023 13:38:40 +0000 (16:38 +0300)
If we are using Bigjoiner dpll_hw_state is supposed to be exactly
same as for master crtc, so no need to save it's state for slave crtc.

Signed-off-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>
Fixes: 0ff0e219d9b8 ("drm/i915: Compute clocks earlier")
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230628141017.18937-1-stanislav.lisovskiy@intel.com
drivers/gpu/drm/i915/display/intel_display.c

index eed0195..797c284 100644 (file)
@@ -4568,7 +4568,6 @@ copy_bigjoiner_crtc_state_modeset(struct intel_atomic_state *state,
        saved_state->uapi = slave_crtc_state->uapi;
        saved_state->scaler_state = slave_crtc_state->scaler_state;
        saved_state->shared_dpll = slave_crtc_state->shared_dpll;
-       saved_state->dpll_hw_state = slave_crtc_state->dpll_hw_state;
        saved_state->crc_enabled = slave_crtc_state->crc_enabled;
 
        intel_crtc_free_hw_state(slave_crtc_state);