From: Anthony Liguori <anthony@codemonkey.ws>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [RfC PATCH 0/2] usb: reorganize source files.
Date: Thu, 08 Mar 2012 15:37:18 -0600 [thread overview]
Message-ID: <4F59268E.8040902@codemonkey.ws> (raw)
In-Reply-To: <1331198182-11516-1-git-send-email-kraxel@redhat.com>
On 03/08/2012 03:16 AM, Gerd Hoffmann wrote:
> Hi,
>
> Plan is basically to move all usb stuff to hw/usb/
>
> Comments?
It's easy to bikeshed something like this but since it's clearly an improvement
over what we have right now, I'm all for it.
Regards,
Anthony Liguori
>
> cheers,
> Gerd
>
> Gerd Hoffmann (2):
> usb: the big rename
> usb: zap hw/ush-{ohic,uhci}.h + init wrappers
>
> Makefile.objs | 22 ++++++++++++----------
> Makefile.target | 2 +-
> configure | 4 +++-
> hw/alpha_sys.h | 1 -
> hw/pc_piix.c | 4 +---
> hw/ppc_newworld.c | 3 +--
> hw/ppc_oldworld.c | 3 +--
> hw/ppc_prep.c | 3 +--
> hw/realview.c | 3 +--
> hw/usb-ohci.h | 9 ---------
> hw/usb-uhci.h | 10 ----------
> hw/{usb-bus.c => usb/bus.c} | 6 +++---
> hw/{usb.c => usb/core.c} | 2 +-
> hw/{usb-desc.c => usb/desc.c} | 4 ++--
> hw/{usb-desc.h => usb/desc.h} | 0
> hw/{usb-audio.c => usb/dev-audio.c} | 8 ++++----
> hw/{usb-bt.c => usb/dev-bluetooth.c} | 6 +++---
> hw/{usb-hid.c => usb/dev-hid.c} | 8 ++++----
> hw/{usb-hub.c => usb/dev-hub.c} | 4 ++--
> hw/{usb-net.c => usb/dev-network.c} | 4 ++--
> hw/{usb-serial.c => usb/dev-serial.c} | 4 ++--
> hw/{usb-ccid.c => usb/dev-smartcard.c} | 0
> hw/{usb-msd.c => usb/dev-storage.c} | 6 +++---
> hw/{usb-wacom.c => usb/dev-wacom.c} | 6 +++---
> hw/{usb-ehci.c => usb/hcd-ehci.c} | 6 +++---
> hw/{usb-musb.c => usb/hcd-musb.c} | 6 +++---
> hw/{usb-ohci.c => usb/hcd-ohci.c} | 16 +++++-----------
> hw/{usb-uhci.c => usb/hcd-uhci.c} | 22 +++-------------------
> hw/{usb-xhci.c => usb/hcd-xhci.c} | 10 +++++-----
> usb-bsd.c => hw/usb/host-bsd.c | 0
> usb-linux.c => hw/usb/host-linux.c | 0
> usb-stub.c => hw/usb/host-stub.c | 0
> hw/{usb-libhw.c => usb/libhw.c} | 2 +-
> usb-redir.c => hw/usb/redirect.c | 0
> hw/versatilepb.c | 3 +--
> 35 files changed, 71 insertions(+), 116 deletions(-)
> delete mode 100644 hw/usb-ohci.h
> delete mode 100644 hw/usb-uhci.h
> rename hw/{usb-bus.c => usb/bus.c} (99%)
> rename hw/{usb.c => usb/core.c} (99%)
> rename hw/{usb-desc.c => usb/desc.c} (99%)
> rename hw/{usb-desc.h => usb/desc.h} (100%)
> rename hw/{usb-audio.c => usb/dev-audio.c} (99%)
> rename hw/{usb-bt.c => usb/dev-bluetooth.c} (99%)
> rename hw/{usb-hid.c => usb/dev-hid.c} (99%)
> rename hw/{usb-hub.c => usb/dev-hub.c} (99%)
> rename hw/{usb-net.c => usb/dev-network.c} (99%)
> rename hw/{usb-serial.c => usb/dev-serial.c} (99%)
> rename hw/{usb-ccid.c => usb/dev-smartcard.c} (100%)
> rename hw/{usb-msd.c => usb/dev-storage.c} (99%)
> rename hw/{usb-wacom.c => usb/dev-wacom.c} (99%)
> rename hw/{usb-ehci.c => usb/hcd-ehci.c} (99%)
> rename hw/{usb-musb.c => usb/hcd-musb.c} (99%)
> rename hw/{usb-ohci.c => usb/hcd-ohci.c} (99%)
> rename hw/{usb-uhci.c => usb/hcd-uhci.c} (98%)
> rename hw/{usb-xhci.c => usb/hcd-xhci.c} (99%)
> rename usb-bsd.c => hw/usb/host-bsd.c (100%)
> rename usb-linux.c => hw/usb/host-linux.c (100%)
> rename usb-stub.c => hw/usb/host-stub.c (100%)
> rename hw/{usb-libhw.c => usb/libhw.c} (99%)
> rename usb-redir.c => hw/usb/redirect.c (100%)
>
>
prev parent reply other threads:[~2012-03-08 21:37 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-08 9:16 [Qemu-devel] [RfC PATCH 0/2] usb: reorganize source files Gerd Hoffmann
2012-03-08 9:16 ` [Qemu-devel] [RfC PATCH 1/2] usb: the big rename Gerd Hoffmann
2012-03-08 9:37 ` Alon Levy
2012-03-09 8:34 ` Paolo Bonzini
2012-03-09 8:42 ` Gerd Hoffmann
2012-03-09 8:52 ` Paolo Bonzini
2012-03-09 9:26 ` Gerd Hoffmann
2012-03-08 9:16 ` [Qemu-devel] [RfC PATCH 2/2] usb: zap hw/ush-{ohic, uhci}.h + init wrappers Gerd Hoffmann
2012-03-08 21:29 ` [Qemu-devel] [RfC PATCH 0/2] usb: reorganize source files Andreas Färber
2012-03-09 6:50 ` Gerd Hoffmann
2012-03-09 8:35 ` Markus Armbruster
2012-03-09 8:46 ` Gerd Hoffmann
2012-03-09 8:37 ` Paolo Bonzini
2012-03-08 21:37 ` Anthony Liguori [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F59268E.8040902@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=kraxel@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).