drm/i915: Don't bail early from intel_dp_initial_fastset_check()
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 22 Sep 2022 19:13:14 +0000 (22:13 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 26 Sep 2022 17:15:53 +0000 (20:15 +0300)
commitfea1beb60db55abc05b2def917bff8e40825d3ed
tree94e13d0185fdb1875922c52228a2d3dea18efe1f
parenteddb4afcb6c533d3f75f5f1a77e292fece27570e
drm/i915: Don't bail early from intel_dp_initial_fastset_check()

Do all the checks in intel_dp_initial_fastset_check() instead
of bailing out on the first condition that triggers.

This makes for better debug logs since we see all the reasons
why the full modeset computation is forced.

Also avoid the risk of someone accidentally adding a check
later in the function that would require connectors_changed=true
(ie. no fastset at all), but an earlier check may have already
bailed out with just mode_changed=true (ie. fastset is still
possible).

Pimp the debugs with the encoder id+name while at it.

v2: Call the return variable 'fastset' to convey its meaning

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220922191314.4252-1-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_dp.c