[alsa-devel] [PATCH] Re: iec958 switch uneffective while playing ac3 stream

[Dominique Dumont]

Takashi Iwai <tiwai at suse.de> writes:

>> Almost fine with me. The option name is a bit terse. A newbie (or an
>> alsa hacker wanabee like me) might not know which command is involved
>> just by reading the parameter name (trace_cmd).
>> 
>> I'd rather have a slightly longer name like trace_codec_cmd
>
> It sounds too long to me :)

At the risk of sounding harsh, we're talking about user interface, so
our *personnal* preferences matter less than average user needs.

So, what would typical user prefer ? (knowing that the codec
communication is a very important part of the driver's behaviour)

Short version:

  $ cat /etc/modprobe.d/alsa 
  options snd-hda-intel trace_cmd=1 model=6stack-dig 

or longer version :

  $ cat /etc/modprobe.d/alsa 
  options snd-hda-intel trace_codec_cmd=1 model=6stack-dig 

Last but not least, a longer name may even save some typing: you may
have less user questions to answer to... ;-)


That said, alsa is still your baby, so it's your call now.

Cheers


-- 
Dominique Dumont 
"Delivering successful solutions requires giving people what they
need, not what they want." Kurt Bittner