Re: [alsa-devel] [RFC] [PATCH] firewire-lib: add support for AV/C deferred transaction

Takashi Sakamoto wrote:

Can I request your comments about attached patch?

This is for adding support of deferred transaction into snd-firewire-lib. I confirm some BeBoB devices use this type of transaction. If you have no negative comments, I'll add this patch for my series.

Looks OK.

For 'undefined interval' between INTERIM and final response, I want to use 'FCP_TIMEOUT_MS' (=125msec) again. In specifications which I can read, there is no limit for this interval. But I need to promise to finish this function for callers.

In theory, a device that uses INTERIM is likely to need a larger timeout than the FCP default, but as long as 125 ms works for those BeBoB devices, increasing the timeout is not necessary.

Regards, Clemens