[Intel-gfx] [alsa-devel] [PATCH 1/7] drm/i915: setup bridge for HDMI LPE audio driver
Pierre-Louis Bossart
pierre-louis.bossart at linux.intel.com
Thu Dec 15 19:04:21 UTC 2016
>>> +static void lpe_audio_irq_unmask(struct irq_data *d) {
>>> + struct drm_i915_private *dev_priv = d->chip_data;
>>> + unsigned long irqflags;
>>> + u32 val = (I915_LPE_PIPE_A_INTERRUPT |
>>> + I915_LPE_PIPE_B_INTERRUPT);
>>
>> PIPE_C missing?
>>
>
> No PIPE_C on vlv
Good catch Daniel.
We initially had PIPE_C everyhwere, thinking that it's harmless on
Baytrail. Ville had a comment that there was no PIPE_C on BYT so we
removed it for Baytrail-only code, here it makes no sense to me. we have
the same problem in lpe_audio_irq_mask
>> Besides the few bikesheds seems all reasonable, with those addressed
>>
>> Acked-by: Daniel Vetter <daniel.vetter at ffwll.ch>
>>
>
> Thanks
>
>> ... for merging through sound tree. Since we're super early in 4.11
a topic
>> branch for me to pull in to avoid sync headaches would be good.
Thanks for the reviews and ack, we'll fix the comments shortly.
More information about the Intel-gfx
mailing list