Commit 109406c9 authored by Jani Nikula's avatar Jani Nikula

drm/i915/mpllb: use I915_STATE_WARN() for state mismatch warnings

The pipe_config_mismatch() function is primarily for logging comparison
results.
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/167e54e13a9a41c944910a274e79cbfd39d963b1.1655372759.git.jani.nikula@intel.com
parent f0978e92
...@@ -6601,14 +6601,12 @@ verify_mpllb_state(struct intel_atomic_state *state, ...@@ -6601,14 +6601,12 @@ verify_mpllb_state(struct intel_atomic_state *state,
encoder = intel_get_crtc_new_encoder(state, new_crtc_state); encoder = intel_get_crtc_new_encoder(state, new_crtc_state);
intel_mpllb_readout_hw_state(encoder, &mpllb_hw_state); intel_mpllb_readout_hw_state(encoder, &mpllb_hw_state);
#define MPLLB_CHECK(name) do { \ #define MPLLB_CHECK(__name) \
if (mpllb_sw_state->name != mpllb_hw_state.name) { \ I915_STATE_WARN(mpllb_sw_state->__name != mpllb_hw_state.__name, \
pipe_config_mismatch(false, crtc, "MPLLB:" __stringify(name), \ "[CRTC:%d:%s] mismatch in MPLLB: %s (expected 0x%08x, found 0x%08x)", \
"(expected 0x%08x, found 0x%08x)", \ crtc->base.base.id, crtc->base.name, \
mpllb_sw_state->name, \ __stringify(__name), \
mpllb_hw_state.name); \ mpllb_sw_state->__name, mpllb_hw_state.__name)
} \
} while (0)
MPLLB_CHECK(mpllb_cp); MPLLB_CHECK(mpllb_cp);
MPLLB_CHECK(mpllb_div); MPLLB_CHECK(mpllb_div);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment