From: Clemens Ladisch <clemens@ladisch.de>
To: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Cc: alsa-devel@alsa-project.org
Subject: Re: include/uapi/firewire.h for other firewire drivers
Date: Mon, 28 Oct 2013 09:29:47 +0100 [thread overview]
Message-ID: <526E207B.5060301@ladisch.de> (raw)
In-Reply-To: <526B5017.8010106@sakamocchi.jp>
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
next prev parent reply other threads:[~2013-10-28 8:29 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 [this message]
2013-10-31 8:30 ` Takashi Sakamoto
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=526E207B.5060301@ladisch.de \
--to=clemens@ladisch.de \
--cc=alsa-devel@alsa-project.org \
--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.