Hot plug events not detected in i965

Bug #1196556 reported by Alex Fiestas
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Low
Unassigned
Raring
Won't Fix
Low
Unassigned
Saucy
Won't Fix
Low
Unassigned

Bug Description

Hotplugs events are not detected in Intel i965 with the current kernel. The bug has been fixed upstream:
https://bugs.freedesktop.org/show_bug.cgi?id=58405

Revision history for this message
Brad Figg (brad-figg) wrote : Missing required logs.

This bug is missing log files that will aid in diagnosing the problem. From a terminal window please run:

apport-collect 1196556

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu):
status: New → Incomplete
Revision history for this message
Alex Fiestas (afiestas) wrote :

The bug has been diagnosed and fixed upstream, there is no need to provide additional info (all info you may need is in the bug report).

Changed in linux (Ubuntu):
importance: Undecided → Low
status: Incomplete → Triaged
Changed in linux (Ubuntu Raring):
status: New → Triaged
importance: Undecided → Low
tags: added: raring saucy
tags: added: kernel-da-key
Revision history for this message
Joseph Salisbury (jsalisbury) wrote :

An SRU to Raring will be needed if the commit that hits Linus' tree is not Cc'd to stable.

Revision history for this message
Joseph Salisbury (jsalisbury) wrote :

The upstream fix can be tested with the following kernel:
http://kernel.ubuntu.com/~kernel-ppa/mainline/drm-intel-nightly/2013-07-01-saucy/

Revision history for this message
Harald Sitter (apachelogger) wrote : Fwd: Patch "drm/i915: Fix up sdvo hpd pins for i965g/gm" has been added to the 3.10-stable tree
Download full text (5.1 KiB)

---------- Forwarded message ----------
From: <email address hidden>
To: <email address hidden>, <email address hidden>,
<email address hidden>, <email address hidden>,
<email address hidden>
Cc: <email address hidden>, <email address hidden>
Date: Tue, 23 Jul 2013 09:56:48 -0700
Subject: Patch "drm/i915: Fix up sdvo hpd pins for i965g/gm" has been
added to the 3.10-stable tree

This is a note to let you know that I've just added the patch titled

    drm/i915: Fix up sdvo hpd pins for i965g/gm

to the 3.10-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     drm-i915-fix-up-sdvo-hpd-pins-for-i965g-gm.patch
and it can be found in the queue-3.10 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <email address hidden> know about it.

From 4f7fd7095d85cd31c86cb9ba87bc301319630ccc Mon Sep 17 00:00:00 2001
From: Daniel Vetter <email address hidden>
Date: Mon, 24 Jun 2013 21:33:28 +0200
Subject: drm/i915: Fix up sdvo hpd pins for i965g/gm

From: Daniel Vetter <email address hidden>

commit 4f7fd7095d85cd31c86cb9ba87bc301319630ccc upstream.

Bspec seems to be full of lies, at least it disagress with reality:
Two systems corrobated that SDVO hpd bits are the same as on gen3.

v2: Update comment a bit.

Tested-by: Chris Wilson <email address hidden>
Cc: Arthur Ranyan <email address hidden>
Reported-and-tested-by: Alex Fiestas <email address hidden>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=58405
Acked-by: Chris Wilson <email address hidden>
Signed-off-by: Daniel Vetter <email address hidden>
Signed-off-by: Greg Kroah-Hartman <email address hidden>

---
 drivers/gpu/drm/i915/i915_irq.c | 13 ++-----------
 drivers/gpu/drm/i915/i915_reg.h | 13 ++++++-------
 2 files changed, 8 insertions(+), 18 deletions(-)

--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -70,15 +70,6 @@ static const u32 hpd_status_gen4[] = {
        [HPD_PORT_D] = PORTD_HOTPLUG_INT_STATUS
 };

-static const u32 hpd_status_i965[] = {
- [HPD_CRT] = CRT_HOTPLUG_INT_STATUS,
- [HPD_SDVO_B] = SDVOB_HOTPLUG_INT_STATUS_I965,
- [HPD_SDVO_C] = SDVOC_HOTPLUG_INT_STATUS_I965,
- [HPD_PORT_B] = PORTB_HOTPLUG_INT_STATUS,
- [HPD_PORT_C] = PORTC_HOTPLUG_INT_STATUS,
- [HPD_PORT_D] = PORTD_HOTPLUG_INT_STATUS
-};
-
 static const u32 hpd_status_i915[] = { /* i915 and valleyview are the same */
        [HPD_CRT] = CRT_HOTPLUG_INT_STATUS,
        [HPD_SDVO_B] = SDVOB_HOTPLUG_INT_STATUS_I915,
@@ -2952,13 +2943,13 @@ static irqreturn_t i965_irq_handler(int
                        u32 hotplug_status = I915_READ(PORT_HOTPLUG_STAT);
                        u32 hotplug_trigger = hotplug_status & (IS_G4X(dev) ?

HOTPLUG_INT_STATUS_G4X :
-
HOTPLUG_INT_STATUS_I965);
+
HOTPLUG_INT_STATUS_I915);

                        DRM_DEBUG_DRIVER("hotplug event received, stat
0x%08x\n",
                                  hotplug_status);
                        if (hotplug_trigger) {
                                if (hotplug_irq_storm_detect(dev,
hotplug...

Read more...

tags: added: kernel-fixed-upstream
Revision history for this message
Joseph Salisbury (jsalisbury) wrote :

This fix is upstream as of v3.11-rc2:

git describe --contains 4f7fd70
v3.11-rc2~17^2~2^2~35

Can you test the latest Saucy kernel and confirm this bug is fixed, since it has been rebased to upstream 3.11? If Saucy is fixed, please change the status for Saucy to "Fix Released".

Raring will receive this fix through the normal stable update process, since it was cc'd to stable.

Changed in linux (Ubuntu Raring):
status: Triaged → Incomplete
Changed in linux (Ubuntu Saucy):
status: Triaged → Incomplete
Revision history for this message
Joseph Salisbury (jsalisbury) wrote : Closing unsupported series nomination.

This bug was nominated against a series that is no longer supported, ie saucy. The bug task representing the saucy nomination is being closed as Won't Fix.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu Saucy):
status: Incomplete → Won't Fix
Revision history for this message
Joseph Salisbury (jsalisbury) wrote :

This bug was nominated against a series that is no longer supported, ie raring. The bug task representing the raring nomination is being closed as Won't Fix.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu Raring):
status: Incomplete → Won't Fix
Revision history for this message
Timo Aaltonen (tjaalton) wrote :

closing as fixed since a few years

Changed in linux (Ubuntu):
status: Incomplete → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.