Re: [PATCH 7/7] ALSA: usb: take startup delay into account

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/3/16 1:46 AM, Takashi Sakamoto wrote:
Hi,

On Sep 30 2016 21:43, Subhransu S. Prusty wrote:
From: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>

For playback usages, the endpoints are started before the prepare
step,

As long as I understand, in ALSA USB audio device class driver,
submitting URBs starts at .prepare called by PCM core. So not before it
as long as the 'prepare step' means .prepare call.

The comment should have said 'started during the prepare step', you're right.


and valid audio data will be rendered with a delay that
cannot be recovered.
Worst-case the initial delay due to buffering of empty URBS can
be up to 12ms

I thinks the aim of this patch is to add proper delay to current
calculation of estimated delay, due to the URBs accumulated between
.prepare and .trigger call. If so, it's better to explain about it in
the comment so that the other developers such as me can follow this
improvement.

Yes indeed. will fix but as replied to Takashi Iwai there are wider questions on why endpoints can't be linked and why the start times can't be set by the class driver.

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux