From: "Michael S. Tsirkin" <mst@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: Yang Zhong <yang.zhong@intel.com>,
qemu-devel@nongnu.org, peter.maydell@linaro.org,
thuth@redhat.com, sameo@linux.intel.com, ehabkost@redhat.com
Subject: Re: [Qemu-devel] [RFC PATCH v3 29/43] build: convert pci.mak to Kconfig
Date: Fri, 18 Jan 2019 14:21:00 -0500 [thread overview]
Message-ID: <20190118141519-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <033a0646-955f-8888-381c-fa4656a2d849@redhat.com>
On Fri, Jan 18, 2019 at 06:03:46PM +0100, Paolo Bonzini wrote:
> On 18/01/19 17:36, Michael S. Tsirkin wrote:
> >> config PCI_GENERIC
> >> bool
> >> + select PCIE
> >>
> >
> > why is this?
> >
>
> PCI_GENERIC is hw/pci-host/gpex.c, the "QEMU Generic PCI Express Bridge
> Emulation".
Oh. Not really descriptive.
I guess we should rename it to PCI_EXPRESS_GENERIC_BRIDGE or something?
> >> config PCI_XILINX
> >> bool
> >> + select PCIE
> >>
> >> config PCI_DESIGNWARE
> >> bool
> >> + select PCIE
> >
> >
> > sure about the above?
>
> The comments at the top of the two files say they are PCIe.
>
> Paolo
So I guess the part that I find confusing is which module
it is. Module should be a prefix. So if module is PCI
then everything should be PCI_ and PCIE should be
PCI_EXPRESS. If PCIE is the module then all these guys
should be renamed.
--
MST
next prev parent reply other threads:[~2019-01-18 19:21 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-18 11:23 [Qemu-devel] [RFC PATCH v3 00/43] Support Kconfig in QEMU Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 01/43] build: actually use CONFIG_PAM Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 02/43] hw/i386/Makefile.objs: Build pc_piix* and pc_q35 boards Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 03/43] hw/arm/Makefile.objs: CONFIG_VIRT created for virt board Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 04/43] hw/m68k/Makefile.objs: Conditionally build boards Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 05/43] hw/microblaze/Makefile.objs: Create configs for petalogix and xilinx boards Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 06/43] hw/mips/Makefile.objs: Create CONFIG_* for r4k, malta, mipssim boards Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 07/43] hw/ppc/Makefile.objs: Build all boards conditinally with CONFIG_* Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 08/43] hw/sh4/Makefile.objs: New CONFIG_* varibales created for sh4 boards and device Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 09/43] hw/sparc/Makefile.objs: CONFIG_* for sun4m and leon3 created Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 10/43] hw/lm32/Makefile.objs: Conditionally build lm32 and milkmyst Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 11/43] hw/xtensa/Makefile.objs: Build xtensa_sim and xtensa_fpga conditionally Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 12/43] hw/nios2/Makefile.objs: Conditionally build nios2 Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 13/43] hw/riscv/Makefile.objs: Create CONFIG_* for riscv boards Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 14/43] hw/sparc64/Makefile.objs: Create CONFIG_* for sparc64 Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 15/43] hw/alpha/Makefile.objs: Create CONFIG_* for alpha Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 16/43] hw/cris/Makefile.objs: Create CONFIG_* for cris Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 17/43] hw/hppa/Makefile.objs: Create CONFIG_* for hppa Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 18/43] hw/moxie/Makefile.objs: Conditionally build moxie Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 19/43] hw/openrisc/Makefile.objs: Create CONFIG_* for openrisc Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 20/43] hw/tricore/Makefile.objs: Create CONFIG_* for tricore Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 21/43] minikconfig: add parser skeleton Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 22/43] minikconfig: add AST Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 23/43] minikconfig: add semantic analysis Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 24/43] hw/display: make edid configurable Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 25/43] kconfig: introduce kconfig files Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 26/43] build: switch to Kconfig Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 27/43] ide: express dependencies with Kconfig Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 28/43] hw/pci/Makefile.objs: make pcie configurable Yang Zhong
2019-01-18 16:33 ` Michael S. Tsirkin
2019-01-18 17:28 ` Paolo Bonzini
2019-01-18 19:28 ` Michael S. Tsirkin
2019-01-21 10:25 ` Paolo Bonzini
2019-01-21 11:26 ` Yang Zhong
2019-01-21 11:54 ` Paolo Bonzini
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 29/43] build: convert pci.mak to Kconfig Yang Zhong
2019-01-18 16:36 ` Michael S. Tsirkin
2019-01-18 17:03 ` Paolo Bonzini
2019-01-18 19:21 ` Michael S. Tsirkin [this message]
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 30/43] build: convert sound.mak " Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 31/43] build: convert usb.mak " Yang Zhong
2019-01-18 11:23 ` [Qemu-devel] [RFC PATCH v3 32/43] scsi: express dependencies with Kconfig Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 33/43] bluetooth: " Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 34/43] isa: express dependencies with kconfig Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 35/43] i386: express dependencies with Kconfig Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 36/43] i2c: " Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 37/43] ptimer: " Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 38/43] edid: express dependencies with kconfig Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 39/43] hyperv: " Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 40/43] virtio: make virtio dependencies with Kconfig Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 41/43] i386-softmmu.mak: remove all CONFIG_* except boards definitions Yang Zhong
2019-01-18 15:28 ` Paolo Bonzini
2019-01-18 15:37 ` Thomas Huth
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 42/43] minikconf: implement allyesconfig, allnoconfig, randconfig, defconfig Yang Zhong
2019-01-18 11:24 ` [Qemu-devel] [RFC PATCH v3 43/43] Makefile: only support defconfig Yang Zhong
2019-01-18 16:30 ` [Qemu-devel] [RFC PATCH v3 00/43] Support Kconfig in QEMU Michael S. Tsirkin
2019-01-18 16:36 ` Paolo Bonzini
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=20190118141519-mutt-send-email-mst@kernel.org \
--to=mst@redhat.com \
--cc=ehabkost@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=sameo@linux.intel.com \
--cc=thuth@redhat.com \
--cc=yang.zhong@intel.com \
/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).