drm/i915/bios: Introduce has_ddi_port_info()
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 17 Dec 2021 15:53:58 +0000 (17:53 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 19 Jan 2022 20:31:26 +0000 (22:31 +0200)
Pull the "do we want to use i915->vbt.ports[]?" check into
a central place.

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

index 262406c..e9b6b6e 100644 (file)
@@ -2053,6 +2053,11 @@ static void parse_ddi_port(struct drm_i915_private *i915,
        i915->vbt.ports[port] = devdata;
 }
 
+static bool has_ddi_port_info(struct drm_i915_private *i915)
+{
+       return HAS_DDI(i915);
+}
+
 static void parse_ddi_ports(struct drm_i915_private *i915)
 {
        struct intel_bios_encoder_data *devdata;
@@ -2653,7 +2658,7 @@ bool intel_bios_is_port_present(struct drm_i915_private *i915, enum port port)
                [PORT_F] = { DVO_PORT_DPF, DVO_PORT_HDMIF, },
        };
 
-       if (HAS_DDI(i915))
+       if (has_ddi_port_info(i915))
                return i915->vbt.ports[port];
 
        /* FIXME maybe deal with port A as well? */
@@ -2693,7 +2698,7 @@ bool intel_bios_is_port_edp(struct drm_i915_private *i915, enum port port)
                [PORT_F] = DVO_PORT_DPF,
        };
 
-       if (HAS_DDI(i915)) {
+       if (has_ddi_port_info(i915)) {
                const struct intel_bios_encoder_data *devdata;
 
                devdata = intel_bios_encoder_data_lookup(i915, port);
@@ -2748,7 +2753,7 @@ bool intel_bios_is_port_dp_dual_mode(struct drm_i915_private *i915,
        };
        const struct intel_bios_encoder_data *devdata;
 
-       if (HAS_DDI(i915)) {
+       if (has_ddi_port_info(i915)) {
                const struct intel_bios_encoder_data *devdata;
 
                devdata = intel_bios_encoder_data_lookup(i915, port);