]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commit
drm/i915: Don't mark PCH underrun reporting as disabled for transcoder B/C on LPT-H
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 5 Aug 2016 17:00:17 +0000 (20:00 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 5 Aug 2016 17:00:17 +0000 (20:00 +0300)
commita168f5b3f1797d6704d2edb36a3674e663154f80
tree6fc69962bd37cc2e5be90097bd84b4d7b85715d7
parent09fa8bb9094569941661ed2a1dc0ff4bd37a0d7d
drm/i915: Don't mark PCH underrun reporting as disabled for transcoder B/C on LPT-H

Marking PCH transcoder FIFO underrun reporting as disabled for
transcoder B/C on LPT-H will block us from enabling the south error
interrupt. So let's only mark transcoder A underrun reporting as
disabled initially.

This is a little tricky to hit since you need a machine with LPT-H, and
the BIOS must enable either pipe B or C at boot. Then i915 would mark
the "transcoder B/C" underrun reporting as disabled and never enable it
again, meaning south interrupts would never get enabled either. The only
other interrupt in there is actually the poison interrupt which, if we
could ever trigger it, would just result in a little error in dmesg.

Here's the resulting change in SDEIMR on my HSW when I boot it with
multiple displays attached:
- (0x000c4004): 0xf115ffff
+ (0x000c4004): 0xf114ffff

My previous attempt [1] tried to fix this a little differently, but
Daniel requested I do this instead.

[1] https://lists.freedesktop.org/archives/intel-gfx/2015-November/081420.html

Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1470416417-15021-1-git-send-email-ville.syrjala@linux.intel.com
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c