|
From: | Gerd Hoffmann |
Subject: | Re: [Qemu-devel] [PATCH 18/18] usb: add ehci adapter |
Date: | Tue, 17 May 2011 15:46:51 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110419 Red Hat/3.1.10-1.el6_0 Thunderbird/3.1.10 |
Hi,
As someone who spent a significant amount of time working on the EHCI code last year I am absolutely not ok with this. The entire contribution history for EHCI lost - and for no reason.There is a reason. I've tried to keep the history, but it was a big mess with conflicts and build errors due to ehci being out-of-tree for a loooooong time.Not true. Back in March it took me less 2-1/2 hours to see a request for a 0.14 version, update my git repo, merge master onto ehci, fix merges, test and send out: http://comments.gmane.org/gmane.comp.emulators.kvm.devel/68898 Again, most of the changes are to the EHCI code. The rest are sprinklings to add the adapter and adhere to USB API.
And it totally ignores the usb changes which went into 0.14 to prepare the qemu usb subsystem for higher speeds. There is no such thing like a v1 or v2 usb bus.
You've mailed out a single monster patch instead of a patch series. That is the problem with merges: You can trivially get a full diff against master, but you can't easily get a nicely review-able patch series.
I've tried to rebase the ehci tree. First, to fix the patch series issue. Second to adapt patches to qemu changes, to make the patch series bisect-able. Also to drop obsolete bits. That didn't work out as described. You didn't even try.
I'm not impressed. cheers, Gerd
[Prev in Thread] | Current Thread | [Next in Thread] |