All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Cc: alsa-devel@alsa-project.org, clemens@ladisch.de,
	ffado-devel@lists.sf.net
Subject: Re: [RFC][PATCH 0/8] ALSA: dice: constrain PCM substreams to current sampling transfer frequency
Date: Sat, 21 Nov 2015 10:46:01 +0100	[thread overview]
Message-ID: <s5h7flbznvq.wl-tiwai@suse.de> (raw)
In-Reply-To: <564FBAE4.9070203@sakamocchi.jp>

On Sat, 21 Nov 2015 01:29:24 +0100,
Takashi Sakamoto wrote:
> > In your original statement:
> >> As a result, userspace applications can request PCM substreams at current
> >> sampling transfer frequency. Therefore, when users want to start PCM
> >> substreams at different rate, they should set the rate in advance by the
> >> other ways (i.e. ffado-dbus-server/ffado-mixer).
> > 
> > So, an application cannot change the PCM rate other than the value
> > currently set by another tool.  Is it correct?
> 
> Correct.

The single rate restriction is fairly common among many drivers.
As this appears like a hardware limitation on DICE, it's fine, per se.
But, requiring a special tool to set the sample rate is different; it
sounds strange to me.

Why it must be *only* by another tool, not by PCM interface itself?
Suppose you playing a single application.  Kernel driver also knows
that it's currently only a single process accessing the hardware.
What prevents it changing the sample rate? 

And, even if we implement in that way -- allowing only the locked
sample rate -- by some reason (e.g. due to the code complexity), why
can't it be controlled via a more common interface like a normal mixer
element or such?  Some drivers do so, simply by providing an enum
control for the master sample rate.

So again: restricting the PCM per one rate itself is understandable.
The main question is, however, how to manage the current sample rate.
If the first-user-allowed rule is applied, there won't be a big
regression, for example.


thanks,

Takashi

  reply	other threads:[~2015-11-21  9:46 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-15  9:25 [RFC][PATCH 0/8] ALSA: dice: constrain PCM substreams to current sampling transfer frequency Takashi Sakamoto
2015-11-15  9:25 ` [PATCH 1/8] ALSA: dice: limit " Takashi Sakamoto
2015-11-15  9:25 ` [PATCH 2/8] ALSA: dice: limit stream " Takashi Sakamoto
2015-11-15  9:25 ` [PATCH 3/8] ALSA: dice: add MIDI ports according to current number of MIDI substreams Takashi Sakamoto
2015-11-15  9:25 ` [PATCH 4/8] ALSA: dice: get the number of MBLA data channel at opening PCM substream Takashi Sakamoto
2015-11-15  9:25 ` [PATCH 5/8] ALSA: dice: purge generating channel cache Takashi Sakamoto
2015-11-20  0:15   ` Stefan Richter
2015-11-15  9:25 ` [PATCH 6/8] ALSA: dice: ensure phase lock before starting streaming Takashi Sakamoto
2015-11-15  9:25 ` [PATCH 7/8] ALSA: dice: expand timeout to wait for Dice notification Takashi Sakamoto
2015-11-15  9:25 ` [PATCH 8/8] ALSA: dice: wait for ensuring phase lock Takashi Sakamoto
2015-11-18 14:13 ` [RFC][PATCH 0/8] ALSA: dice: constrain PCM substreams to current sampling transfer frequency Takashi Iwai
2015-11-19  3:34   ` Takashi Sakamoto
2015-11-19 10:19     ` Takashi Iwai
2015-11-20  4:11       ` Takashi Sakamoto
2015-11-20  9:25         ` Takashi Iwai
2015-11-20 11:19           ` Takashi Sakamoto
2015-11-20 11:29             ` Takashi Iwai
2015-11-21  0:29               ` Takashi Sakamoto
2015-11-21  9:46                 ` Takashi Iwai [this message]
2015-11-24 15:04                   ` Takashi Sakamoto
2015-11-24 15:33                     ` Takashi Iwai
2015-11-25  0:08                       ` Takashi Sakamoto

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=s5h7flbznvq.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=clemens@ladisch.de \
    --cc=ffado-devel@lists.sf.net \
    --cc=o-takashi@sakamocchi.jp \
    /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.