Re: [PATCH 0/2] alsa-lib patches for rawmidi tstamp framing

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

 



On Tue, 17 Aug 2021 12:07:39 +0200,
David Henningsson wrote:
> 
> 
> On 2021-08-17 07:35, Takashi Iwai wrote:
> > On Mon, 16 Aug 2021 18:06:23 +0200,
> > David Henningsson wrote:
> >> Well, now that we're about to release a kernel with support for rawmidi tstamp framing,
> >> how about some alsa-lib support for it as well?
> > Sure, that's the missing piece.
> > But, could you give a bit more description to each patch?
> Do you mean commit message?

Yes.

> I could just write this for patch 1/2 (i e, copy the doc section added
> in the patch):
> 
> Optionally, incoming rawmidi bytes can be put inside a frame of type snd_rawmidi_framing_tstamp_t.
> The main current benefit is that can enable in-kernel timestamping of incoming bytes, and that
> timestamp is likely to be more precise than what userspace can offer.
> 
> Tstamp type framing requires a kernel >= 5.14 and a buffer size that is a multiple of
> sizeof(snd_rawmidi_framing_tstamp_t). It is only available on input streams.
> 
> And this for patch 2/2:
> 
> Adds a "clock type" parameter to the "rawmidi" test program, that when
> used and combined with -i (and -v to be useful), activates tstamp type
> framing with the specified clock type.
> 
> I hope this description makes it possible for you to review the patch
> content as well.

The texts look OK, please resubmit.


thanks,

Takashi



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

  Powered by Linux