All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: Gerd Hoffmann <kraxel@redhat.com>,
	Anthony Liguori <anthony@codemonkey.ws>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [RfC PATCH 0/2] usb: reorganize source files.
Date: Thu, 08 Mar 2012 22:29:40 +0100	[thread overview]
Message-ID: <4F5924C4.3020303@suse.de> (raw)
In-Reply-To: <1331198182-11516-1-git-send-email-kraxel@redhat.com>

Am 08.03.2012 10:16, schrieb Gerd Hoffmann:
> Plan is basically to move all usb stuff to hw/usb/
> 
> Comments?

hw/usb/ is fine with me (hw/ide/ is precedence), but I strongly dislike
the dev-*.c convention you've used:

For one thing, we'd have to rename 90+% of hw/ to be consistent.

For another, dev-foo.c is not natural English for a certain type of
device - foo-dev.c would be better but I'd prefer without "dev", i.e.,
hw/usb/{audio,ccid,...}.c. And then of course only devices should go
into hw/.

Also, Anthony was recently talking about building up a new Linux-like
directory structure inside qom/ but I don't see that working yet and
renames are cheap in git anyway, so I guess that's no reason against
hw/usb/ as an interim improvement.

Anthony further said he had a patch to avoid having to create
subdirectories in configure for the out-of-tree builds. That would
simplify both this and my *-user QOM series. Any update on that?

Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

  parent reply	other threads:[~2012-03-08 21:29 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 ` Andreas Färber [this message]
2012-03-09  6:50   ` [Qemu-devel] [RfC PATCH 0/2] usb: reorganize source files 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

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=4F5924C4.3020303@suse.de \
    --to=afaerber@suse.de \
    --cc=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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.