Attachment 134524 Details for Bug 100553 – drm/i915/bios: parse DDI ports also for CHV for HDMI DDC pin and DP AUX channel

[patch] drm/i915/bios: parse DDI ports also for CHV for HDMI DDC pin and DP AUX channel

0001-drm-i915-bios-parse-DDI-ports-also-for-CHV-for-HDMI-.patch (text/plain), 1.57 KB, created by Jani Nikula on 2017-09-28 08:06:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jani Nikula

Created: 2017-09-28 08:06:56 UTC

Size: 1.57 KB

patch

obsolete

>From d0815082cb98487618429b62414854137049b888 Mon Sep 17 00:00:00 2001
>From: Jani Nikula <jani.nikula@intel.com>
>Date: Thu, 28 Sep 2017 10:55:59 +0300
>Subject: [PATCH] drm/i915/bios: parse DDI ports also for CHV for HDMI DDC pin
> and DP AUX channel
>Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
>Cc: Jani Nikula <jani.nikula@intel.com>
>
>While technically CHV isn't DDI, we do look at the VBT based DDI port
>info for HDMI DDC pin and DP AUX channel. (We call these "alternate",
>but they're really just something that aren't platform defaults.)
>
>In commit e4ab73a13291 ("drm/i915: Respect alternate_ddc_pin for all DDI
>ports") Ville writes, "IIRC there may be CHV system that might actually
>need this."
>
>I'm not sure why there couldn't be even more platforms that need this,
>but start conservative, and parse the info for CHV in addition to DDI.
>
>Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100553
>Reported-by: Marek Wilczewski <mw@3cte.pl>
>Cc: stable@vger.kernel.org
>Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>---
> drivers/gpu/drm/i915/intel_bios.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c
>index 3747d8df0175..6b8d396ab605 100644
>--- a/drivers/gpu/drm/i915/intel_bios.c
>+++ b/drivers/gpu/drm/i915/intel_bios.c
>@@ -1247,7 +1247,7 @@ static void parse_ddi_ports(struct drm_i915_private *dev_priv,
> {
> 	enum port port;
> 
>-	if (!HAS_DDI(dev_priv))
>+	if (!HAS_DDI(dev_priv) && !IS_CHERRYVIEW(dev_priv))
> 		return;
> 
> 	if (!dev_priv->vbt.child_dev_num)
>-- 
>2.11.0
>

Actions: View | Splinter Review

Attachments on bug 100553: 130667 | 130895 | 134524

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.