[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 15/25] isa: express dependencies with kconfi
From: |
Yang Zhong |
Subject: |
Re: [Qemu-devel] [RFC PATCH 15/25] isa: express dependencies with kconfig |
Date: |
Tue, 8 Jan 2019 17:08:53 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Jan 04, 2019 at 02:55:20PM +0100, Thomas Huth wrote:
> On 2018-12-27 07:34, Yang Zhong wrote:
> > From: Paolo Bonzini <address@hidden>
> >
> > Signed-off-by: Paolo Bonzini <address@hidden>
> > Signed-off-by: Yang Zhong <address@hidden>
> > ---
> [...]
> > diff --git a/hw/display/Kconfig b/hw/display/Kconfig
> > index 5e73277cc2..e0d7919a06 100644
> > --- a/hw/display/Kconfig
> > +++ b/hw/display/Kconfig
> > @@ -9,6 +9,9 @@ config ADS7846
> >
> > config VGA_CIRRUS
> > bool
> > + default y
> > + depends on ISA_BUS && PCI
> > + select VGA
>
> Please note that the cirrus-isa device has recently been split into a
> separate file, see commit:
>
> ce3cf70edaaf1b8f9a566b36011350c642287879
> Move "isa-cirrus-vga" device into a separate file
>
> I guess we need a dedicated VGA_CIRRUS_ISA switch for that file now?
>
Thanks for Thomas's reminder, you are right.
The cirrus_vga_isa.c depend on "config VGA_CIRRUS" and "config VGA_ISA"
common-obj-$(call
land,$(CONFIG_VGA_CIRRUS),$(CONFIG_VGA_ISA))+=cirrus_vga_isa.o
Those have been defined in this hw/display/Kconfig:
config VGA_CIRRUS
bool
default y
depends on ISA_BUS && PCI
select VGA
config VGA_ISA
bool
depends on ISA_BUS
select VGA
Since isa has been splited out from cirrus_vga.c file
common-obj-$(CONFIG_VGA_CIRRUS) += cirrus_vga.o
The new "config VGA_CIRRUS" will remove ISA_BUS, like below:
config VGA_CIRRUS
bool
default y
depends on PCI
select VGA
Regards,
Yang
> > diff --git a/hw/i386/Kconfig b/hw/i386/Kconfig
> > index 427bda3717..68a9064558 100644
> > --- a/hw/i386/Kconfig
> > +++ b/hw/i386/Kconfig
> > @@ -9,6 +9,7 @@ config I440FX
> >
> > config ISAPC
> > bool
> > + select ISA_BUS
> >
> > config Q35
> > bool
>
> Can Q35 and I440FX work completely without ISA_BUS already, or should
> they also select ISA_BUS here?
>
> Thomas
Thanks for comments, and "select ISA_BUS" for i440fx and Q35 are defined
in hw/isa/Kconfig file of this patch.
config PIIX4
bool
+ select ISA_BUS
config LPC_ICH9
bool
+ select ISA_BUS
+ select ACPI_X86_ICH
Regards,
Yang