[alsa-devel] HD-audio regression after commit 34588709af61be1550b4e2bcee5c85d0ac4f34d4

[Takashi Iwai]

At Thu, 17 Jan 2013 11:51:48 +0800,
Raymond Yau wrote:
> 
> > > >> >> >> >> >> >Yes, testing the latest code with real machines is
> highly
> > > >> >> >> >> >> >appreciated.  Try sound-unstable git tree, either
> master or
> > > >> >> >> >> >> >test/hda-migrate branch.
> > > >> >> >> >> >> >  git://
> > > >> >> >> >
> git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-unstable.git
> > > >> >> >> >> >> >
> > > >> >> >> >> >> >Alternatively you can build the external alsa-driver
> modules from
> > > >> >> >> >> >> >alsa-driver-unstable snapshot tarball, too
> > > >> >> >> >> >> >
> > > >> >> >> >> >
> > > >> >> >> >
> ftp://ftp.suse.com/pub/people/tiwai/snapshot/alsa-driver-unstable-snapshot.tar.gz
> > > >> >> >> >> >> >
> > > >> >> >> >> >> >Note that only the behavior of model=auto has been
> changed in the tree
> > > >> >> >> >> >> >above.  The other model behaviors of AD codecs are
> unchanged at all.
> 
> > > >> >> >> >
> > > >> >> >> >Seen many ad1988 controls are missing ?
> > > >> >> >> >
> > > >> >> >> >1) rear Mic, line , front Mic  playback volume and switch
> > > >> >> >> >2) CD playback switch
> > > >> >> >> >3) input mix in input source
> > > >> >> >> >4) analog loopback mix volume and mute switch
> 
> > > But the number of controls is smaller than in linux stable 3.7.2. I
> > > don't know if that's a regression.
> >
> >
> > Unless you see *functional* degradation, it's no regression but
> > regarded rather as a cleanup
> 
> Refer to commit
> 
> http://git.kernel.org/?p=linux/kernel/git/tiwai/sound-unstable.git;a=commitdiff;h=f3fc0b0b1fe31e0ec9a72ab85b421e74c696f00d;hp=3a65bcdc577a338712c2eaefc194909de79d4982
> 
> The Stereo Mix of those VIA codecs allow ADD capture  DAC signal
> 
> ad1988 and those realtek codecs can select input source from the audio
> mixer which can mix the input pins
> 
> - snd_hda_add_imux_item(imux, "Mix", 9, NULL);
> 
> Those  static models (3stack , 3stack-dig, 6stack and 6stack-dig )still
> have the input source "mix"
> 
> static const struct hda_input_mux ad1988_6stack_capture_source = {
>         .num_items = 5,
>        .items = {
>                { "Front Mic", 0x1 },   /* port-B */
>                 { "Line", 0x2 },        /* port-C */
>                 { "Mic", 0x4 },         /* port-E */
>                 { "CD", 0x5 },
>                  { "Mix", 0x9 },
>        },
> };

This can be added now by passing a hint "add_stereo_mix_input=true".
Maybe it's better to enable it unconditionally for AD1988, as this
codec is mostly used for desktops, not for laptops.  (On laptops with
auto-mic switching, the driver doesn't want to have capture sources
other than mics.)


Takashi