[alsa-devel] [RFC PATCH 1/2] ALSA: hda - Fix "PCM" name being used on one DAC when there are two DACs

[Raymond Yau]

> > >> > > > >>  > >
> > >> > > > >>  > > >> >
> > >> > > > >>  > > >> > In the scenario where there is one "Line Out", one
> > >> > "Speaker"
> > >> > > > >> and one
> > >> > > > >>  > > >> > "Headphone", and there are only two DACs, two
outputs
> > >> > > > >> will
> > >> > > > >> share a DAC.
> > >> > > > >>  > > >> > Currently any mixer on such a DAC will get the
"PCM"
> > >> > > > >> name,
> > >> > > > >> which is
> > >> > > > >>  > > >> > misleading. Instead use "Headphone+LO" or
"Speaker+LO"
> > >> > > > >> to
> > >> > > > better
> > >> > > > >>  > > >> > specify what the volume actually controls.
> > >> > > > >>  > > >>
> > >> > > > >>  > > >> Are there any examples ?
> > >> > > > >>  > > >>
> > >> > > > >>  > > >
> > >> > > > >>  > > > I used "hda-emu
> > >> > > > >>  > >
> > >> > codecs/canonical/alc3226-dell-precision-m2800-ccert-201404-14986 -i
> > >> > > > >> 1" when
> > >> > > > >>  > > developing the patches.
> > >> > > > >>  > > >
> > >> > > > >>  > > > I don't have any hardware available myself that
exposes
> > >> > > > >> this
> > >> > > > >> behavior,
> > >> > > > >>  > > but I can maybe fake one with hdajackretask, if that
> > >> > > > >> counts...
> > >> > > > >>  > > >
> > >> > > > >>  > >
> > >> > > > >>  > > How about adding these names to slaves of virtual
master
> > >> > > > >> volume/switch ?
> > >> > > > >>  > >
> > >> > > > >>  > > hdajackretask won't help if the topology of the codecs
are
> > >> > > > different
> > >> > > > >>  > >
> > >> > > > >>  > > Seem the badness still prevent the driver to support
> > >> > > > >> surround
> > >> > 5.1
> > >> > > > with
> > >> > > > >>  > > three rear panel jacks, internal speaker and front
panel
> > >> > headphone
> > >> > > > for
> > >> > > > >>  > > Thinkcenter A58 using alc662
> > >> > > > >>  > >
> > >> > > > >>  > >
> > >> > > > >>
> > >> > > >
> > >> >
> > >> >
https://www.mail-archive.com/alsa-user@lists.sourceforge.net/msg29203.html
> > >> > > > >>  > >
> > >> > > > >>  > > Why 3stack desktops with 6 channels codecs not using
> > >> > > > >> "Headphone+LO"  or
> > >> > > > >>  > > "Speaker+LO" ?
> > >> > > > >>  >
> > >> > > > >>  > The problem is just the lack of DACs, so it cannot cover
all
> > >> > > > >> three
> > >> > > > >>  > outputs, no matter how the pins are chosen.  That is,
it's no
> > >> > > > >> 6
> > >> > > > >>  > channels at all but 4 channels at most.
> > >> > > > >>  >
> > >> > > > >>  >
> > >> > > > >>
> > >> > > > >>
http://shop.lenovo.com/us/en/desktops/thinkcentre/a-series/a58/
> > >> > > > >>
> > >> > > > >> The technical specification of a58
> > >> > > > >>
> > >> > > > >> 2 pin internal speaker connector
> > >> > > > >> Alc662 5.1
> > >> > > > >>
> > >> > > > >> https://bbs.archlinux.org/viewtopic.php?id=156433
> > >> > > > >>
> > >> > > > >> Seem windows support surround 5.1
> > >> > > > >
> > >> > > > >
> > >> > > > > Raymond, there can certainly be cases which this patch does
not
> > >> > > > > cover
> > >> > -
> > >> > > > after all, it's mostly a band aid given the lack of topology
> > >> > information -
> > >> > > > but do you see cases where this patch actually causes a
> > >> > > > *regression*? If
> > >> > > > so, could you point me to alsa-info for the machine where this
patch
> > >> > causes
> > >> > > > a regression?
> > >> > > > >
> > >> > > >
> > >> > > >
> > >> >
> > >> >
www.realtek.com.tw/products/productsView.aspx?Langid=1&PNid=24&PFid=27&Level=5&Conn=4
> > >> > > >
> > >> > > > I expect those desktop with 2+2 HD Audio codec support
independent
> > >> > headphone
> > >> > > >
> > >> > > > This mean the driver should not create "Lineout+HP" Playback
volume
> > >> > > >
> > >> > > > Line out and headphone cannot share same DAC
> > >> > >
> > >> > > Do you have a pincfg representing this?  Then we can check it
easily
> > >> > > via hda-emu.
> > >> > >
> > >> >
> > >> >
> > >> >
http://git.kernel.org/cgit/linux/kernel/git/tiwai/hda-emu.git/tree/codecs/canonical/alc221--hp-prodesk-400-g1-sff-ccert-201408-15383?id=HEAD
> > >>
> > >> Both HP (0x21) and LO (0x14) can share the same DAC (0x03) over mixer
> > >> (0x0d).  So the statement above "Line out and headphone cannot share
> > >> same DAC" isn't correct.
> > >>
> > >> > Refer to user manual
> > >> >
> > >> > Multi-streaming can be enabled in the Realtek control panel to
allow
> > >> > independent audio streams to be sent to/from the front and rear
jacks
> > >>
> > >> This is the implementation detail and it's a matter of taste which
> > >> should be dependent and which not.  There is no bible here.
> > >>
> > >> You can forcibly do that by a fixed DAC assignment (or add some
> > >> badness) if you want, too...
> > >>
> > >>
> > >
> > >
http://mailman.alsa-project.org/pipermail/alsa-devel/2014-February/072775.html
> > >
> > >
> > > Seem headphone not working properly on hp prodesk 600 g1
> >
> > As I mentioned on Bugzilla, that problem ended up being due to some
> > kind of motherboard defect in the particular machine. On the
> > replacement 800 G1 machine, the headphone jack works fine. I can't
> > really say whether or not dual independent audio streams work, though.
>
> If your prodesk 800 g1 are still using alc221, you can disable the
speaker node 0x17 by hda-jack-retask and add hint indep_hp=1 to
hda-jack-retask.fw
>
> [Hint]
> indep_hp=1
>
> or
>
> in hda/hda_generic.c
>
> Force the driver to add badness when assign dac to speaker first by
>
>     if (spec-indep_hp && !indep_hp_possible(codec))
>          badness += BAD_NO_INDEP_HP;
> +  if (cfg->hp_outs > 0 && cfg->line_outs > 0)
> +      if (get_defcfg_location(snd_hda_codec_get_pincfg(codec,
cfg->line_out_pins[0])) == AC_JACK_LOC_REAR)
> +              if (fill_hardwired)
> +                      badness += BAD_NO_INDEP_HP;
>
> Driver create volume control for line out instead of speaker first
>
> Driver create "Speaker+LO" instead of "Headphone+LO"  when you apply
david's patch
>
> and add the following to patch_alc269 in patch_realtek.c
>
> spec->gen.indep_hp=1;
>
> The driver will create
>
> device 2 alt analog
>
> and independent hp switch
>
> You can play another audio stream to device 2 while playing stereo stream
to line out through device 0 at the same time when independent hp switch is
"ON"
>
> Assume alc221 is at card 1
>
> speaker-test -c2 -D hw:1,0 -t wav
>
> aplay -v -D hw:1,2  stereo.wav

To ensure independent hp control not add to those notebooks with dock line
out

codecs/canonical/alc3226-dell-precision-m2800-ccert-201404-14986

static bool indep_hp_possible(struct hda_codec *codec)

+ if (get_defcfg_location(snd_hda_codec_get_pincfg(codec,
cfg->line_out_pins[0])) != AC_JACK_LOC_REAR)
+               return false;