[Bug 89008] [BYT] No HDMI hotplugging

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue Mar 31 11:53:24 PDT 2015


https://bugs.freedesktop.org/show_bug.cgi?id=89008

Jesse Barnes <jbarnes at virtuousgeek.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #17 from Jesse Barnes <jbarnes at virtuousgeek.org> ---
There's probably a nicer way to do this, but it seems like preserving HPD
behavior by default is the right thing to do.  Having a runtime way to disable
it would be nice though, or a config option, so if your userspace can handle
the polling we could allow the DISP2D power well to shut down.

Egbert, can you give it a try?

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index 92f8300..1ffeef2 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -2441,6 +2441,7 @@ struct i915_params {
     bool mmio_debug;
     bool verbose_state_checks;
     bool nuclear_pageflip;
+    bool keep_hpd_enabled;
 };
 extern struct i915_params i915 __read_mostly;

diff --git a/drivers/gpu/drm/i915/i915_params.c
b/drivers/gpu/drm/i915/i915_params.c
index 44f2262..48acf4b 100644
--- a/drivers/gpu/drm/i915/i915_params.c
+++ b/drivers/gpu/drm/i915/i915_params.c
@@ -53,6 +53,7 @@ struct i915_params i915 __read_mostly = {
     .mmio_debug = 0,
     .verbose_state_checks = 1,
     .nuclear_pageflip = 0,
+    .keep_hpd_enabled = 1,
 };

 module_param_named(modeset, i915.modeset, int, 0400);
@@ -183,3 +184,6 @@ MODULE_PARM_DESC(verbose_state_checks,
 module_param_named_unsafe(nuclear_pageflip, i915.nuclear_pageflip, bool,
0600);
 MODULE_PARM_DESC(nuclear_pageflip,
          "Force atomic modeset functionality; only planes work for now
(default: false).");
+
+module_param_named(keep_hpd_enabled, i915.keep_hpd_enabled, bool, 0600);
+MODULE_PARM_DESC(keep_hpd_enabled, "Keep hotplug detection enabled (costs some
power) (default: true)");
diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c
b/drivers/gpu/drm/i915/intel_runtime_pm.c
index 6d8e29a..1d45609 100644
--- a/drivers/gpu/drm/i915/intel_runtime_pm.c
+++ b/drivers/gpu/drm/i915/intel_runtime_pm.c
@@ -544,6 +544,9 @@ static void vlv_display_power_well_disable(struct
drm_i915_private *dev_priv,
 {
     WARN_ON_ONCE(power_well->data != PUNIT_POWER_WELL_DISP2D);

+    if (i915.keep_hpd_enabled)
+        return;
+
     spin_lock_irq(&dev_priv->irq_lock);
     valleyview_disable_display_irqs(dev_priv);
     spin_unlock_irq(&dev_priv->irq_lock);

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20150331/f3c1a3dc/attachment-0001.html>


More information about the intel-gfx-bugs mailing list