Alsa-Devel Archive on lore.kernel.org
 help / color / Atom feed
From: Tanu Kaskinen <tanuk@iki.fi>
To: Jaroslav Kysela <perex@perex.cz>,
	General PulseAudio Discussion
	<pulseaudio-discuss@lists.freedesktop.org>,
	alsa-devel <alsa-devel@alsa-project.org>
Subject: Re: [pulseaudio-discuss] [alsa-devel] Question about the various mixer options in UCM
Date: Sat, 22 Feb 2020 08:53:13 +0200
Message-ID: <9c35a20952f53624c8cd082a5e7de33d2d34adca.camel@iki.fi> (raw)
In-Reply-To: <fc493e83-3eaa-3580-483b-4bb7754245ef@perex.cz>

On Sun, 2020-02-16 at 18:38 +0100, Jaroslav Kysela wrote:
> Dne 16. 02. 20 v 11:06 Tanu Kaskinen napsal(a):
> > On Sun, 2020-02-16 at 11:42 +0200, Tanu Kaskinen wrote:
> > > + pulseaudio-discuss@lists.freedesktop.org
> > > 
> > > On Sat, 2020-02-15 at 17:25 +0100, Jaroslav Kysela wrote:
> > > > Dne 15. 02. 20 v 7:29 Tanu Kaskinen napsal(a):
> > > > > What's the difference between PlaybackVolume, PlaybackMixerElem and
> > > > > PlaybackMasterElem? Other than the obvious difference that
> > > > > PlaybackVolume is used only to configure the volume control, whereas
> > > > > PlaybackMixerElem and PlaybackMasterElem are used also to configure the
> > > > > mute control.
> > > > 
> > > > At first, I don't really know if someone uses PlaybackVolume/PlaybackSwitch.
> > > > It was defined for the direct control interface (not the mixer interface). I
> > > > do not think that we should support this.
> > > 
> > > PlaybackVolume/Switch is currently defined for PandaBoard,
> > > PandaBoardES, SDP4430 and sof-hda-dsp. Do you mean those definitions
> > > should be removed and replaced with PlaybackMixerElem? It seems that
> > > PlaybackVolume and PlaybackSwitch don't always match the same simple
> > > mixer element (e.g. PandaBoard), so I'm not sure if it's possible to
> > > use PlaybackMixerElem with those platforms.
> > > 
> > > As you know, PulseAudio added support for PlaybackVolume recently.
> > > Should we remove the support? I'm not against that, if
> > > PlaybackMixerElem is the canonical way to control volume.
> > > 
> > > AFAIK CRAS is a major user of UCM, are there others? I think the CRAS
> > > developers' opinion would be very useful here.
> > > 
> > > > I defined new PlaybackMixerElem to select the simple mixer element which
> > > > controls both volume and switch (mute) in the ALSA API. The master volume
> > > > might be also in the chain (thus PlaybackMasterElem) was introduced.
> > > > 
> > > > It seems that it might be not enough and I play with an idea to build custom
> > > > mixer description to handle the special cases (like several speakers with the
> > > > different volume controls connected to the single stereo stream etc.).
> > > > 
> > > > To keep things simple, I would probably hide all functionality to
> > > > PlaybackMixer/PlaybackMixerElem and CaptureMixer/CaptureMixerElem . The
> > > > special mixer name will create the abstract mixer for the applications and
> > > > only one simple mixer element control will set the appropriate volume for
> > > > the stream (like pulseaudio actually does for the legacy ALSA support - volume
> > > > synthetizer). So UCM will describe the mixer for alsa-lib and application will
> > > > use only abstract interface to set / get the volume and mute state.
> > > 
> > > Hiding everything behind an abstract mixer element sounds VERY good,
> > > but how to handle a situation where the application uses two devices at
> > > the same time and the devices share a volume control? For example, a
> > > phone playing a ringtone to both headphones and speakers. If the
> > > headphone output has its own abstract mixer element and the speaker
> > > output has its own abstract mixer element, the application may imagine
> > > that it can change the volumes independently, but if the outputs share
> > > a master volume control that is used by both abstract volume elements,
> > > the volumes aren't really independent, and the resulting volumes may
> > > not be what the application intended.
> 
> This is something which should UCM take in the account. In this case, the 
> "common" parts should be probably set to a maximum value (or the safe maximum 
> value) in UCM. Another way is to mark those devices as "conflicting", thus the 
> I/O won't be used simultaneously.

Hmm, in my example I assumed (but I didn't say) that the headphones and
speaker use the same PCM device. In this case the devices will be
marked as conflicting anyway, which I didn't take into account, so if
an application nevertheless wants to use both outputs, there needs to
be a separate headphones+speaker device, which can set up the volume
control in a sensible way. The application won't have control over the
independent volumes, unless some new UCM feature is invented, but
that's a separate issue. In conclusion, in this case the abstract
element idea works well.

If the PCMs are separate, I guess it's less likely that the two devices
will share a master volume control.

> > > > Actually, I am also trying to resolve the description of the speaker
> > > > configuration. It may not be only enough to give the PCM device, because we
> > > > don't know, if user connected the stereo or surround speakers to the sound
> > > > card output for example. I play with an idea to add device variants to UCM,
> > > > but the question is, how we can map this to pulseaudio profile/port schematics.
> > > > 
> > > > My quick idea is to export those variants via the verbs, so the exported verb
> > > > names might look like:
> > > > 
> > > > HiFi:Speaker-Stereo
> > > > HiFi:Speaker-5.1
> > > > 
> > > > Where 'HiFi' is the verb name, 'Speaker' is the device name and 'Stereo' is
> > > > the variant name.
> > > > 
> > > > If we need to define multiple variants, all may be exported like:
> > > > 
> > > > HiFi:Speaker-5.1,Mic-4.0
> > > > 
> > > > Also, we can enhance this and store the configuration to a file, thus 'HiFi'
> > > > can refer to 'HiFi@Speaker-5.1,Mic-4.0' by default.
> > > 
> > > Verb + list-of-device-variants sounds like a good way to map UCM
> > > devices to pulseaudio profiles (and if there's just one verb, which I
> > > expect to be the common case, don't show it in the profile name). I
> > > don't know how the variants should be configured in UCM, but I know
> > > that device variants should be able to declare conflicts with other
> > > devices (or device variants). For example, 5.1 speaker output may make
> > > recording impossible, while stereo speaker output can be used toghether
> > > with a mic. If this information is not provided by UCM, pulseaudio will
> > > have to probe all variant combinations (like it currently does with the
> > > legacy mixer system).
> > 
> > Sorry, now I realized that the Verb + list-of-device-variants scheme
> > doesn't really work after all as the profile scheme. Or maybe it does,
> > but it's significantly different from what we do currently. Switching
> > between Headphones and Speaker-Stereo often doesn't require reopening
> > the PCM device, so there's no need for separate Headphones and Speaker-
> > Stereo profiles. I guess we could still create separate profiles, it
> > just means that the profile list will get much longer.
> > 
> > We could add a separate optimization step to the profile creation
> > process. That is, first create all possible device-variant combinations
> > as the initial profile list, and then inspect which profiles can be
> > merged. Naming the merged profiles becomes a problem, but I imagine
> > it's solvable with static rules (e.g. merging Speakers-Stereo and
> > Headphones becomes Analog-Stereo), and if necessary the merging can be
> > improved gradually over time.
> > 
> > > From profile creation perspective the ideal scheme would be not based
> > on UCM devices but on PCM devices and their configuration variants, but
> > I imagine naming would be an even bigger problem with this scheme (how
> > to map PCM device names to sensible user friendly names?).
> 
> I think that I didn't explain my idea in detail. The variant verbs may be 
> almost identical (thus all devices will be defined) like the "master" one. But 
> the specific variant configuration will be returned to the application. So it 
> will differ only in the channel count value for the Speaker device or so. The 
> PCM device name + PCM parameters check will be fine. I don't think that we 
> should modify something on the PA side. PA activates only one verb per 
> soundcard now.

Oh, you want to create variant verbs? Is the idea that all possible
device combinations will be made separate verbs? That would make life
easier for PulseAudio, but wouldn't it mean a lot more work for UCM
configuration writers? Rather than generating the device combinations
automatically in PulseAudio, the combinations would have to be manually
listed in every UCM configuration.

I think it would be better to define only one verb per sound card and
declare the conflicts between the devices or device variants, and let
PulseAudio automatically generate the device combinations as profiles.

I may be misunderstanding something, I didn't for example understand
what you meant by "the PCM device name + PCM parameters check will be
fine". Some examples could be useful. Let's say that there's a sound
card that has stereo output (either headphones or line-out), 4.0
output, 5.1 output, SPDIF output and stereo input. If input is used,
5.1 output can't be used at the same time. Would UCM define the
following verbs?:

HiFi:Headphones,SPDIF,Mic
HiFi:Line-Stereo,SPDIF,Mic
HiFi:Line-4.0,SPDIF,Mic
HiFi:Line-5.1,SPDIF

-- 
Tanu

https://www.patreon.com/tanuk
https://liberapay.com/tanuk


  reply index

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-15  6:29 Tanu Kaskinen
2020-02-15 16:25 ` Jaroslav Kysela
2020-02-16  9:42   ` Tanu Kaskinen
2020-02-16 10:06     ` [alsa-devel] [pulseaudio-discuss] " Tanu Kaskinen
2020-02-16 17:38       ` Jaroslav Kysela
2020-02-22  6:53         ` Tanu Kaskinen [this message]
2020-02-23  9:00           ` [pulseaudio-discuss] [alsa-devel] " Tanu Kaskinen
2020-02-23 13:55             ` Jaroslav Kysela
2020-02-26  8:05               ` Tanu Kaskinen

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=9c35a20952f53624c8cd082a5e7de33d2d34adca.camel@iki.fi \
    --to=tanuk@iki.fi \
    --cc=alsa-devel@alsa-project.org \
    --cc=perex@perex.cz \
    --cc=pulseaudio-discuss@lists.freedesktop.org \
    /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

Alsa-Devel Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/alsa-devel/0 alsa-devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 alsa-devel alsa-devel/ https://lore.kernel.org/alsa-devel \
		alsa-devel@alsa-project.org
	public-inbox-index alsa-devel

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.alsa-project.alsa-devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git