[Bug 59841] [IVB cpu eDP] panel is broken due to 657445fe86 (eDP bpp clamping)

[bugzilla-daemon at bugzilla.kernel.org]

https://bugzilla.kernel.org/show_bug.cgi?id=59841

--- Comment #81 from jkp <jkp at iki.fi> ---
Proposed patch (to be applied in addition to the previous one) for some debug
output and disabling the use of the EDP data when info says EDP is not
available. Tested to work on the Asus TX300.

Hmm, on second thought, more correct would probably be just:

if (!dev_priv->vbt.edp_support) { ...

instead of including SUPPORTS_EDT as in the patch.

--- linux-3.11-rc7.virgin/drivers/gpu/drm/i915/intel_bios.c     2013-08-26
03:43:22.000000000 +0300
+++ linux-3.11-rc7/drivers/gpu/drm/i915/intel_bios.c    2013-09-02
00:55:30.488888674 +0300
@@ -499,6 +499,15 @@
        struct edp_power_seq *edp_pps;
        struct edp_link_params *edp_link_params;

+       DRM_DEBUG_KMS("edp_support: %d\n", dev_priv->vbt.edp_support);
+       DRM_DEBUG_KMS("SUPPORTS_EDP: %d\n", SUPPORTS_EDP(dev_priv->dev));
+       DRM_DEBUG_KMS("BDB panel type: %d \n", panel_type);
+
+       if (!SUPPORTS_EDP(dev_priv->dev) && !dev_priv->vbt.edp_support) {
+               DRM_DEBUG_KMS("No eDP support, not trying to parse BDB_EDP
section.\n");
+               return;
+       }
+
        edp = find_section(bdb, BDB_EDP);
        if (!edp) {
                if (SUPPORTS_EDP(dev_priv->dev) && dev_priv->vbt.edp_support)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.