All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Ujfalusi <peter.ujfalusi@nokia.com>
To: ext Tanu Kaskinen <tanu.kaskinen@digia.com>
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	Clemens Ladisch <clemens@ladisch.de>
Subject: Re: Question	about	snd_mixer_selem_get_playback_dB_range()
Date: Tue, 22 Mar 2011 09:02:18 +0200	[thread overview]
Message-ID: <4D88497A.4090702@nokia.com> (raw)
In-Reply-To: <1300718695.24854.310.camel@IT-W2076>

On 03/21/11 16:44, ext Tanu Kaskinen wrote:
>>> Is it a bug in alsa (the driver or alsa-lib or whatever) if
>>> snd_mixer_selem_get_playback_dB_range() returns something else than
>>> SND_CTL_TLV_DB_GAIN_MUTE as the min gain, if the lowest volume step for
>>> the element is mute?
>>
>> Yes.
> 
> Thank you for the answer!

AFAIK the SND_CTL_TLV_DB_GAIN_MUTE received a fix regarding to this:
2f6206da0c1ff88235e6eca0077343f22a4b43ee
tlv: fix returned dB information for min-is-mute controls

Looking at the patch itself, it does going to return
min=SND_CTL_TLV_DB_GAIN_MUTE from snd_tlv_get_dB_range function.

However I have my doubts, if this is the correct way...

Take these two gain controls for example:
RAW value   GAIN1  GAIN2
   0        mute   mute
   1        -2dB   0dB
   2         0dB   1dB
   3         2dB   2dB

If user space asks for the dB_range:
        min    max
GAIN1   mute   2dB
GAIN2   mute   2dB

They are going to return with the same range, however if you ask for
-2dB on both:
GAIN1 will be -2dB
GAIN2 will be muted

So we are kind of hiding the dB range for the control.
I don't know how to fix it (well, I have some ideas, but it would need
changes in the API IMHO).

-- 
Péter

  reply	other threads:[~2011-03-22  7:02 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-21 12:52 Question about snd_mixer_selem_get_playback_dB_range() Tanu Kaskinen
2011-03-21 13:43 ` Raymond Yau
2011-03-21 13:55   ` Tanu Kaskinen
2011-03-21 14:38     ` Clemens Ladisch
2011-03-21 14:44       ` Tanu Kaskinen
2011-03-22  7:02         ` Peter Ujfalusi [this message]
2011-03-22  7:11           ` Clemens Ladisch
2011-04-11  2:19             ` Raymond Yau

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=4D88497A.4090702@nokia.com \
    --to=peter.ujfalusi@nokia.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=clemens@ladisch.de \
    --cc=tanu.kaskinen@digia.com \
    /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.