[alsa-devel] HD-audio generic parser merged to linux-next

[Takashi Iwai]

At Wed, 13 Feb 2013 14:17:14 +0100,
Takashi Iwai wrote:
> 
> At Wed, 13 Feb 2013 14:02:17 +0100,
> Takashi Iwai wrote:
> > 
> > At Wed, 13 Feb 2013 20:16:10 +0800,
> > Raymond Yau wrote:
> > > 
> > > >>
> > > >> 6 DAC channels for 5.1 surround
> > > >
> > > > It's an issue with the output path selection, and basically irrelevant
> > > > with the HP independent output.
> > > >
> > > >> Can independent headphone enabled by default since this mean that lost of
> > > >> surround 5.1 for those desktop with three audio jacks at rear panel ?
> > > >
> > > > Not really.  The independent HP feature is just an additional stuff
> > > > that is enabled only when the configuration is available in the
> > > > selected paths.  It doesn't influence on the path selection by
> > > > itself.
> > > >
> > > 
> > > The driver create "alt analog" device but does not create "Channel
> > > mode" or Surround, Center Lfe playback volume controls for intel
> > > motherboard with 3 audio jacks for those 6 channels hda codecs (e.g.
> > > alc662, ad1882, ...)
> > > 
> > > hda-emu codecs/alc662-intel-d945gclf2
> > 
> > But it's irrelevant from whether you enable independent HP mode or
> > not.  It's just the policy of badness evaluation.  It's possible to
> > have a different result by adjusting the badness parameters.
> 
> IOW, the patch like below will make surrounds back for the device,
> for the cost of losing individual headphone volume control.
> 
> It seems that it would make more sense for having surrounds for such
> desktop setups, and the patch doesn't appear to affect too severely.
> Only ALC662/663 and one CMI9880 configs have been changed by this, and
> all look reasonable.  So I'm going to apply it.

BTW, I should mention beforehand that the previous patch won't suffice
for the device like ThinkCenter A58 where the headphone can't be
shared with the primary DAC.  For allowing the surrounds on such
machines, we need to implement more complex workaround (i.e. make
surrounds exclusive, and turn off others while using surrounds).


Takashi

> 
> 
> Takashi
> 
> ---
> diff --git a/sound/pci/hda/hda_generic.c b/sound/pci/hda/hda_generic.c
> index 6af5ade..78897d0 100644
> --- a/sound/pci/hda/hda_generic.c
> +++ b/sound/pci/hda/hda_generic.c
> @@ -988,7 +988,7 @@ enum {
>  	/* No DAC is found for the extra output */
>  	BAD_NO_DAC = 0x4000,
>  	/* No possible multi-ios */
> -	BAD_MULTI_IO = 0x103,
> +	BAD_MULTI_IO = 0x120,
>  	/* No individual DAC for extra output */
>  	BAD_NO_EXTRA_DAC = 0x102,
>  	/* No individual DAC for extra surrounds */