[pulseaudio-discuss] Issue with multiple hdmi codecs in radeonhd, hacked to work

[Deucher, Alexander]

> -----Original Message-----
> From: Takashi Iwai [mailto:tiwai at suse.de]
> Sent: Monday, December 09, 2013 4:26 AM
> To: Anssi Hannula
> Cc: Raymond Yau; John Ettedgui; ALSA Development Mailing List;
> erutrot at gmail.com; pulseaudio-discuss at lists.freedesktop.org; Deucher,
> Alexander
> Subject: Re: [pulseaudio-discuss] Issue with multiple hdmi codecs in
> radeonhd, hacked to work
> 
> At Mon, 09 Dec 2013 08:01:17 +0200,
> Anssi Hannula wrote:
> >
> > 09.12.2013 06:50, Raymond Yau kirjoitti:
> > > 2013/12/9 John Ettedgui <john.ettedgui at gmail.com
> > > <mailto:john.ettedgui at gmail.com>>
> > >
> > >
> > >     I am using the gitcompile script to configure, compile...
> > >     There is no configure by default, I was able to autogen it, but then
> > >     I had issues while building so I reverted back to gitcompile
> > >
> > > [    4.841881] ALSA hda_intel.c:3170 0000:01:00.1: Handle VGA-switcheroo
> > > audio client
> > > [    4.841885] ALSA hda_intel.c:3364 0000:01:00.1: Using LPIB position fix
> > > [    4.841886] ALSA hda_intel.c:3510 0000:01:00.1: Force to non-snoop
> mode
> > > [    4.841920] snd_hda_intel 0000:01:00.1: irq 47 for MSI/MSI-X
> > > [    4.841940] ALSA hda_intel.c:3635 0000:01:00.1: chipset global
> > > capabilities = 0x0
> > >
> > > seem the max number of playback streams is hardcoded in the driver
> >
> > Indeed it is hardcoded to 1 (previous ATI/AMD codecs had a single HDMI
> > pin). This is almost certainly not true anymore.
> >
> > I wonder what the limit is now, maybe the amount of HDMI pins so that
> > there can be 6 or 7 independent streams? Alex?
> 
> It's surprising that even the recent AMD chips don't set GCAPS
> properly...
> 
> We can set it max 8 or 16 for AZX_DRIVER_ATIHDMI* cases.  You'd need a
> bit more spaces for BDL and azx_dev structs, but it's small enough and
> negligible.
> 
> An untested patch is attached below.
> 

I'll see if I can find out how it should work.

Alex

> 
> thanks,
> 
> Takashi
> 
> -- 8< --
> From: Takashi Iwai <tiwai at suse.de>
> Subject: [PATCH] ALSA: hda - Increment default stream numbers for AMD
> HDMI
>  controllers
> 
> It turned out that some AMD HDMI controllers still don't provide
> proper values in GCAP register (all zero), and the driver assigns only
> one stream in that case, although the connected codec chip supports
> more than one stream.
> 
> In this patch, the default max number of streams for AMD HDMI
> controllers is increased to 8, which should suffice for most use
> cases.  This is a default value that takes effect only when no GCAP
> register is given.  The overhead by this increase is only azx_dev
> struct and BDL allocations, so it's negligible.
> 
> Signed-off-by: Takashi Iwai <tiwai at suse.de>
> ---
>  sound/pci/hda/hda_intel.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
> index af86c71f27bf..08d3a634bf42 100644
> --- a/sound/pci/hda/hda_intel.c
> +++ b/sound/pci/hda/hda_intel.c
> @@ -297,9 +297,9 @@ enum { SDI0, SDI1, SDI2, SDI3, SDO0, SDO1, SDO2,
> SDO3 };
>  #define ULI_NUM_CAPTURE		5
>  #define ULI_NUM_PLAYBACK	6
> 
> -/* ATI HDMI has 1 playback and 0 capture */
> +/* ATI HDMI may have up to 8 playbacks and 0 capture */
>  #define ATIHDMI_NUM_CAPTURE	0
> -#define ATIHDMI_NUM_PLAYBACK	1
> +#define ATIHDMI_NUM_PLAYBACK	8
> 
>  /* TERA has 4 playback and 3 capture */
>  #define TERA_NUM_CAPTURE	3
> --
> 1.8.5
>