drm/i915: Extract intel_dp_has_dsc()

Extract a helper to check whether the source+sink combo
supports DSC. That basic check is needed both during mode
validation and compute config. We'll also need to add extra
checks to both places, so having a single place for it is nicer.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240517145356.26103-3-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>

diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index 09a06fb232f3a819232be21d9bd0e0c7dc615690..a1792da3b6c725aca7a76706c3844ddb26014139 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -1171,6 +1171,19 @@ bool intel_dp_need_joiner(struct intel_dp *intel_dp,
               connector->force_bigjoiner_enable;
 }
 
+static bool intel_dp_has_dsc(const struct intel_connector *connector)
+{
+       struct drm_i915_private *i915 = to_i915(connector->base.dev);
+
+       if (!HAS_DSC(i915))
+               return false;
+
+       if (!drm_dp_sink_supports_dsc(connector->dp.dsc_dpcd))
+               return false;
+
+       return true;
+}
+
 static enum drm_mode_status
 intel_dp_mode_valid(struct drm_connector *_connector,
                    struct drm_display_mode *mode)
@@ -1225,8 +1238,7 @@ intel_dp_mode_valid(struct drm_connector *_connector,
        mode_rate = intel_dp_link_required(target_clock,
                                           intel_dp_mode_min_output_bpp(connector, mode));
 
-       if (HAS_DSC(dev_priv) &&
-           drm_dp_sink_supports_dsc(connector->dp.dsc_dpcd)) {
+       if (intel_dp_has_dsc(connector)) {
                enum intel_output_format sink_format, output_format;
                int pipe_bpp;