All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Hofman <pavel.hofman@ivitera.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: ajay khandelwal <ajay.lforum@gmail.com>, alsa-devel@alsa-project.org
Subject: Re: Spdif: Control interface
Date: Wed, 09 Nov 2011 09:19:23 +0100	[thread overview]
Message-ID: <4EBA378B.9010704@ivitera.com> (raw)
In-Reply-To: <s5hlirplnmq.wl%tiwai@suse.de>

Dne 9.11.2011 09:13, Takashi Iwai napsal(a):
> 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.

Fair enough :)

Thanks,

Pavel.

      reply	other threads:[~2011-11-09  8:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-08 14:01 Spdif: Control interface ajay khandelwal
2011-11-08 20:43 ` Pavel Hofman
2011-11-09  5:28   ` ajay khandelwal
2011-11-09  7:50     ` Pavel Hofman
2011-11-09  8:13       ` Takashi Iwai
2011-11-09  8:19         ` Pavel Hofman [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EBA378B.9010704@ivitera.com \
    --to=pavel.hofman@ivitera.com \
    --cc=ajay.lforum@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.