All of lore.kernel.org
 help / color / mirror / Atom feed
From: Clemens Ladisch <clemens@ladisch.de>
To: o-takashi@sakamocchi.jp
Cc: tiwai@suse.de, alsa-devel@alsa-project.org,
	linux1394-devel@lists.sourceforge.net, ffado-devel@lists.sf.net
Subject: Re: [PATCH 0/8] [RFC] new driver for Echo Audio's Fireworks based devices
Date: Mon, 03 Jun 2013 13:18:04 +0200	[thread overview]
Message-ID: <51AC7B6C.7000108@ladisch.de> (raw)
In-Reply-To: <1370102158-24389-1-git-send-email-o-takashi@sakamocchi.jp>

o-takashi@sakamocchi.jp wrote:
> Currently this driver module support just PCM/MIDI kernel streaming via ALSA interfaces.
> So there are some issues about this driver.
> 1.whether adding each PCM devices for analog and digital interface

ALSA drivers should reflect the actual hardware design as much as
possible; conversions should be done in user space by alsa-lib.

If analog and digital channels are part of the same AMDTP stream, then
that is how the PCM device should look like.  (This is also how other
drivers like ice1712 work.)

There are other functions to describe and label channels.
(And when I say "there are", I actually mean "there will be,
eventually".  ;-)

> 2.where the codes for device control like volume, routing and etc are

The idea was to let the kernel driver handle only the actual streaming
(and everything else necessary for that, such as device enumeration,
sample rate, and clock source management), and let user space handle
everything else.

Volume and routing do not affect the streaming part in any way, so the
kernel driver should not bother to implement them.


4. clock source handling

AMDTP itself was designed for consumer devices, where streaming is in
only one direction, and where the receiving device always synchronizes
to the playback device; this is the equivalent of the "SYT Match" clock
source.

This is how the existing drivers (isight, firewire-speakers) work.

However, for full duplex and other clock sources, streaming becomes
more complicated.  When in "SYT match" mode, the playback stream should
be running even when only a capture PCM device is open.  In all other
modes, the PC must synchronize to the device, so the capture stream must
be running when _any_ PCM/MIDI stream is open (and, of course, the
capture packets must be used to determine when to send packets).


Regards,
Clemens

  parent reply	other threads:[~2013-06-03 11:18 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-01 15:55 [PATCH 0/8] [RFC] new driver for Echo Audio's Fireworks based devices o-takashi
2013-06-01 15:55 ` [PATCH 1/8] add main file of driver module o-takashi
2013-06-03 11:18   ` Clemens Ladisch
2013-06-01 15:55 ` [PATCH 2/8] add device specific command o-takashi
2013-06-03 11:18   ` Clemens Ladisch
2013-06-06 17:33     ` Takashi Sakamoto
2013-06-06 22:49       ` Takashi Sakamoto
2013-06-01 15:55 ` [PATCH 3/8] add control interfaces o-takashi
2013-06-01 15:55 ` [PATCH 4/8] add handling AMDTP stream o-takashi
2013-06-01 15:55 ` [PATCH 5/8] add MIDI interface o-takashi
2013-06-03 11:18   ` Clemens Ladisch
2013-06-01 15:55 ` [PATCH 6/8] add PCM interface o-takashi
2013-06-03 11:18   ` Clemens Ladisch
2013-06-01 15:55 ` [PATCH 7/8] add proc interface o-takashi
2013-06-01 15:55 ` [PATCH 8/8] modify Makefile and Kconfig to build this module o-takashi
2013-06-03 11:19   ` Clemens Ladisch
2013-06-03 11:18 ` Clemens Ladisch [this message]
2013-06-07 12:16   ` [PATCH 0/8] [RFC] new driver for Echo Audio's Fireworks based devices Takashi Sakamoto
2013-06-08  9:29     ` Clemens Ladisch

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=51AC7B6C.7000108@ladisch.de \
    --to=clemens@ladisch.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=ffado-devel@lists.sf.net \
    --cc=linux1394-devel@lists.sourceforge.net \
    --cc=o-takashi@sakamocchi.jp \
    --cc=tiwai@suse.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.