[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 20/35] hw: move ISA bridges and devices to hw/is
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH 20/35] hw: move ISA bridges and devices to hw/isa/, configure with default-configs/ |
Date: |
Wed, 3 Apr 2013 13:51:51 +0300 |
On Wed, Apr 03, 2013 at 01:42:19PM +0300, Michael S. Tsirkin wrote:
> On Tue, Apr 02, 2013 at 04:58:54PM +0200, Paolo Bonzini wrote:
> > Signed-off-by: Paolo Bonzini <address@hidden>
> > ---
> > default-configs/i386-softmmu.mak | 4 ++++
> > default-configs/x86_64-softmmu.mak | 4 ++++
> > hw/i386/Makefile.objs | 7 ++-----
> > hw/isa/Makefile.objs | 6 ++++++
> > hw/{ => isa}/debugexit.c | 0
> > hw/{ => isa}/lpc_ich9.c | 0
> > hw/{ => isa}/pc-testdev.c | 0
>
> This at least seems wrong. debug devices should be in
> hw/debug or hw/test. And it does not matter that LPC
> uses some ISA ports. It's a chipset device.
> So hw/chipset or something ...
>
Maybe hw/system/ or hw/bridge/
> > hw/{ => isa}/sga.c | 0
> > hw/{ => isa}/vmport.c | 0
> > hw/{ => isa}/vt82c686.c | 0
> > hw/mips/Makefile.objs | 2 +-
> > 11 files changed, 17 insertions(+), 6 deletions(-)
> > rename hw/{ => isa}/debugexit.c (100%)
> > rename hw/{ => isa}/lpc_ich9.c (100%)
> > rename hw/{ => isa}/pc-testdev.c (100%)
> > rename hw/{ => isa}/sga.c (100%)
> > rename hw/{ => isa}/vmport.c (100%)
> > rename hw/{ => isa}/vt82c686.c (100%)
> >
> > diff --git a/default-configs/i386-softmmu.mak
> > b/default-configs/i386-softmmu.mak
> > index 60010b1..594921c 100644
> > --- a/default-configs/i386-softmmu.mak
> > +++ b/default-configs/i386-softmmu.mak
> > @@ -35,3 +35,7 @@ CONFIG_WDT_IB700=y
> > CONFIG_PC_SYSFW=y
> > CONFIG_XEN_I386=$(CONFIG_XEN)
> > CONFIG_ISA_DEBUG=y
> > +CONFIG_ISA_TESTDEV=y
> > +CONFIG_VMPORT=y
> > +CONFIG_SGA=y
> > +CONFIG_LPC_ICH9=y
> > diff --git a/default-configs/x86_64-softmmu.mak
> > b/default-configs/x86_64-softmmu.mak
> > index c2a3d4e..c0d152f 100644
> > --- a/default-configs/x86_64-softmmu.mak
> > +++ b/default-configs/x86_64-softmmu.mak
> > @@ -35,3 +35,7 @@ CONFIG_WDT_IB700=y
> > CONFIG_PC_SYSFW=y
> > CONFIG_XEN_I386=$(CONFIG_XEN)
> > CONFIG_ISA_DEBUG=y
> > +CONFIG_ISA_TESTDEV=y
> > +CONFIG_VMPORT=y
> > +CONFIG_SGA=y
> > +CONFIG_LPC_ICH9=y
> > diff --git a/hw/i386/Makefile.objs b/hw/i386/Makefile.objs
> > index 5e91d1e..1dc9575 100644
> > --- a/hw/i386/Makefile.objs
> > +++ b/hw/i386/Makefile.objs
> > @@ -1,10 +1,7 @@
> > obj-y += apic_common.o apic.o
> > -obj-y += sga.o ioapic_common.o ioapic.o piix_pci.o
> > -obj-y += vmport.o
> > -obj-y += debugexit.o
> > -obj-y += lpc_ich9.o q35.o
> > +obj-y += ioapic_common.o ioapic.o piix_pci.o
> > +obj-y += q35.o
> > obj-y += kvm/
> > -obj-y += pc-testdev.o
> >
> > obj-y := $(addprefix ../,$(obj-y))
> >
> > diff --git a/hw/isa/Makefile.objs b/hw/isa/Makefile.objs
> > index cbee381..18e2256 100644
> > --- a/hw/isa/Makefile.objs
> > +++ b/hw/isa/Makefile.objs
> > @@ -3,7 +3,13 @@ common-obj-$(CONFIG_APM) += apm.o
> > common-obj-$(CONFIG_APPLESMC) += applesmc.o
> > common-obj-$(CONFIG_I82378) += i82378.o
> > common-obj-$(CONFIG_I82374) += i82374.o
> > +common-obj-$(CONFIG_ISA_DEBUG) += debugexit.o
> > common-obj-$(CONFIG_ISA_MMIO) += isa_mmio.o
> > common-obj-$(CONFIG_PC87312) += pc87312.o
> > common-obj-$(CONFIG_PIIX4) += piix4.o
> > +common-obj-$(CONFIG_FULONG) += vt82c686.o
> > +common-obj-$(CONFIG_SGA) += sga.o
> > +common-obj-$(CONFIG_ISA_TESTDEV) += pc-testdev.o
> >
> > +obj-$(CONFIG_VMPORT) += vmport.o
> > +obj-$(CONFIG_LPC_ICH9) += lpc_ich9.o
> > diff --git a/hw/debugexit.c b/hw/isa/debugexit.c
> > similarity index 100%
> > rename from hw/debugexit.c
> > rename to hw/isa/debugexit.c
> > diff --git a/hw/lpc_ich9.c b/hw/isa/lpc_ich9.c
> > similarity index 100%
> > rename from hw/lpc_ich9.c
> > rename to hw/isa/lpc_ich9.c
> > diff --git a/hw/pc-testdev.c b/hw/isa/pc-testdev.c
> > similarity index 100%
> > rename from hw/pc-testdev.c
> > rename to hw/isa/pc-testdev.c
> > diff --git a/hw/sga.c b/hw/isa/sga.c
> > similarity index 100%
> > rename from hw/sga.c
> > rename to hw/isa/sga.c
> > diff --git a/hw/vmport.c b/hw/isa/vmport.c
> > similarity index 100%
> > rename from hw/vmport.c
> > rename to hw/isa/vmport.c
> > diff --git a/hw/vt82c686.c b/hw/isa/vt82c686.c
> > similarity index 100%
> > rename from hw/vt82c686.c
> > rename to hw/isa/vt82c686.c
> > diff --git a/hw/mips/Makefile.objs b/hw/mips/Makefile.objs
> > index e173a2d..af4d1f9 100644
> > --- a/hw/mips/Makefile.objs
> > +++ b/hw/mips/Makefile.objs
> > @@ -1,5 +1,5 @@
> > obj-y += gt64xxx.o
> > -obj-$(CONFIG_FULONG) += bonito.o vt82c686.o
> > +obj-$(CONFIG_FULONG) += bonito.o
> >
> > obj-y := $(addprefix ../,$(obj-y))
> >
> > --
> > 1.8.1.4
> >
> >
- [Qemu-devel] [PATCH v2 00/36] hw/ reorganization, part 2, Paolo Bonzini, 2013/04/02
- [Qemu-devel] [PATCH 14/35] hw: move display devices to hw/display/, configure via default-configs/, Paolo Bonzini, 2013/04/02
- [Qemu-devel] [PATCH 16/35] hw: move SSI controllers to hw/ssi/, configure via default-configs/, Paolo Bonzini, 2013/04/02
- [Qemu-devel] [PATCH 17/35] hw: move SCSI controllers to hw/scsi/, configure via default-configs/, Paolo Bonzini, 2013/04/02
- [Qemu-devel] [PATCH 15/35] hw: move I2C controllers to hw/i2c/, configure via default-configs/, Paolo Bonzini, 2013/04/02
- [Qemu-devel] [PATCH 20/35] hw: move ISA bridges and devices to hw/isa/, configure with default-configs/, Paolo Bonzini, 2013/04/02
[Qemu-devel] [PATCH 18/35] hw: move more files to hw/xen/, Paolo Bonzini, 2013/04/02
[Qemu-devel] [PATCH 01/35] hw: move headers to include/, Paolo Bonzini, 2013/04/02
[Qemu-devel] [PATCH 30/35] hw: move ARM CPU cores to hw/cpu/, configure with default-configs/, Paolo Bonzini, 2013/04/02
[Qemu-devel] [PATCH 29/35] hw: move other devices to hw/misc/, configure with default-configs/, Paolo Bonzini, 2013/04/02
[Qemu-devel] [PATCH 28/35] hw: move NVRAM interfaces to hw/nvram/, configure with default-configs/, Paolo Bonzini, 2013/04/02
[Qemu-devel] [PATCH 31/35] hw: move hw/kvm/ to hw/i386/kvm, Paolo Bonzini, 2013/04/02
[Qemu-devel] [PATCH 32/35] hw: move another file to hw/alpha/, Paolo Bonzini, 2013/04/02
[Qemu-devel] [PATCH 34/35] MAINTAINERS: update for source code movement, Paolo Bonzini, 2013/04/02