[alsa-devel] Spdif: Control interface

[Takashi Iwai]

At Wed, 09 Nov 2011 08:50:12 +0100,
Pavel Hofman wrote:
> 
> Dne 9.11.2011 06:28, ajay khandelwal napsal(a):
> > Hi Pavel,
> >  Thanks for reply
> > 
> > Simple mixer control 'IEC958 Playback AC97-SPSA',0
> >   Capabilities: volume volume-joined penum
> >   Playback channels: Mono
> >   Capture channels: Mono
> >   Limits: 0 - 3
> >   Mono: 3 [100%]
> 
> Thanks. I assume you mean this control. For meaning of this control see
> e.g. http://alsa.opensrc.org/Realtek_ALC650#SPDIF_support
> 
> It is not related to volume, IMO capabilities of this control are
> incorrect, there should be "enum" instead. Perhaps changing
> SNDRV_CTL_ELEM_TYPE_INTEGER to SNDRV_CTL_ELEM_TYPE_ENUMERATED ?

Yes, from the design POV, it should have been an enum.
But changing this would break many working user-space setups, so no-go
for now.  Since AC97 is a legacy stuff, the worst thing is to break
something.


Takashi