From: Takashi Sakamoto <o-takashi@sakamocchi.jp>
To: Clemens Ladisch <clemens@ladisch.de>
Cc: alsa-devel@alsa-project.org
Subject: Re: include/uapi/firewire.h for other firewire drivers
Date: Thu, 31 Oct 2013 17:30:20 +0900 [thread overview]
Message-ID: <5272151C.2010208@sakamocchi.jp> (raw)
In-Reply-To: <526E207B.5060301@ladisch.de>
Hi, Clemens,
I have two more questions.
> (For example, the sample rate is set by the application on the
> ALSA device; it would be a bad idea for the driver
> to _ask_ the control panel application to change it.)
I had a doubt that all of models can follow this rule because there are
some models which is not designed for stand-alone use. This type of
device cannot work without connection/streams so I assumed that changing
sampling rate or clock source brings some bad situations.
But now I confirmed all of models which I have can follow this rule.
Related to this, my drivers have PCM rules for auto-adjustment of
rate/channels. Should I remove them and fix rate/channels according to
current sampling rate?
> This is another example of a setting that must be in the
> kernel driver.
As long as I know, this type of device changes its channel formation of
AMDTP stream according to current sampling rate and current digital
input interface. The driver can get current sampling rate by AV/C
command but cannot get current digital input interface because it's
'write-only'.
For this setting, which is better for control panel/mixer application,
control interface or hwdep interface with firewire.h?
Thanks for your patience to answer my questions.
Takashi Sakamoto
(2013年10月28日 17:29), Clemens Ladisch wrote:
> Takashi Sakamoto wrote:
>>> Because every device can have settings that should not be changed
>>> while streaming.
>>
>> Currently my Fireworks/BeBoB driver give Control interfaces for these
>> settings, like changing sampling rate, switching digital interface.
>> The primarry purpose of these is the same, a prevention from stopping
>> streaming. The secondary purpose is debug codes for these functionality.
>>
>> But based on firewire.h interface, it is the applications'
>> responsibility, not drivers'. Is my understanding correct?
>
> Yes; the intent is for the kernel driver to handle only those settings
> that would be impossible or very difficult to handle with a separate
> user space process. (For example, the sample rate is set by the
> application on the ALSA device; it would be a bad idea for the driver to
> _ask_ the control panel application to change it.)
>
>> Here some devices has "write-only" settings. For such device, my
>> driver remember these settings. If the applications has such
>> responsibility, there may be some inconsistency.
>
> This is another example of a setting that must be in the kernel driver.
>
>
> Regards,
> Clemens
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2013-10-31 8:30 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-25 12:14 include/uapi/firewire.h for other firewire drivers Takashi Sakamoto
2013-10-25 13:04 ` Clemens Ladisch
2013-10-25 14:54 ` Takashi Sakamoto
2013-10-26 5:16 ` Takashi Sakamoto
2013-10-28 8:29 ` Clemens Ladisch
2013-10-31 8:30 ` Takashi Sakamoto [this message]
2013-11-01 8:36 ` Clemens Ladisch
2013-11-03 14:20 ` 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=5272151C.2010208@sakamocchi.jp \
--to=o-takashi@sakamocchi.jp \
--cc=alsa-devel@alsa-project.org \
--cc=clemens@ladisch.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.