Attachment 125085 Details for Bug 96675 – [PATCH] drm/i915: Treat eDP as always connected, again

[patch] [PATCH] drm/i915: Treat eDP as always connected, again

0001-drm-i915-Treat-eDP-as-always-connected-again.patch (text/plain), 1.79 KB, created by Ville Syrjala on 2016-07-15 10:16:18 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ville Syrjala

Created: 2016-07-15 10:16:18 UTC

Size: 1.79 KB

patch

obsolete

>From a1e06b3fecbb857ecaf290148cec8a9042b0a497 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Ville=20Syrj=C3=A4l=C3=A4?= <ville.syrjala@linux.intel.com>
>Date: Fri, 15 Jul 2016 13:08:09 +0300
>Subject: [PATCH] drm/i915: Treat eDP as always connected, again
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>eDP should be treated as connected even if doesn't have an EDID. In that
>case we'll use the timings from the VBT. That used to be the case until
>commit f21a21983ef1 ("drm/i915: Splitting intel_dp_detect")
>broke things by considering even eDP disconnected if we fail to get
>an EDID for it.
>
>Fix things up again by treating eDP as always connected.
>
>Cc: Shubhangi Shrivastava <shubhangi.shrivastava@intel.com>
>Cc: Nathan D Ciobanu <nathan.d.ciobanu@intel.com>
>Cc: Sivakumar Thulasimani <sivakumar.thulasimani@intel.com>
>Cc: Ander Conselvan de Oliveira <conselvan2@gmail.com>
>Cc: Larry Finger <larry.finger@lwfinger.net>
>Reported-by: Larry Finger <larry.finger@lwfinger.net>
>Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96675
>Cc: drm-intel-fixes@lists.freedesktop.org
>Fixes: f21a21983ef1 ("drm/i915: Splitting intel_dp_detect")
>Signed-off-by: Ville SyrjÃ¤lÃ¤ <ville.syrjala@linux.intel.com>
>---
> drivers/gpu/drm/i915/intel_dp.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
>index 0c5ba3410a1e..21b04c3eda41 100644
>--- a/drivers/gpu/drm/i915/intel_dp.c
>+++ b/drivers/gpu/drm/i915/intel_dp.c
>@@ -4336,7 +4336,7 @@ intel_dp_detect(struct drm_connector *connector, bool force)
> 
> 	intel_dp->detect_done = false;
> 
>-	if (intel_connector->detect_edid)
>+	if (is_edp(intel_dp) || intel_connector->detect_edid)
> 		return connector_status_connected;
> 	else
> 		return connector_status_disconnected;
>-- 
>2.7.4
>

Actions: View | Splinter Review

Attachments on bug 96675: 124717 | 124718 | 125079 | 125081 | 125085

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.