From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:43317) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QMNdH-0001OZ-DZ for qemu-devel@nongnu.org; Tue, 17 May 2011 13:00:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QMNdG-0002H6-BQ for qemu-devel@nongnu.org; Tue, 17 May 2011 13:00:31 -0400 Received: from sj-iport-3.cisco.com ([171.71.176.72]:49532) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QMNdG-0002Gu-0A for qemu-devel@nongnu.org; Tue, 17 May 2011 13:00:30 -0400 Message-ID: <4DD2A9AB.7080101@cisco.com> Date: Tue, 17 May 2011 11:00:27 -0600 From: David Ahern MIME-Version: 1.0 References: <1305575782-31766-1-git-send-email-kraxel@redhat.com> <1305575782-31766-19-git-send-email-kraxel@redhat.com> <4DD1E1DD.3060605@cisco.com> <4DD221AB.7070009@redhat.com> In-Reply-To: <4DD221AB.7070009@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 18/18] usb: add ehci adapter List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gerd Hoffmann Cc: qemu-devel@nongnu.org On 05/17/11 01:20, Gerd Hoffmann wrote: >> On 05/16/11 13:56, Gerd Hoffmann wrote: >>> This patch finally merges the EHCI host adapter aka USB 2.0 support. >>> >>> Based on git://git.kiszka.org/qemu.git ehci >>> >>> Changes: >>> - Adapt to recent changes in the usb subsystem. >>> - Don't create device automagically, use -device instead. >>> - Add quickstart text file, see docs/usb2.txt. >>> - A bunch of codestyle fixups. >>> - Add authors+contributers list. >>> - Zap EHCI_NOMICROFRAMES, qemu can't handle a 8 kHz >>> wakeup rate anyway. >>> - A few bug fixes. >>> >>> Signed-off-by: Gerd Hoffmann >> >> 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. > >> The inclusion of EHCI into >> qemu can be done in such a way as to maintain the history. > > Prove it. Give me a git tree with ehci history I can use as replacement > for patch 18 and I'll pull it in. To come back to this one: Back in December when you got a hankering to to be the USB maintainer Jan could have created a pull request that you sucked into your repo. I am not a git guru, but I believe the syntax is something like: git request-pull master git://git.kiszka.org/qemu.git At this point you clearly have no motivation or intention to save the history. The least you can do is summarize it with a better commit message. One that references the original code and the efforts by Mark Burkley. I believe this is the original code: http://lists.gnu.org/archive/html/qemu-devel/2008-10/msg00764.html Followed by contributions by various folks and my efforts to make the code usable - validated by the fact that for a linux host USB storage devices worked (mostly) as well as printers and scanners. There are about 16 commits to usb-ehci; surely that is not too cumbersome to read the commit logs and properly summarize. David > > cheers, > Gerd >