command 0x7e7 sent on STAC9200 / STAC9228

command 0x7e7 sent on STAC9200 / STAC9228

[David Henningsson]

Hi Takashi,

While working with fixing the stuff that my automated test tool has 
discovered, here's one I'm not sure what to do with.

If you run hda-emu on any of these three:

codecs/canonical/stac9200-dell-inspiron-6400-ccert-200705-83
codecs/canonical/stac9200-dell-inspiron-6400-ccert-200712-207
codecs/canonical/stac9228-dell-inspiron-1420-ccert-200708-95

Hda-emu prints out:
invalid command: NID=0x1, verb=0x7e7, parm=0x0

In this case, I'm not sure if the command is invalid (and thus should be 
removed), or if it's a secret IDT verb that we need to set.

 From patch_sigmatel.c:

	/* Configure GPIOx as CMOS */
	snd_hda_codec_write(codec, codec->afg, 0, 0x7e7, 0);

But in hda-spec.c this verb is only in the verb table of IDT codecs, not 
STAC codecs.

Can you offer some enlightement?

-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic

Re: command 0x7e7 sent on STAC9200 / STAC9228

[Takashi Iwai]

At Thu, 16 Aug 2012 14:03:39 +0200,
David Henningsson wrote:
> 
> Hi Takashi,
> 
> While working with fixing the stuff that my automated test tool has 
> discovered, here's one I'm not sure what to do with.
> 
> If you run hda-emu on any of these three:
> 
> codecs/canonical/stac9200-dell-inspiron-6400-ccert-200705-83
> codecs/canonical/stac9200-dell-inspiron-6400-ccert-200712-207
> codecs/canonical/stac9228-dell-inspiron-1420-ccert-200708-95
> 
> Hda-emu prints out:
> invalid command: NID=0x1, verb=0x7e7, parm=0x0
>
> In this case, I'm not sure if the command is invalid (and thus should be 
> removed), or if it's a secret IDT verb that we need to set.
> 
>  From patch_sigmatel.c:
> 
> 	/* Configure GPIOx as CMOS */
> 	snd_hda_codec_write(codec, codec->afg, 0, 0x7e7, 0);
> 
> But in hda-spec.c this verb is only in the verb table of IDT codecs, not 
> STAC codecs.
> 
> Can you offer some enlightement?

I was just too lazy to add the old exception.
It be a valid verb.  I can't find it in datasheet, too, but it was the
code snippet provided by Sigmatel at that time.


Takashi