Re: [PATCH v5 1/2] amidi: add delay option

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

 



Hi Clemens,

On 23/08/16 11:51, Clemens Ladisch wrote:
> Felipe Ferreri Tonello wrote:
>> On 23/08/16 11:37, Clemens Ladisch wrote:
>>> Felipe Ferreri Tonello wrote:
>>>> On 22/08/16 13:43, Clemens Ladisch wrote:
>>>>> Felipe F. Tonello wrote:
>>>>>> +				if (data > send_data)
>>>>>> +					usleep(delay * 1000);
>>>>>
>>>>> The MIDI data is sent asynchonously, so this delay is not _between_ messages.
>>>>> You'd have to add 320 µs for any byte not yet sent (I guess you have to use
>>>>> snd_rawmidi_status_get_avail() to find out how many have been sent).
>>>>
>>>> Right. Or I can just use snd_rawmidi_drain()?
>>>
>>> This function not only drains the ALSA buffer, but also waits for the
>>> hardware FIFO to empty.  And most drivers do not implement this, in
>>> which case the framework just does a fixed wait of 50 ms, which would
>>> correspond to a FIFO size of 156 bytes.
>>
>> Ok, so does that mean that it is not safe to use drain()?
> 
> That additional 50 ms delay might be safe, but not what you want.

Ok.

BTW: Why 320 µs for any byte not yet sent?

Thanks,

-- 
Felipe

Attachment: 0x92698E6A.asc
Description: application/pgp-keys

_______________________________________________
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