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: Fri, 01 Nov 2013 09:36:46 +0100 [thread overview]
Message-ID: <5273681E.6010701@ladisch.de> (raw)
In-Reply-To: <5272151C.2010208@sakamocchi.jp>
Takashi Sakamoto wrote:
> my drivers have PCM rules for auto-adjustment of rate/channels. Should
> I remove them and fix rate/channels according to current sampling rate?
When the clock source is internal or the PC, the driver can change the
sample rate. When the clock source is external, the rate is constrained.
> 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?
For settings (like this) that the kernel driver knows about, it should
use a mixer control, because the control interface already has mechanisms
to notify others of changes, and to lock controls.
The hwdep interface is appropriate for commands that must go through the
driver for technical reasons (such as EFC's response address), but where
the driver does not care about the actual meaning of the command.
(For other commands, like A/VC, the application can go directly through
/dev/fw*.)
Regards,
Clemens
next prev parent reply other threads:[~2013-11-01 8:37 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
2013-11-01 8:36 ` Clemens Ladisch [this message]
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=5273681E.6010701@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.