At Thu, 3 Mar 2011 18:49:42 +0000, Austin, Brian wrote:
So the verb sequence that this refers to needs to be done _before_ the controls are built. If the controls are built, then this sequence can clobber some outputs and inputs. It's my fault, I didn¹t put this code in the correct place. This sequence should go into the beginning of cs_build_controls() so that it is run before build_output and build_input are called.
The init is actually called before build_controls. So this should be fine as is.
The problem was that this verb is a part of initialization after the power-status change. The power-status change is done before calling the init (otherwise the tree-parsing might fail), and the power-status isn't changed in the init itself.
thanks,
Takashi
I can make the changes to fix this.
Thanks again for the help Brian
On 3/3/11 8:20 AM, "Takashi Iwai" tiwai@suse.de wrote:
At Thu, 03 Mar 2011 19:37:52 +0530, Arun Raghavan wrote:
On Thu, 2011-03-03 at 14:58 +0100, Takashi Iwai wrote:
At Thu, 03 Mar 2011 17:50:22 +0530, Arun Raghavan wrote:
On Thu, 2011-03-03 at 13:01 +0100, Takashi Iwai wrote:
At Thu, 3 Mar 2011 03:18:18 +0530, Arun Raghavan wrote: > > From: Arun Raghavan arun@accosted.net > > The commit a769cbcf60cee51f4431c0938acd39e7e5b76b8d > ALSA: hda - Add errata initverb sequence for CS42xx codecs > applies a fix for a chip erratum that is specific to CS4207 > unconditionally. On CS4206, this causes digital output to never
be
> activated.
It might be the power-state the verb table sets? Try to comment out only the line setting SPDIF Tx to D3, {0x08, AC_VERB_SET_POWER_STATE, 0x03} instead of disabling the whole verbs, just for testing.
Indeed, this also works.
OK, to be sure, try the patch below.
Yep, works just fine.
Thanks for checking. Merged now to sound git tree.
Takashi