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

[Takashi Iwai]

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.


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 */