[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v4 29/44] hw/pci/Makefile.objs: make pcie co
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [RFC PATCH v4 29/44] hw/pci/Makefile.objs: make pcie configurable |
Date: |
Thu, 24 Jan 2019 21:43:44 -0500 |
On Fri, Jan 25, 2019 at 10:10:53AM +0800, Yang Zhong wrote:
> On Wed, Jan 23, 2019 at 09:23:49AM -0500, Michael S. Tsirkin wrote:
> > On Wed, Jan 23, 2019 at 02:56:03PM +0800, Yang Zhong wrote:
> > > Make pcie splited from pci and make it configurable.
> > >
> > > Signed-off-by: Yang Zhong <address@hidden>
> > > Cc: Michael S. Tsirkin <address@hidden>
> > > Reviewed-by: Thomas Huth <address@hidden>
> > > ---
> > > default-configs/pci.mak | 1 +
> > > hw/pci/Kconfig | 3 +++
> > > hw/pci/Makefile.objs | 5 +++--
> > > 3 files changed, 7 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/default-configs/pci.mak b/default-configs/pci.mak
> > > index f7b3690bbd..b17b456b1e 100644
> > > --- a/default-configs/pci.mak
> > > +++ b/default-configs/pci.mak
> > > @@ -1,4 +1,5 @@
> > > CONFIG_PCI=y
> > > +CONFIG_PCI_EXPRESS=y
> > > # For now, CONFIG_IDE_CORE requires ISA, so we enable it here
> > > CONFIG_ISA_BUS=y
> > > CONFIG_VIRTIO_PCI=y
> > > diff --git a/hw/pci/Kconfig b/hw/pci/Kconfig
> > > index d3d2205577..81533b9dc0 100644
> > > --- a/hw/pci/Kconfig
> > > +++ b/hw/pci/Kconfig
> > > @@ -1,2 +1,5 @@
> > > config PCI
> > > bool
> > > +
> > > +config PCI_EXPRESS
> > > + bool
> >
> > Hmm this allows PCIE without PCI.
> > Should PCI_EXPRESS select PCI?
> >
> > It's selected itself so can't depend on PCI.
> >
> Hello Michael,
>
> I did this in patch 30 as below:
>
> diff --git a/hw/pci/Kconfig b/hw/pci/Kconfig
> index 81533b9dc0..4ca2537980 100644
> --- a/hw/pci/Kconfig
> +++ b/hw/pci/Kconfig
> @@ -3,3 +3,4 @@ config PCI
>
> config PCI_EXPRESS
> bool
> + select PCI
>
> Regards,
>
> Yang
Maybe squash this into patch 29 then.
> >
> > > diff --git a/hw/pci/Makefile.objs b/hw/pci/Makefile.objs
> > > index 9f905e6344..d30eb32cbb 100644
> > > --- a/hw/pci/Makefile.objs
> > > +++ b/hw/pci/Makefile.objs
> > > @@ -2,8 +2,9 @@ common-obj-$(CONFIG_PCI) += pci.o pci_bridge.o
> > > common-obj-$(CONFIG_PCI) += msix.o msi.o
> > > common-obj-$(CONFIG_PCI) += shpc.o
> > > common-obj-$(CONFIG_PCI) += slotid_cap.o
> > > -common-obj-$(CONFIG_PCI) += pci_host.o pcie_host.o
> > > -common-obj-$(CONFIG_PCI) += pcie.o pcie_aer.o pcie_port.o
> > > +common-obj-$(CONFIG_PCI) += pci_host.o
> > > +common-obj-$(CONFIG_PCI_EXPRESS) += pcie.o pcie_aer.o
> > > +common-obj-$(CONFIG_PCI_EXPRESS) += pcie_port.o pcie_host.o
> > >
> > > common-obj-$(call lnot,$(CONFIG_PCI)) += pci-stub.o
> > > common-obj-$(CONFIG_ALL) += pci-stub.o
> > > --
> > > 2.17.1
- [Qemu-devel] [RFC PATCH v4 25/44] hw/display: make edid configurable, (continued)
- [Qemu-devel] [RFC PATCH v4 25/44] hw/display: make edid configurable, Yang Zhong, 2019/01/23
- [Qemu-devel] [RFC PATCH v4 24/44] minikconfig: add semantic analysis, Yang Zhong, 2019/01/23
- [Qemu-devel] [RFC PATCH v4 28/44] ide: express dependencies with Kconfig, Yang Zhong, 2019/01/23
- [Qemu-devel] [RFC PATCH v4 27/44] build: switch to Kconfig, Yang Zhong, 2019/01/23
- [Qemu-devel] [RFC PATCH v4 26/44] kconfig: introduce kconfig files, Yang Zhong, 2019/01/23
- [Qemu-devel] [RFC PATCH v4 29/44] hw/pci/Makefile.objs: make pcie configurable, Yang Zhong, 2019/01/23
[Qemu-devel] [RFC PATCH v4 31/44] build: convert sound.mak to Kconfig, Yang Zhong, 2019/01/23
[Qemu-devel] [RFC PATCH v4 30/44] build: convert pci.mak to Kconfig, Yang Zhong, 2019/01/23
[Qemu-devel] [RFC PATCH v4 34/44] bluetooth: express dependencies with Kconfig, Yang Zhong, 2019/01/23
[Qemu-devel] [RFC PATCH v4 32/44] build: convert usb.mak to Kconfig, Yang Zhong, 2019/01/23
[Qemu-devel] [RFC PATCH v4 33/44] scsi: express dependencies with Kconfig, Yang Zhong, 2019/01/23
[Qemu-devel] [RFC PATCH v4 36/44] i386: express dependencies with Kconfig, Yang Zhong, 2019/01/23
[Qemu-devel] [RFC PATCH v4 35/44] isa: express dependencies with kconfig, Yang Zhong, 2019/01/23