i915, HDMI/DP audio with multiple monitors

On Thu, 13 Sep 2018 09:25:37 +0200,
Bruno Prémont wrote:
> 
> On Wed, 12 Sep 2018 20:06:43 +0200 Takashi Iwai wrote:
> > On Wed, 12 Sep 2018 19:46:58 +0200,
> > Ville Syrjälä wrote:
> > > 
> > > On Tue, Sep 11, 2018 at 03:50:13PM +0200, Bruno Prémont wrote:  
> > > > Hi,
> > > > 
> > > > I have a system with multiple monitors and would like to send
> > > > notification sounds to the monitor on which corresponding
> > > > window is visible.
> > > > 
> > > > For a workstation and a tiny computer things look different:
> > > > - workstation (Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz):
> > > >  00:02.0 VGA compatible controller [0300]: Intel Corporation Xeon E3-1200 v3/4th Gen Core Processor Integrated Graphics Controller [8086:0412] (rev 06)
> > > >  00:03.0 Audio device [0403]: Intel Corporation Xeon E3-1200 v3/4th Gen Core Processor HD Audio Controller [8086:0c0c] (rev 06)
> > > >  00:1b.0 Audio device [0403]: Intel Corporation 8 Series/C220 Series Chipset High Definition Audio Controller [8086:8c20] (rev 04)
> > > > 
> > > >  Here alsa show me two cards:
> > > >  - HDA Intel PCH (Realtek ALC671)
> > > >  - HDA Intel HDMI (Intel Generic)
> > > > 
> > > >  **** List of PLAYBACK Hardware Devices ****
> > > >  card 0: HDMI [HDA Intel HDMI], device 3: Generic Digital [Generic Digital]
> > > >    Subdevices: 1/1
> > > >    Subdevice #0: subdevice #0  
> > 
> > Is a proper kernel config (CONFIG_SND_HDA_CODEC_HDMI) enabled?
> 
> It was missing and adding it helps a lot.
> Would there be a way to auto-select it when corresponding DRM driver is
> selected?
> 
> Kind of
>   select SND_HDA_CODEC_HDMI if SND_HDA
> or at least mention it in description, maybe as conditional comment is
> done for HDA codecs.

Yeah, a patch like below should work.

--- a/drivers/gpu/drm/i915/Kconfig
+++ b/drivers/gpu/drm/i915/Kconfig
@@ -23,6 +23,7 @@ config DRM_I915
 	select SYNC_FILE
 	select IOSF_MBI
 	select CRC32
+	select SND_HDA_CODEC_HDMI if SND_HDA_INTEL
 	select SND_HDA_I915 if SND_HDA_CORE
 	select CEC_CORE if CEC_NOTIFIER
 	help

But I'm not going to advocate it.  Feel free to cook up and submit a
proper patch if you really need it.


Takashi