[alsa-devel] [PATCH 1/3] ALSA: usb: move ua101 driver
Takashi Iwai
tiwai at suse.de
Mon Mar 1 15:23:37 CET 2010
At Mon, 1 Mar 2010 15:18:48 +0100,
Daniel Mack wrote:
>
> On Mon, Mar 01, 2010 at 03:06:57PM +0100, Takashi Iwai wrote:
> > At Mon, 1 Mar 2010 15:01:29 +0100,
> > Daniel Mack wrote:
> > >
> > > Ok, will do. However, I wonder which way is best to do that. Given that
> > > you posted patches on top of mine, can I take this as a general
> > > agreement to my changeset? If so, I will just provide a patch on top to
> > > do the ua101 -> misc rename, because otherwise, I would have to rebase
> > > all three patches for that.
> >
> > Well, honestly, I prefer seeing the fixes by Clemens based on the
> > current code, and merge Daniel's refactoring patches after 2.6.34
> > merge window.
>
> Clemens fixes only make sense for v2 of the USB audio spec, which isn't
> fully supported yet. IOW, the code as it will go into 2.6.34 is not able
> to support that, with or without my refactoring. So rebasing them
> doesn't make sense IMO.
OK, then we can merge them together later.
> > Refactoring is a good thing but it can easily introduce any careless
> > bugs (like copy&paste error), and this split action makes really hard
> > to track the changes. Thus I'd like to keep this well ripened before
> > reaching to the Linus tree, not only for a few days.
>
> Yes, certainly. I didn't expect that to be taken for .34.
Good that we agree :)
> I would be fine with having it in a branch on your side, and getting
> it merged to your for-next after any .34 issues are sorted out.
>
> There will be quite some more patches to fully support the most common
> features, but they won't be as intrusive and big of course.
Fair enough.
thanks,
Takashi
More information about the Alsa-devel
mailing list