Re: [PATCH] hda: Added new IDT codec family

[Maxim Levitsky <maximlevitsky@gmail.com>]

On Monday 05 November 2007 14:30:07 Takashi Iwai wrote:
> At Mon, 05 Nov 2007 10:06:57 -0500,
> Matthew Ranostay wrote:
> > 
> > diff -r 3e1a17f8baca pci/hda/hda_codec.h
> > --- a/pci/hda/hda_codec.h	Wed Oct 31 17:36:20 2007 +0100
> > +++ b/pci/hda/hda_codec.h	Sat Nov 03 22:55:14 2007 -0500
> > @@ -92,6 +92,8 @@ enum {
> >   */
> >  #define AC_VERB_SET_STREAM_FORMAT		0x200
> >  #define AC_VERB_SET_AMP_GAIN_MUTE		0x300
> > +#define AC_VERB_SET_RIGHT_AMP_IN_MUTE	0x350
> > +#define AC_VERB_SET_LEFT_AMP_IN_MUTE	0x360
> 
> Usually we pass the values for left or right bits in the parameter
> value (AC_AMP_SET_LEFT and AC_AMP_SET_RIGHT) instead of the verb
> value.  So, no need to redefine them.
> 
> >  #define AC_VERB_SET_PROC_COEF			0x400
Any chance to see data-sheets of newer IDT chips?
There seems to be nothing on the web.
Are they available to general public?

For example STAC9872 has very limited support due to no datasheets.
 
>+static struct snd_kcontrol_new stac92hd71bxx_mixer[] = {
>+       STAC_DIGITAL_INPUT_SOURCE(1),
>+       STAC_INPUT_SOURCE(2),
>+       STAC_VOLKNOB(0x28),
>+
>+       /* hardware gain controls */
>+       HDA_CODEC_VOLUME("Digital Mic 1 Volume", 0x18, 0, HDA_OUTPUT),
>+       HDA_CODEC_VOLUME("Digital Mic 2 Volume", 0x19, 0, HDA_OUTPUT),
>+       
>+       HDA_CODEC_VOLUME("Capture Volume", 0x1c, 0, HDA_OUTPUT),
>+       HDA_CODEC_MUTE("Capture Switch", 0x1c, 0, HDA_OUTPUT),
>+       HDA_CODEC_VOLUME("Capture Mux Volume", 0x1a, 0, HDA_OUTPUT),
>        { } /* end */

The chip has two ADCs, but why just one "Capture Volume" ?
No analog loopback?

Best regards,
	Maxim Levitsky