[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] hw/char/parallel: Make it possible to compil
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2] hw/char/parallel: Make it possible to compile also without CONFIG_PARALLEL |
Date: |
Tue, 12 Mar 2019 16:14:01 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 11/03/19 10:50, Thomas Huth wrote:
> For the downstream distribution of QEMU, we want to compile without
> CONFIG_PARALLEL. Commit 9157eee1b1c076ff3 already moved the function
> parallel_hds_isa_init() (which is still required for linking) into a file
> that is included anyway, but commit bb3d5ea858e7f888563a moved it
> to a separate file which is only compiled again if CONFIG_PARALLEL is
> set. To be able to link QEMU again without CONFIG_PARALLEL, the file
> should be considered for linking for all targets that have CONFIG_ISA_BUS.
> And while we're at it, add a proper comment in there with the rationale
> for the separate file.
>
> Signed-off-by: Thomas Huth <address@hidden>
> ---
> v2:
> - Include parallel-isa.o only for CONFIG_ISA_BUS (since it depends on ISA)
> - Do not rename the file anymore
> - Removed the check via object_class_by_name() - otherwise user might get
> confused if the specify one parallel port which is then silently ignored
>
> hw/char/Makefile.objs | 2 +-
> hw/char/parallel-isa.c | 3 +++
> hw/i386/Kconfig | 3 +--
> 3 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/hw/char/Makefile.objs b/hw/char/Makefile.objs
> index c4947d7..cf086e7 100644
> --- a/hw/char/Makefile.objs
> +++ b/hw/char/Makefile.objs
> @@ -2,7 +2,7 @@ common-obj-$(CONFIG_IPACK) += ipoctal232.o
> common-obj-$(CONFIG_ESCC) += escc.o
> common-obj-$(CONFIG_NRF51_SOC) += nrf51_uart.o
> common-obj-$(CONFIG_PARALLEL) += parallel.o
> -common-obj-$(CONFIG_PARALLEL) += parallel-isa.o
> +common-obj-$(CONFIG_ISA_BUS) += parallel-isa.o
> common-obj-$(CONFIG_PL011) += pl011.o
> common-obj-$(CONFIG_SERIAL) += serial.o
> common-obj-$(CONFIG_SERIAL_ISA) += serial-isa.o
> diff --git a/hw/char/parallel-isa.c b/hw/char/parallel-isa.c
> index 639e179..a043832 100644
> --- a/hw/char/parallel-isa.c
> +++ b/hw/char/parallel-isa.c
> @@ -1,6 +1,9 @@
> /*
> * QEMU Parallel PORT (ISA bus helpers)
> *
> + * These functions reside in a separate file since they also might be
> + * required for linking when compiling QEMU without CONFIG_PARALLEL.
> + *
> * Copyright (c) 2003 Fabrice Bellard
> *
> * SPDX-License-Identifier: MIT
> diff --git a/hw/i386/Kconfig b/hw/i386/Kconfig
> index 78fd703..4e4415f 100644
> --- a/hw/i386/Kconfig
> +++ b/hw/i386/Kconfig
> @@ -9,6 +9,7 @@ config PC
> imply ISA_IPMI_KCS
> imply ISA_IPMI_BT
> imply ISA_DEBUG
> + imply PARALLEL
> imply PCI_DEVICES
> imply PVPANIC
> imply QXL
> @@ -25,8 +26,6 @@ config PC
> select I82374
> select I8257
> select MC146818RTC
> - # Needed by the board code:
> - select PARALLEL
> # For ACPI builder:
> select SERIAL_ISA
> select ACPI_VMGENID
>
And also:
diff --git a/hw/sparc64/Kconfig b/hw/sparc64/Kconfig
index 4a8166ebb7..d4d76a89be 100644
--- a/hw/sparc64/Kconfig
+++ b/hw/sparc64/Kconfig
@@ -3,13 +3,13 @@ config SUN4U
imply PCI_DEVICES
imply SUNHME
imply TEST_DEVICES
+ imply PARALLEL
select M48T59
select ISA_BUS
select FDC
select SERIAL_ISA
select PCI_SABRE
select IDE_CMD646
- select PARALLEL
select PCKBD
select SIMBA
Queued, thanks.
Paolo