* [PATCH 0/7] Trivial: Mark some more files as target-independant
@ 2023-01-12 13:49 Thomas Huth
2023-01-12 13:49 ` [PATCH 1/7] hw/display: Move omap_lcdc.c out of target-specific source set Thomas Huth
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: Thomas Huth @ 2023-01-12 13:49 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-arm, qemu-trivial
Here's a collection of low-hanging fruits to mark some more files
as target-independent (so that they do not have to be compiled twice,
once for qemu-system-arm and once for qemu-system-aarch64).
Philippe's patches have been on the list before, but I slightly
modified some of them (like fixing typos in the subject etc.).
My patches are new.
Philippe Mathieu-Daudé (4):
hw/display: Move omap_lcdc.c out of target-specific source set
hw/intc: Move some files out of the target-specific source set
hw/tpm: Move tpm_ppi.c out of target-specific source set
hw/arm: Move various units to softmmu_ss[]
Thomas Huth (3):
hw/cpu: Mark arm11 and realview mpcore as target-independent code
hw/intc: Mark more interrupt-controller files as target independent
hw/usb: Mark the XLNX_VERSAL-related files as target-independent
hw/arm/meson.build | 11 +++++++----
hw/cpu/meson.build | 4 ++--
hw/display/meson.build | 2 +-
hw/intc/meson.build | 12 ++++++------
hw/tpm/meson.build | 4 ++--
hw/usb/meson.build | 4 ++--
6 files changed, 20 insertions(+), 17 deletions(-)
--
2.31.1
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/7] hw/display: Move omap_lcdc.c out of target-specific source set
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
@ 2023-01-12 13:49 ` Thomas Huth
2023-01-12 13:49 ` [PATCH 2/7] hw/intc: Move some files out of the " Thomas Huth
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Thomas Huth @ 2023-01-12 13:49 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-arm, qemu-trivial
From: Philippe Mathieu-Daudé <philmd@linaro.org>
While only used by the ARM targets, this device can be built
once for all.
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221209170042.71169-2-philmd@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/display/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/display/meson.build b/hw/display/meson.build
index 7a725ed80e..2cab7a8ff2 100644
--- a/hw/display/meson.build
+++ b/hw/display/meson.build
@@ -111,7 +111,7 @@ if config_all_devices.has_key('CONFIG_VIRTIO_VGA')
hw_display_modules += {'virtio-vga-gl': virtio_vga_gl_ss}
endif
-specific_ss.add(when: 'CONFIG_OMAP', if_true: files('omap_lcdc.c'))
+softmmu_ss.add(when: 'CONFIG_OMAP', if_true: files('omap_lcdc.c'))
softmmu_ss.add(when: 'CONFIG_ALL', if_true: files('acpi-vga-stub.c'))
modules += { 'hw-display': hw_display_modules }
--
2.31.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/7] hw/intc: Move some files out of the target-specific source set
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
2023-01-12 13:49 ` [PATCH 1/7] hw/display: Move omap_lcdc.c out of target-specific source set Thomas Huth
@ 2023-01-12 13:49 ` Thomas Huth
2023-01-12 13:49 ` [PATCH 3/7] hw/tpm: Move tpm_ppi.c out of " Thomas Huth
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Thomas Huth @ 2023-01-12 13:49 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-arm, qemu-trivial
From: Philippe Mathieu-Daudé <philmd@linaro.org>
The Goldfish interrupt controller is not target specific.
While the Exynos interrupt combiner is only used by the ARM
targets, we can build this device once for all.
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221209170042.71169-3-philmd@linaro.org>
[thuth: Change patch title, and also move 'exynos4210_gic.c']
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/intc/meson.build | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/intc/meson.build b/hw/intc/meson.build
index bcbf22ff51..eeb45db58e 100644
--- a/hw/intc/meson.build
+++ b/hw/intc/meson.build
@@ -13,6 +13,8 @@ softmmu_ss.add(when: 'CONFIG_ARM_GICV3_TCG', if_true: files(
'arm_gicv3_redist.c',
))
softmmu_ss.add(when: 'CONFIG_ETRAXFS', if_true: files('etraxfs_pic.c'))
+softmmu_ss.add(when: 'CONFIG_EXYNOS4', if_true: files('exynos4210_gic.c', 'exynos4210_combiner.c'))
+softmmu_ss.add(when: 'CONFIG_GOLDFISH_PIC', if_true: files('goldfish_pic.c'))
softmmu_ss.add(when: 'CONFIG_HEATHROW_PIC', if_true: files('heathrow_pic.c'))
softmmu_ss.add(when: 'CONFIG_I8259', if_true: files('i8259_common.c', 'i8259.c'))
softmmu_ss.add(when: 'CONFIG_IMX', if_true: files('imx_avic.c', 'imx_gpcv2.c'))
@@ -33,7 +35,6 @@ specific_ss.add(when: 'CONFIG_ARM_GIC_KVM', if_true: files('arm_gic_kvm.c'))
specific_ss.add(when: ['CONFIG_ARM_GIC_KVM', 'TARGET_AARCH64'], if_true: files('arm_gicv3_kvm.c', 'arm_gicv3_its_kvm.c'))
specific_ss.add(when: 'CONFIG_ARM_V7M', if_true: files('armv7m_nvic.c'))
specific_ss.add(when: 'CONFIG_ASPEED_SOC', if_true: files('aspeed_vic.c'))
-specific_ss.add(when: 'CONFIG_EXYNOS4', if_true: files('exynos4210_gic.c', 'exynos4210_combiner.c'))
specific_ss.add(when: 'CONFIG_GRLIB', if_true: files('grlib_irqmp.c'))
specific_ss.add(when: 'CONFIG_IOAPIC', if_true: files('ioapic.c'))
specific_ss.add(when: 'CONFIG_LOONGSON_LIOINTC', if_true: files('loongson_liointc.c'))
@@ -60,7 +61,6 @@ specific_ss.add(when: 'CONFIG_PSERIES', if_true: files('xics_spapr.c', 'spapr_xi
specific_ss.add(when: 'CONFIG_XIVE', if_true: files('xive.c'))
specific_ss.add(when: ['CONFIG_KVM', 'CONFIG_XIVE'],
if_true: files('spapr_xive_kvm.c'))
-specific_ss.add(when: 'CONFIG_GOLDFISH_PIC', if_true: files('goldfish_pic.c'))
specific_ss.add(when: 'CONFIG_M68K_IRQC', if_true: files('m68k_irqc.c'))
specific_ss.add(when: 'CONFIG_NIOS2_VIC', if_true: files('nios2_vic.c'))
specific_ss.add(when: 'CONFIG_LOONGARCH_IPI', if_true: files('loongarch_ipi.c'))
--
2.31.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/7] hw/tpm: Move tpm_ppi.c out of target-specific source set
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
2023-01-12 13:49 ` [PATCH 1/7] hw/display: Move omap_lcdc.c out of target-specific source set Thomas Huth
2023-01-12 13:49 ` [PATCH 2/7] hw/intc: Move some files out of the " Thomas Huth
@ 2023-01-12 13:49 ` Thomas Huth
2023-01-12 13:49 ` [PATCH 4/7] hw/arm: Move various units to softmmu_ss[] Thomas Huth
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Thomas Huth @ 2023-01-12 13:49 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-arm, qemu-trivial
From: Philippe Mathieu-Daudé <philmd@linaro.org>
The TPM Physical Presence Interface is not target specific.
Build this file once for all targets.
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221209170042.71169-4-philmd@linaro.org>
[thuth: Drop the CONFIG_SOFTMMU statements, they are not needed here]
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/tpm/meson.build | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/tpm/meson.build b/hw/tpm/meson.build
index 1c68d81d6a..7abc2d794a 100644
--- a/hw/tpm/meson.build
+++ b/hw/tpm/meson.build
@@ -2,7 +2,7 @@ softmmu_ss.add(when: 'CONFIG_TPM_TIS', if_true: files('tpm_tis_common.c'))
softmmu_ss.add(when: 'CONFIG_TPM_TIS_ISA', if_true: files('tpm_tis_isa.c'))
softmmu_ss.add(when: 'CONFIG_TPM_TIS_SYSBUS', if_true: files('tpm_tis_sysbus.c'))
softmmu_ss.add(when: 'CONFIG_TPM_CRB', if_true: files('tpm_crb.c'))
+softmmu_ss.add(when: 'CONFIG_TPM_TIS', if_true: files('tpm_ppi.c'))
+softmmu_ss.add(when: 'CONFIG_TPM_CRB', if_true: files('tpm_ppi.c'))
-specific_ss.add(when: ['CONFIG_SOFTMMU', 'CONFIG_TPM_TIS'], if_true: files('tpm_ppi.c'))
-specific_ss.add(when: ['CONFIG_SOFTMMU', 'CONFIG_TPM_CRB'], if_true: files('tpm_ppi.c'))
specific_ss.add(when: 'CONFIG_TPM_SPAPR', if_true: files('tpm_spapr.c'))
--
2.31.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 4/7] hw/arm: Move various units to softmmu_ss[]
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
` (2 preceding siblings ...)
2023-01-12 13:49 ` [PATCH 3/7] hw/tpm: Move tpm_ppi.c out of " Thomas Huth
@ 2023-01-12 13:49 ` Thomas Huth
2023-01-12 13:49 ` [PATCH 5/7] hw/cpu: Mark arm11 and realview mpcore as target-independent code Thomas Huth
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Thomas Huth @ 2023-01-12 13:49 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-arm, qemu-trivial
From: Philippe Mathieu-Daudé <philmd@linaro.org>
arm_ss[] units are built twice: once for 32-bit word size and
once for 64-bit. The following units don't require any word
size knowledge and can be moved to softmmu_ss[] (where they
are built once):
- smmu-common.c
- exynos4_boards.c
- bcm2835_peripherals.c
- tosa.c
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20230110164406.94366-2-philmd@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/arm/meson.build | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/hw/arm/meson.build b/hw/arm/meson.build
index 92f9f6e000..4babaa8dfc 100644
--- a/hw/arm/meson.build
+++ b/hw/arm/meson.build
@@ -3,7 +3,6 @@ arm_ss.add(files('boot.c'), fdt)
arm_ss.add(when: 'CONFIG_ARM_VIRT', if_true: files('virt.c'))
arm_ss.add(when: 'CONFIG_ACPI', if_true: files('virt-acpi-build.c'))
arm_ss.add(when: 'CONFIG_DIGIC', if_true: files('digic_boards.c'))
-arm_ss.add(when: 'CONFIG_EXYNOS4', if_true: files('exynos4_boards.c'))
arm_ss.add(when: 'CONFIG_EMCRAFT_SF2', if_true: files('msf2-som.c'))
arm_ss.add(when: 'CONFIG_HIGHBANK', if_true: files('highbank.c'))
arm_ss.add(when: 'CONFIG_INTEGRATOR', if_true: files('integratorcp.c'))
@@ -18,7 +17,6 @@ arm_ss.add(when: 'CONFIG_SX1', if_true: files('omap_sx1.c'))
arm_ss.add(when: 'CONFIG_CHEETAH', if_true: files('palm.c'))
arm_ss.add(when: 'CONFIG_GUMSTIX', if_true: files('gumstix.c'))
arm_ss.add(when: 'CONFIG_SPITZ', if_true: files('spitz.c'))
-arm_ss.add(when: 'CONFIG_TOSA', if_true: files('tosa.c'))
arm_ss.add(when: 'CONFIG_Z2', if_true: files('z2.c'))
arm_ss.add(when: 'CONFIG_REALVIEW', if_true: files('realview.c'))
arm_ss.add(when: 'CONFIG_SBSA_REF', if_true: files('sbsa-ref.c'))
@@ -38,7 +36,7 @@ arm_ss.add(when: 'CONFIG_OMAP', if_true: files('omap1.c', 'omap2.c'))
arm_ss.add(when: 'CONFIG_STRONGARM', if_true: files('strongarm.c'))
arm_ss.add(when: 'CONFIG_ALLWINNER_A10', if_true: files('allwinner-a10.c', 'cubieboard.c'))
arm_ss.add(when: 'CONFIG_ALLWINNER_H3', if_true: files('allwinner-h3.c', 'orangepi.c'))
-arm_ss.add(when: 'CONFIG_RASPI', if_true: files('bcm2835_peripherals.c', 'bcm2836.c', 'raspi.c'))
+arm_ss.add(when: 'CONFIG_RASPI', if_true: files('bcm2836.c', 'raspi.c'))
arm_ss.add(when: 'CONFIG_STM32F100_SOC', if_true: files('stm32f100_soc.c'))
arm_ss.add(when: 'CONFIG_STM32F205_SOC', if_true: files('stm32f205_soc.c'))
arm_ss.add(when: 'CONFIG_STM32F405_SOC', if_true: files('stm32f405_soc.c'))
@@ -59,8 +57,13 @@ arm_ss.add(when: 'CONFIG_MSF2', if_true: files('msf2-soc.c'))
arm_ss.add(when: 'CONFIG_MUSCA', if_true: files('musca.c'))
arm_ss.add(when: 'CONFIG_ARMSSE', if_true: files('armsse.c'))
arm_ss.add(when: 'CONFIG_FSL_IMX7', if_true: files('fsl-imx7.c', 'mcimx7d-sabre.c'))
-arm_ss.add(when: 'CONFIG_ARM_SMMUV3', if_true: files('smmu-common.c', 'smmuv3.c'))
+arm_ss.add(when: 'CONFIG_ARM_SMMUV3', if_true: files('smmuv3.c'))
arm_ss.add(when: 'CONFIG_FSL_IMX6UL', if_true: files('fsl-imx6ul.c', 'mcimx6ul-evk.c'))
arm_ss.add(when: 'CONFIG_NRF51_SOC', if_true: files('nrf51_soc.c'))
+softmmu_ss.add(when: 'CONFIG_ARM_SMMUV3', if_true: files('smmu-common.c'))
+softmmu_ss.add(when: 'CONFIG_EXYNOS4', if_true: files('exynos4_boards.c'))
+softmmu_ss.add(when: 'CONFIG_RASPI', if_true: files('bcm2835_peripherals.c'))
+softmmu_ss.add(when: 'CONFIG_TOSA', if_true: files('tosa.c'))
+
hw_arch += {'arm': arm_ss}
--
2.31.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 5/7] hw/cpu: Mark arm11 and realview mpcore as target-independent code
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
` (3 preceding siblings ...)
2023-01-12 13:49 ` [PATCH 4/7] hw/arm: Move various units to softmmu_ss[] Thomas Huth
@ 2023-01-12 13:49 ` Thomas Huth
2023-01-12 13:49 ` [PATCH 6/7] hw/intc: Mark more interrupt-controller files as target independent Thomas Huth
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Thomas Huth @ 2023-01-12 13:49 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-arm, qemu-trivial
Seems like there is nothing target-specific in here, so these files
can be moved to softmmu_ss to avoid that they get compiled twice
(once for qemu-system-arm and once for qemu-system-aarch64).
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/cpu/meson.build | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/cpu/meson.build b/hw/cpu/meson.build
index 9e52fee9e7..e37490074f 100644
--- a/hw/cpu/meson.build
+++ b/hw/cpu/meson.build
@@ -1,6 +1,6 @@
softmmu_ss.add(files('core.c', 'cluster.c'))
-specific_ss.add(when: 'CONFIG_ARM11MPCORE', if_true: files('arm11mpcore.c'))
-specific_ss.add(when: 'CONFIG_REALVIEW', if_true: files('realview_mpcore.c'))
+softmmu_ss.add(when: 'CONFIG_ARM11MPCORE', if_true: files('arm11mpcore.c'))
+softmmu_ss.add(when: 'CONFIG_REALVIEW', if_true: files('realview_mpcore.c'))
specific_ss.add(when: 'CONFIG_A9MPCORE', if_true: files('a9mpcore.c'))
specific_ss.add(when: 'CONFIG_A15MPCORE', if_true: files('a15mpcore.c'))
--
2.31.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 6/7] hw/intc: Mark more interrupt-controller files as target independent
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
` (4 preceding siblings ...)
2023-01-12 13:49 ` [PATCH 5/7] hw/cpu: Mark arm11 and realview mpcore as target-independent code Thomas Huth
@ 2023-01-12 13:49 ` Thomas Huth
2023-01-12 13:49 ` [PATCH 7/7] hw/usb: Mark the XLNX_VERSAL-related files as target-independent Thomas Huth
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Thomas Huth @ 2023-01-12 13:49 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-arm, qemu-trivial
Seems like there is also nothing target-specific in here, so these
files can be moved to softmmu_ss to avoid that they get compiled
twice (once for qemu-system-arm and once for qemu-system-aarch64).
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/intc/meson.build | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/intc/meson.build b/hw/intc/meson.build
index eeb45db58e..ad8953f3ed 100644
--- a/hw/intc/meson.build
+++ b/hw/intc/meson.build
@@ -12,6 +12,8 @@ softmmu_ss.add(when: 'CONFIG_ARM_GICV3_TCG', if_true: files(
'arm_gicv3_its.c',
'arm_gicv3_redist.c',
))
+softmmu_ss.add(when: 'CONFIG_ALLWINNER_A10_PIC', if_true: files('allwinner-a10-pic.c'))
+softmmu_ss.add(when: 'CONFIG_ASPEED_SOC', if_true: files('aspeed_vic.c'))
softmmu_ss.add(when: 'CONFIG_ETRAXFS', if_true: files('etraxfs_pic.c'))
softmmu_ss.add(when: 'CONFIG_EXYNOS4', if_true: files('exynos4210_gic.c', 'exynos4210_combiner.c'))
softmmu_ss.add(when: 'CONFIG_GOLDFISH_PIC', if_true: files('goldfish_pic.c'))
@@ -19,33 +21,31 @@ softmmu_ss.add(when: 'CONFIG_HEATHROW_PIC', if_true: files('heathrow_pic.c'))
softmmu_ss.add(when: 'CONFIG_I8259', if_true: files('i8259_common.c', 'i8259.c'))
softmmu_ss.add(when: 'CONFIG_IMX', if_true: files('imx_avic.c', 'imx_gpcv2.c'))
softmmu_ss.add(when: 'CONFIG_IOAPIC', if_true: files('ioapic_common.c'))
+softmmu_ss.add(when: 'CONFIG_OMAP', if_true: files('omap_intc.c'))
softmmu_ss.add(when: 'CONFIG_OPENPIC', if_true: files('openpic.c'))
softmmu_ss.add(when: 'CONFIG_PL190', if_true: files('pl190.c'))
+softmmu_ss.add(when: 'CONFIG_RASPI', if_true: files('bcm2835_ic.c', 'bcm2836_control.c'))
softmmu_ss.add(when: 'CONFIG_REALVIEW', if_true: files('realview_gic.c'))
softmmu_ss.add(when: 'CONFIG_SLAVIO', if_true: files('slavio_intctl.c'))
softmmu_ss.add(when: 'CONFIG_XILINX', if_true: files('xilinx_intc.c'))
softmmu_ss.add(when: 'CONFIG_XLNX_ZYNQMP', if_true: files('xlnx-zynqmp-ipi.c'))
softmmu_ss.add(when: 'CONFIG_XLNX_ZYNQMP_PMU', if_true: files('xlnx-pmu-iomod-intc.c'))
-specific_ss.add(when: 'CONFIG_ALLWINNER_A10_PIC', if_true: files('allwinner-a10-pic.c'))
specific_ss.add(when: 'CONFIG_APIC', if_true: files('apic.c', 'apic_common.c'))
specific_ss.add(when: 'CONFIG_ARM_GIC', if_true: files('arm_gicv3_cpuif_common.c'))
specific_ss.add(when: 'CONFIG_ARM_GICV3_TCG', if_true: files('arm_gicv3_cpuif.c'))
specific_ss.add(when: 'CONFIG_ARM_GIC_KVM', if_true: files('arm_gic_kvm.c'))
specific_ss.add(when: ['CONFIG_ARM_GIC_KVM', 'TARGET_AARCH64'], if_true: files('arm_gicv3_kvm.c', 'arm_gicv3_its_kvm.c'))
specific_ss.add(when: 'CONFIG_ARM_V7M', if_true: files('armv7m_nvic.c'))
-specific_ss.add(when: 'CONFIG_ASPEED_SOC', if_true: files('aspeed_vic.c'))
specific_ss.add(when: 'CONFIG_GRLIB', if_true: files('grlib_irqmp.c'))
specific_ss.add(when: 'CONFIG_IOAPIC', if_true: files('ioapic.c'))
specific_ss.add(when: 'CONFIG_LOONGSON_LIOINTC', if_true: files('loongson_liointc.c'))
specific_ss.add(when: 'CONFIG_MIPS_CPS', if_true: files('mips_gic.c'))
-specific_ss.add(when: 'CONFIG_OMAP', if_true: files('omap_intc.c'))
specific_ss.add(when: 'CONFIG_OMPIC', if_true: files('ompic.c'))
specific_ss.add(when: ['CONFIG_KVM', 'CONFIG_OPENPIC'],
if_true: files('openpic_kvm.c'))
specific_ss.add(when: 'CONFIG_POWERNV', if_true: files('xics_pnv.c', 'pnv_xive.c', 'pnv_xive2.c'))
specific_ss.add(when: 'CONFIG_PPC_UIC', if_true: files('ppc-uic.c'))
-specific_ss.add(when: 'CONFIG_RASPI', if_true: files('bcm2835_ic.c', 'bcm2836_control.c'))
specific_ss.add(when: 'CONFIG_RX_ICU', if_true: files('rx_icu.c'))
specific_ss.add(when: 'CONFIG_S390_FLIC', if_true: files('s390_flic.c'))
specific_ss.add(when: 'CONFIG_S390_FLIC_KVM', if_true: files('s390_flic_kvm.c'))
--
2.31.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 7/7] hw/usb: Mark the XLNX_VERSAL-related files as target-independent
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
` (5 preceding siblings ...)
2023-01-12 13:49 ` [PATCH 6/7] hw/intc: Mark more interrupt-controller files as target independent Thomas Huth
@ 2023-01-12 13:49 ` Thomas Huth
2023-01-13 19:20 ` [PATCH 0/7] Trivial: Mark some more files as target-independant Richard Henderson
2023-01-16 16:58 ` Laurent Vivier
8 siblings, 0 replies; 10+ messages in thread
From: Thomas Huth @ 2023-01-12 13:49 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-arm, qemu-trivial
Seems like there is nothing target-specific in here, so these files
can be moved to softmmu_ss to avoid that they get compiled twice
(once for qemu-system-arm and once for qemu-system-aarch64).
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/usb/meson.build | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/usb/meson.build b/hw/usb/meson.build
index 793df42e21..bdf34cbd3e 100644
--- a/hw/usb/meson.build
+++ b/hw/usb/meson.build
@@ -30,8 +30,8 @@ softmmu_ss.add(when: 'CONFIG_TUSB6010', if_true: files('tusb6010.c'))
softmmu_ss.add(when: 'CONFIG_IMX', if_true: files('chipidea.c'))
softmmu_ss.add(when: 'CONFIG_IMX_USBPHY', if_true: files('imx-usb-phy.c'))
softmmu_ss.add(when: 'CONFIG_VT82C686', if_true: files('vt82c686-uhci-pci.c'))
-specific_ss.add(when: 'CONFIG_XLNX_VERSAL', if_true: files('xlnx-versal-usb2-ctrl-regs.c'))
-specific_ss.add(when: 'CONFIG_XLNX_USB_SUBSYS', if_true: files('xlnx-usb-subsystem.c'))
+softmmu_ss.add(when: 'CONFIG_XLNX_VERSAL', if_true: files('xlnx-versal-usb2-ctrl-regs.c'))
+softmmu_ss.add(when: 'CONFIG_XLNX_USB_SUBSYS', if_true: files('xlnx-usb-subsystem.c'))
# emulated usb devices
softmmu_ss.add(when: 'CONFIG_USB', if_true: files('dev-hub.c'))
--
2.31.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 0/7] Trivial: Mark some more files as target-independant
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
` (6 preceding siblings ...)
2023-01-12 13:49 ` [PATCH 7/7] hw/usb: Mark the XLNX_VERSAL-related files as target-independent Thomas Huth
@ 2023-01-13 19:20 ` Richard Henderson
2023-01-16 16:58 ` Laurent Vivier
8 siblings, 0 replies; 10+ messages in thread
From: Richard Henderson @ 2023-01-13 19:20 UTC (permalink / raw)
To: Thomas Huth, qemu-devel; +Cc: qemu-arm, qemu-trivial
On 1/12/23 05:49, Thomas Huth wrote:
> Philippe Mathieu-Daudé (4):
> hw/display: Move omap_lcdc.c out of target-specific source set
> hw/intc: Move some files out of the target-specific source set
> hw/tpm: Move tpm_ppi.c out of target-specific source set
> hw/arm: Move various units to softmmu_ss[]
>
> Thomas Huth (3):
> hw/cpu: Mark arm11 and realview mpcore as target-independent code
> hw/intc: Mark more interrupt-controller files as target independent
> hw/usb: Mark the XLNX_VERSAL-related files as target-independent
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 0/7] Trivial: Mark some more files as target-independant
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
` (7 preceding siblings ...)
2023-01-13 19:20 ` [PATCH 0/7] Trivial: Mark some more files as target-independant Richard Henderson
@ 2023-01-16 16:58 ` Laurent Vivier
8 siblings, 0 replies; 10+ messages in thread
From: Laurent Vivier @ 2023-01-16 16:58 UTC (permalink / raw)
To: Thomas Huth, qemu-devel; +Cc: qemu-arm, qemu-trivial
Le 12/01/2023 à 14:49, Thomas Huth a écrit :
> Here's a collection of low-hanging fruits to mark some more files
> as target-independent (so that they do not have to be compiled twice,
> once for qemu-system-arm and once for qemu-system-aarch64).
>
> Philippe's patches have been on the list before, but I slightly
> modified some of them (like fixing typos in the subject etc.).
> My patches are new.
>
> Philippe Mathieu-Daudé (4):
> hw/display: Move omap_lcdc.c out of target-specific source set
> hw/intc: Move some files out of the target-specific source set
> hw/tpm: Move tpm_ppi.c out of target-specific source set
> hw/arm: Move various units to softmmu_ss[]
>
> Thomas Huth (3):
> hw/cpu: Mark arm11 and realview mpcore as target-independent code
> hw/intc: Mark more interrupt-controller files as target independent
> hw/usb: Mark the XLNX_VERSAL-related files as target-independent
>
> hw/arm/meson.build | 11 +++++++----
> hw/cpu/meson.build | 4 ++--
> hw/display/meson.build | 2 +-
> hw/intc/meson.build | 12 ++++++------
> hw/tpm/meson.build | 4 ++--
> hw/usb/meson.build | 4 ++--
> 6 files changed, 20 insertions(+), 17 deletions(-)
>
Series applied to my trivial-patches branch.
Thanks,
Laurent
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2023-01-16 16:58 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-12 13:49 [PATCH 0/7] Trivial: Mark some more files as target-independant Thomas Huth
2023-01-12 13:49 ` [PATCH 1/7] hw/display: Move omap_lcdc.c out of target-specific source set Thomas Huth
2023-01-12 13:49 ` [PATCH 2/7] hw/intc: Move some files out of the " Thomas Huth
2023-01-12 13:49 ` [PATCH 3/7] hw/tpm: Move tpm_ppi.c out of " Thomas Huth
2023-01-12 13:49 ` [PATCH 4/7] hw/arm: Move various units to softmmu_ss[] Thomas Huth
2023-01-12 13:49 ` [PATCH 5/7] hw/cpu: Mark arm11 and realview mpcore as target-independent code Thomas Huth
2023-01-12 13:49 ` [PATCH 6/7] hw/intc: Mark more interrupt-controller files as target independent Thomas Huth
2023-01-12 13:49 ` [PATCH 7/7] hw/usb: Mark the XLNX_VERSAL-related files as target-independent Thomas Huth
2023-01-13 19:20 ` [PATCH 0/7] Trivial: Mark some more files as target-independant Richard Henderson
2023-01-16 16:58 ` Laurent Vivier
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).