--- On Tue, 4/14/09, Takashi Iwai <tiwai@xxxxxxx> wrote:
>
> Thanks for the patch. The change looks almost good to me.
> See some comments below
>
>
>
> > + spec->num_dmics = 1;
> > + spec->num_dmuxes = 2;
> > + stac92xx_auto_set_pinctl(codec, 0x1c, 2);
>
> The number "2" doesn't look cute...
>
> Could you fix and repost the patch?
>
>
> thanks,
>
> Takashi
Are you worried it looks like a magic number? Or am I supposed to be looking for something else? Or would a '#define' be more what you're talking about?
Oh, and by the way, I figured out what pulseaudio was doing. I still don't like it, but it seems to be the norm for it.
Can you (simple) explain to me what the PCI_QUIRK stuff does? I've tried adding to it to get the autodetect to work and I really don't know what I'm doing there. If not, manual setting is fine, I just wanted to make the patch work automatically. I've read the source, read your driver handbook, and I still don't get it. :P
Thanks,
James
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
