diff options
author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2017-04-27 19:02:22 +0300 |
---|---|---|
committer | Jani Nikula <jani.nikula@intel.com> | 2017-05-26 11:51:18 +0300 |
commit | 9bd9590997b92fbd79fd028f704f6c584b4439d7 (patch) | |
tree | 68a9d0604c9c74bd2f838f49907ce3e3fcaf577f /samples | |
parent | 12ea39f8daf206eaaf926adf8f162b2f230bfb38 (diff) |
drm/i915: Stop pretending to mask/unmask LPE audio interruptsdrm-intel-fixes-2017-05-29
vlv_display_irq_postinstall() enables the LPE audio interrupts
regardless of whether the LPE audio irq chip has masked/unmasked
them. Also the irqchip masking/unmasking doesn't consider the state
of the display power well or the device, and hence just leads to
dmesg spew when it tries to access the hardware while it's powered
down.
If the current way works, then we don't need to do anything in the
mask/unmask hooks. If it doesn't work, well, then we'd need to properly
track whether the irqchip has masked/unmasked the interrupts when
we enable display interrupts. And the mask/unmask hooks would need
to check whether display interrupts are even enabled before frobbing
with he registers.
So let's just assume the current way works and neuter the mask/unmask
hooks. Also clean up vlv_display_irq_postinstall() a bit and stop
it from trying to unmask/enable the LPE C interrupt on VLV since it
doesn't exist.
Cc: Takashi Iwai <tiwai@suse.de>
Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170427160231.13337-4-ville.syrjala@linux.intel.com
Reviewed-by: Takashi Iwai <tiwai@suse.de>
(cherry picked from commit ebf5f921478b9b55ed4e634b994571dd23a8fca3)
Reference: http://mid.mail-archive.com/874cf6d3-4e45-d4cf-e662-eb972490d2ce@redhat.com
Tested-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Diffstat (limited to 'samples')
0 files changed, 0 insertions, 0 deletions