All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org
Subject: Re: [PATCH 0/2] RFC: support for audio wall clock
Date: Fri, 20 Jul 2012 14:47:24 -0500	[thread overview]
Message-ID: <5009B5CC.7070106@linux.intel.com> (raw)
In-Reply-To: <s5hpq7tuh9u.wl%tiwai@suse.de>

On 7/18/2012 12:10 PM, Takashi Iwai wrote:
> Sorry for the late reply. I've looked over the things around my 
> vacation... This version looks better, so if other people have no 
> objection, I can merge it. The extension of mmap_status seems still 
> fitting with 64bit char size, so it should be OK. But, one thing -- 
> can't we reuse the normal tstamp field for the audio wallclock? That 
> is, add a new tstamp mode, and then put wallclock tstamp there instead 
> of the normal system tstamp. Just an idea. thanks, Takashi

Thanks for reviewing the updated patches. No, the audio wall clock needs 
to come in addition to the regular system tstamp. This is needed for 
applications such as PulseAudio who try to estimate the drift between 
system time and audio time. If you remove the system time information, 
they can't program a timer any longer.
Better wait a bit to merge, I need to work on all the TODOs, complete 
some additional validation for corner cases and get your feedback live 
at LPC.
-Pierre

  reply	other threads:[~2012-07-20 19:47 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-28 21:12 [PATCH 0/2] RFC: support for audio wall clock Pierre-Louis Bossart
2012-06-28 21:12 ` [PATCH 1/2] ALSA: core: add hooks for audio timestamps read from WALLCLOCK Pierre-Louis Bossart
2012-06-28 21:12 ` [PATCH 2/2] ALSA: hda: support for wallclock timestamps Pierre-Louis Bossart
2012-06-30  1:03 ` [PATCH 0/2] RFC: support for audio wall clock Raymond Yau
2012-07-20 19:57   ` Pierre-Louis Bossart
2012-07-18 17:10 ` Takashi Iwai
2012-07-20 19:47   ` Pierre-Louis Bossart [this message]
2012-07-21  6:20     ` Takashi Iwai
  -- strict thread matches above, loose matches on Subject: below --
2012-06-13 20:26 Pierre-Louis Bossart

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=5009B5CC.7070106@linux.intel.com \
    --to=pierre-louis.bossart@linux.intel.com \
    --cc=alsa-devel@alsa-project.org \
    --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.