alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
To: alsa-devel@alsa-project.org
Subject: Re: [PATCH 1/2] ALSA: update sync header when streams are linked/unlinked
Date: Wed, 23 May 2012 17:01:54 -0500	[thread overview]
Message-ID: <4FBD5E52.9050203@linux.intel.com> (raw)
In-Reply-To: <4FBD3B38.4050005@ladisch.de>

>> If I understand you well, the sync id should be a unique identifier
>> shared by all linked streams in the same group.
> Just to clarify: does the sync id identify streams that are linked, or
> streams that can be started atomically when linked?  Because at the
> moment, all drivers implement the latter.  Furthermore, it's possible
> to link completely unrelated devices, so not even the card number could
> be used for the former.
I may have a very Intel-centric view, but it should be the former, 
identify all streams currently linked. All devices controlled by the 
same HDAudio controller can be linked at any time, providing the list of 
possible streams to link to does help anyone since the information is 
known by default.
However, I've also seen that a lot of non-HDAudio drivers seem to 
provide the ability to link only playback and capture for synchronized 
full-duplex operation. This is a much simpler case than multiple 
playback/capture devices, most serial links (SSI, McBSP, SSP, etc) 
provide such capabilities. Bottom line is that maybe the sync 
information needs to provide both the devices that can be linked and the 
devices currently linked.

Regarding the card #, linking between devices handled by different 
controllers is not supported in hardware, only 32 devices controlled 
with the same SSYNC register can be linked. In addition there are tests 
in the HDAudio code to yank devices with a different card number from 
the group (see azx_pcm_trigger() in hda_intel.c)
-Pierre

  reply	other threads:[~2012-05-23 22:01 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-22 19:54 [PATCH 1/2] ALSA: update sync header when streams are linked/unlinked Pierre-Louis Bossart
2012-05-22 19:54 ` [PATCH 2/2] ALSA: core: group read of pointer, tstamp and jiffies Pierre-Louis Bossart
2012-05-22 23:52   ` Takashi Iwai
2012-05-22 23:47 ` [PATCH 1/2] ALSA: update sync header when streams are linked/unlinked Takashi Iwai
2012-05-23 18:57   ` Pierre-Louis Bossart
2012-05-23 19:32     ` Clemens Ladisch
2012-05-23 22:01       ` Pierre-Louis Bossart [this message]
2012-05-25  5:54     ` Takashi Iwai

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=4FBD5E52.9050203@linux.intel.com \
    --to=pierre-louis.bossart@linux.intel.com \
    --cc=alsa-devel@alsa-project.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).