Re: [PATCH 00/44 v3] Enhancement of support for Firewire devices

[David Henningsson <david.henningsson@canonical.com>]

On 03/21/2014 12:09 PM, Takashi Sakamoto wrote:
> This series is an update for my previous series:
> [alsa-devel] [GIT PULL][PATCH 00/39 v2] Enhancement of support for Firewire
> devices
> http://mailman.alsa-project.org/pipermail/alsa-devel/2014-March/073868.html

Hi,

I've tested Takashi's patches [1] with my one Firewire device (Presonus
FP10) and I've found it to work well, both midi and audio.
In addition, during my testing I've found Takashi Sakamoto to be very
good to work with, and I'm confident he will continue to look after the
driver for some time ahead.

While I haven't looked through the patches in detail - after all, the
firewire stack is not really my point of expertise (yet!) - but from a
pragmatic view, having ALSA kernel support for firewire devices is a
very welcome addition. (Although a late one, considering firewire's
declining popularity.)

While not everything is perfect yet - e g PulseAudio and Audacity are
two applications that would require some coding on either end to
facilitate better integration - no driver is ever perfect, and the
current driver is a big step in the right direction.

To sum up, from my pragmatic view I've found the patch series [1] to be
stable enough to mainline, and I recommend merging this patch series for
the 3.15 kernel merge window.

-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic

[1] Not actually this patch series, but a later version of these
patches, as can be found in Takashi Sakamoto's github tree.