On Wed, 26 Sep 2001, Thomas Hood wrote:

> Hi.
>
> There are some problems with the controls on the cs4236+ chips.
>
> 1) The same control bit is given two different names:
> > CS4236_SINGLEC("S/PDIF Output 3D", 0, 3, 5, 1, 0),
> and
> > CS4236_SINGLEC("3D Control - S/PDIF", 0, 3, 5, 1, 0)

It's already in CVS.

> 2) The following control is omitted:
>   CS4236_SINGLEC("the name you think appropriate", 0, 4, 7, 1, 0)

It's already there. See the CS4236_IEC958_ENABLE.

> If you decide to make revisions to the the control section,
> I would ask that you consider the following:
>
> 3) The output is not, strictly speaking, S/PDIF but IEC958
> and the controls should be named accordingly.

Done.

> 4) It would help greatly if the controls were arranged in a
> systematic order within each group, such as:
>    master controls, playback controls, capture controls
> Presently the controls do not always follow a regular order
> and the result is that GUI mixer programs lay the controls
> out in a more confusing way than they otherwise would.
> I realize that the mixer programs should be written so that
> control layout can be customized, but there is nothing wrong
> with providing a default order that is, er, orderly.

If you want to change the order, there are two functions in ALSA library
for control/names sorting:

hcontrol.c:static int get_compare_weight(const snd_ctl_elem_id_t *id)
simple.c:static int get_compare_weight(const char *name, unsigned int idx)

Probably, it would be better to make these sorting tables configuration at
run-time (store default configuration to /usr/share/alsa/mixer.conf etc.).

                                                Jaroslav

-----
Jaroslav Kysela <[EMAIL PROTECTED]>
SuSE Linux    http://www.suse.com
ALSA project  http://www.alsa-project.org


_______________________________________________
Alsa-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/alsa-devel

Reply via email to