drm/i915: Simplify DP vs. eDP detection
Reduce the eDP detection to just checking if it's port A, or if the VBT tells us that the port is eDP for the other ports. Suggested-by: Jesse Barnes <jbarnes@virtuousgeek.org> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
This commit is contained in:
parent
5d8a77529b
commit
3b32a35b31
@ -3337,6 +3337,9 @@ bool intel_dp_is_edp(struct drm_device *dev, enum port port)
|
|||||||
[PORT_D] = PORT_IDPD,
|
[PORT_D] = PORT_IDPD,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
if (port == PORT_A)
|
||||||
|
return true;
|
||||||
|
|
||||||
if (!dev_priv->vbt.child_dev_num)
|
if (!dev_priv->vbt.child_dev_num)
|
||||||
return false;
|
return false;
|
||||||
|
|
||||||
@ -3621,27 +3624,10 @@ intel_dp_init_connector(struct intel_digital_port *intel_dig_port,
|
|||||||
intel_dp->DP = I915_READ(intel_dp->output_reg);
|
intel_dp->DP = I915_READ(intel_dp->output_reg);
|
||||||
intel_dp->attached_connector = intel_connector;
|
intel_dp->attached_connector = intel_connector;
|
||||||
|
|
||||||
type = DRM_MODE_CONNECTOR_DisplayPort;
|
if (intel_dp_is_edp(dev, port))
|
||||||
/*
|
|
||||||
* FIXME : We need to initialize built-in panels before external panels.
|
|
||||||
* For X0, DP_C is fixed as eDP. Revisit this as part of VLV eDP cleanup
|
|
||||||
*/
|
|
||||||
switch (port) {
|
|
||||||
case PORT_A:
|
|
||||||
type = DRM_MODE_CONNECTOR_eDP;
|
type = DRM_MODE_CONNECTOR_eDP;
|
||||||
break;
|
else
|
||||||
case PORT_B:
|
type = DRM_MODE_CONNECTOR_DisplayPort;
|
||||||
case PORT_C:
|
|
||||||
if (IS_VALLEYVIEW(dev) && intel_dp_is_edp(dev, port))
|
|
||||||
type = DRM_MODE_CONNECTOR_eDP;
|
|
||||||
break;
|
|
||||||
case PORT_D:
|
|
||||||
if (HAS_PCH_SPLIT(dev) && intel_dp_is_edp(dev, port))
|
|
||||||
type = DRM_MODE_CONNECTOR_eDP;
|
|
||||||
break;
|
|
||||||
default: /* silence GCC warning */
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* For eDP we always set the encoder type to INTEL_OUTPUT_EDP, but
|
* For eDP we always set the encoder type to INTEL_OUTPUT_EDP, but
|
||||||
|
Loading…
Reference in New Issue
Block a user