* [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices
@ 2019-05-14 10:00 Thomas Huth
2019-05-14 10:00 ` [Qemu-devel] [PATCH 1/4] hw/core: Add a config switch for the "register" device Thomas Huth
` (4 more replies)
0 siblings, 5 replies; 17+ messages in thread
From: Thomas Huth @ 2019-05-14 10:00 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: qemu-trivial, qemu-arm, philmd
Here are some more Kconfig patches that introduce proper config
switches for some devices in the hw/core and hw/misc directories.
Thomas Huth (4):
hw/core: Add a config switch for the "register" device
hw/core: Add a config switch for the "or-irq" device
hw/core: Add a config switch for the "split-irq" device
hw/misc: Add a config switch for the "unimplemented" device
hw/arm/Kconfig | 12 ++++++++++++
hw/core/Kconfig | 9 +++++++++
hw/core/Makefile.objs | 6 +++---
hw/dma/Kconfig | 1 +
hw/microblaze/Kconfig | 1 +
hw/misc/Kconfig | 3 +++
hw/misc/Makefile.objs | 2 +-
hw/pci-host/Kconfig | 3 ++-
hw/sparc64/Kconfig | 1 +
hw/timer/Kconfig | 1 +
10 files changed, 34 insertions(+), 5 deletions(-)
--
2.21.0
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 1/4] hw/core: Add a config switch for the "register" device
2019-05-14 10:00 [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices Thomas Huth
@ 2019-05-14 10:00 ` Thomas Huth
2019-05-14 10:31 ` Philippe Mathieu-Daudé
2019-05-14 10:00 ` [Qemu-devel] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device Thomas Huth
` (3 subsequent siblings)
4 siblings, 1 reply; 17+ messages in thread
From: Thomas Huth @ 2019-05-14 10:00 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: qemu-trivial, qemu-arm, philmd
The "register" device is only used by certain machines. Let's add
a proper config switch for it so that it only gets compiled when we
really need it.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/core/Kconfig | 3 +++
hw/core/Makefile.objs | 2 +-
hw/dma/Kconfig | 1 +
hw/timer/Kconfig | 1 +
4 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/hw/core/Kconfig b/hw/core/Kconfig
index c2a1ae8122..d11920fcb3 100644
--- a/hw/core/Kconfig
+++ b/hw/core/Kconfig
@@ -9,3 +9,6 @@ config FITLOADER
config PLATFORM_BUS
bool
+
+config REGISTER
+ bool
diff --git a/hw/core/Makefile.objs b/hw/core/Makefile.objs
index a799c83815..d493a051ee 100644
--- a/hw/core/Makefile.objs
+++ b/hw/core/Makefile.objs
@@ -16,7 +16,7 @@ common-obj-$(CONFIG_SOFTMMU) += machine.o
common-obj-$(CONFIG_SOFTMMU) += loader.o
common-obj-$(CONFIG_FITLOADER) += loader-fit.o
common-obj-$(CONFIG_SOFTMMU) += qdev-properties-system.o
-common-obj-$(CONFIG_SOFTMMU) += register.o
+common-obj-$(CONFIG_REGISTER) += register.o
common-obj-$(CONFIG_SOFTMMU) += or-irq.o
common-obj-$(CONFIG_SOFTMMU) += split-irq.o
common-obj-$(CONFIG_PLATFORM_BUS) += platform-bus.o
diff --git a/hw/dma/Kconfig b/hw/dma/Kconfig
index 751dec5426..5c61b67bc0 100644
--- a/hw/dma/Kconfig
+++ b/hw/dma/Kconfig
@@ -16,6 +16,7 @@ config I8257
config ZYNQ_DEVCFG
bool
+ select REGISTER
config STP2000
bool
diff --git a/hw/timer/Kconfig b/hw/timer/Kconfig
index 51921eb63f..f575481210 100644
--- a/hw/timer/Kconfig
+++ b/hw/timer/Kconfig
@@ -36,6 +36,7 @@ config TWL92230
config XLNX_ZYNQMP
bool
+ select REGISTER
config ALTERA_TIMER
bool
--
2.21.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device
2019-05-14 10:00 [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices Thomas Huth
2019-05-14 10:00 ` [Qemu-devel] [PATCH 1/4] hw/core: Add a config switch for the "register" device Thomas Huth
@ 2019-05-14 10:00 ` Thomas Huth
2019-05-14 10:06 ` [Qemu-devel] [Qemu-arm] " Peter Maydell
2019-05-14 10:35 ` [Qemu-devel] " Philippe Mathieu-Daudé
2019-05-14 10:00 ` [Qemu-devel] [PATCH 3/4] hw/core: Add a config switch for the "split-irq" device Thomas Huth
` (2 subsequent siblings)
4 siblings, 2 replies; 17+ messages in thread
From: Thomas Huth @ 2019-05-14 10:00 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: qemu-trivial, qemu-arm, philmd
The "or-irq" device is only used by certain machines. Let's add
a proper config switch for it so that it only gets compiled when we
really need it.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/arm/Kconfig | 2 ++
hw/core/Kconfig | 3 +++
hw/core/Makefile.objs | 2 +-
hw/pci-host/Kconfig | 3 ++-
4 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
index af8cffde9c..0bb3bbe9d3 100644
--- a/hw/arm/Kconfig
+++ b/hw/arm/Kconfig
@@ -277,6 +277,7 @@ config RASPI
config STM32F205_SOC
bool
select ARM_V7M
+ select OR_IRQ
select STM32F2XX_TIMER
select STM32F2XX_USART
select STM32F2XX_SYSCFG
@@ -424,6 +425,7 @@ config ARMSSE
select IOTKIT_SECCTL
select IOTKIT_SYSCTL
select IOTKIT_SYSINFO
+ select OR_IRQ
select TZ_MPC
select TZ_MSC
select TZ_PPC
diff --git a/hw/core/Kconfig b/hw/core/Kconfig
index d11920fcb3..984143456a 100644
--- a/hw/core/Kconfig
+++ b/hw/core/Kconfig
@@ -7,6 +7,9 @@ config PTIMER
config FITLOADER
bool
+config OR_IRQ
+ bool
+
config PLATFORM_BUS
bool
diff --git a/hw/core/Makefile.objs b/hw/core/Makefile.objs
index d493a051ee..dd2c2ca812 100644
--- a/hw/core/Makefile.objs
+++ b/hw/core/Makefile.objs
@@ -17,7 +17,7 @@ common-obj-$(CONFIG_SOFTMMU) += loader.o
common-obj-$(CONFIG_FITLOADER) += loader-fit.o
common-obj-$(CONFIG_SOFTMMU) += qdev-properties-system.o
common-obj-$(CONFIG_REGISTER) += register.o
-common-obj-$(CONFIG_SOFTMMU) += or-irq.o
+common-obj-$(CONFIG_OR_IRQ) += or-irq.o
common-obj-$(CONFIG_SOFTMMU) += split-irq.o
common-obj-$(CONFIG_PLATFORM_BUS) += platform-bus.o
common-obj-$(CONFIG_SOFTMMU) += generic-loader.o
diff --git a/hw/pci-host/Kconfig b/hw/pci-host/Kconfig
index 8c16d96b3f..1edc1a31d4 100644
--- a/hw/pci-host/Kconfig
+++ b/hw/pci-host/Kconfig
@@ -2,8 +2,9 @@ config PAM
bool
config PREP_PCI
- select PCI
bool
+ select PCI
+ select OR_IRQ
config GRACKLE_PCI
select PCI
--
2.21.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 3/4] hw/core: Add a config switch for the "split-irq" device
2019-05-14 10:00 [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices Thomas Huth
2019-05-14 10:00 ` [Qemu-devel] [PATCH 1/4] hw/core: Add a config switch for the "register" device Thomas Huth
2019-05-14 10:00 ` [Qemu-devel] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device Thomas Huth
@ 2019-05-14 10:00 ` Thomas Huth
2019-05-14 10:00 ` [Qemu-devel] [PATCH 4/4] hw/misc: Add a config switch for the "unimplemented" device Thomas Huth
2019-05-14 10:07 ` [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices Paolo Bonzini
4 siblings, 0 replies; 17+ messages in thread
From: Thomas Huth @ 2019-05-14 10:00 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: qemu-trivial, qemu-arm, philmd
The "split-irq" device is currently only used by machines that use
CONFIG_ARMSSE. Let's add a proper CONFIG_SPLIT_IRQ switch for this
so that it only gets compiled when we really need it.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/arm/Kconfig | 1 +
hw/core/Kconfig | 3 +++
hw/core/Makefile.objs | 2 +-
3 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
index 0bb3bbe9d3..ac1e94f63a 100644
--- a/hw/arm/Kconfig
+++ b/hw/arm/Kconfig
@@ -426,6 +426,7 @@ config ARMSSE
select IOTKIT_SYSCTL
select IOTKIT_SYSINFO
select OR_IRQ
+ select SPLIT_IRQ
select TZ_MPC
select TZ_MSC
select TZ_PPC
diff --git a/hw/core/Kconfig b/hw/core/Kconfig
index 984143456a..fffb3d62b2 100644
--- a/hw/core/Kconfig
+++ b/hw/core/Kconfig
@@ -15,3 +15,6 @@ config PLATFORM_BUS
config REGISTER
bool
+
+config SPLIT_IRQ
+ bool
diff --git a/hw/core/Makefile.objs b/hw/core/Makefile.objs
index dd2c2ca812..d8c908da14 100644
--- a/hw/core/Makefile.objs
+++ b/hw/core/Makefile.objs
@@ -18,7 +18,7 @@ common-obj-$(CONFIG_FITLOADER) += loader-fit.o
common-obj-$(CONFIG_SOFTMMU) += qdev-properties-system.o
common-obj-$(CONFIG_REGISTER) += register.o
common-obj-$(CONFIG_OR_IRQ) += or-irq.o
-common-obj-$(CONFIG_SOFTMMU) += split-irq.o
+common-obj-$(CONFIG_SPLIT_IRQ) += split-irq.o
common-obj-$(CONFIG_PLATFORM_BUS) += platform-bus.o
common-obj-$(CONFIG_SOFTMMU) += generic-loader.o
common-obj-$(CONFIG_SOFTMMU) += null-machine.o
--
2.21.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 4/4] hw/misc: Add a config switch for the "unimplemented" device
2019-05-14 10:00 [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices Thomas Huth
` (2 preceding siblings ...)
2019-05-14 10:00 ` [Qemu-devel] [PATCH 3/4] hw/core: Add a config switch for the "split-irq" device Thomas Huth
@ 2019-05-14 10:00 ` Thomas Huth
2019-05-14 10:08 ` [Qemu-devel] [Qemu-arm] " Peter Maydell
2019-05-14 10:07 ` [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices Paolo Bonzini
4 siblings, 1 reply; 17+ messages in thread
From: Thomas Huth @ 2019-05-14 10:00 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: qemu-trivial, qemu-arm, philmd
The device is only used by certain Arm boards. Now that we have
fine-grained Kconfig for these machines, too, we can enable the
"unimplemented" devices only for the machines that really need it.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/arm/Kconfig | 9 +++++++++
hw/microblaze/Kconfig | 1 +
hw/misc/Kconfig | 3 +++
hw/misc/Makefile.objs | 2 +-
hw/sparc64/Kconfig | 1 +
5 files changed, 15 insertions(+), 1 deletion(-)
diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
index ac1e94f63a..7e261f5d73 100644
--- a/hw/arm/Kconfig
+++ b/hw/arm/Kconfig
@@ -201,6 +201,7 @@ config STELLARIS
select SSI_SD
select STELLARIS_INPUT
select STELLARIS_ENET # ethernet
+ select UNIMP
config STRONGARM
bool
@@ -267,6 +268,7 @@ config ALLWINNER_A10
select ALLWINNER_A10_PIC
select ALLWINNER_EMAC
select SERIAL
+ select UNIMP
config RASPI
bool
@@ -304,6 +306,7 @@ config XLNX_VERSAL
select PL011
select CADENCE
select VIRTIO_MMIO
+ select UNIMP
config FSL_IMX25
bool
@@ -339,6 +342,7 @@ config ASPEED_SOC
select SSI_M25P80
select TMP105
select TMP421
+ select UNIMP
config MPS2
bool
@@ -360,6 +364,7 @@ config FSL_IMX7
select IMX_I2C
select PCI_EXPRESS_DESIGNWARE
select SDHCI
+ select UNIMP
config ARM_SMMUV3
bool
@@ -371,6 +376,7 @@ config FSL_IMX6UL
select IMX_FEC
select IMX_I2C
select SDHCI
+ select UNIMP
config MICROBIT
bool
@@ -380,6 +386,7 @@ config NRF51_SOC
bool
select I2C
select ARM_V7M
+ select UNIMP
config EMCRAFT_SF2
bool
@@ -392,6 +399,7 @@ config MSF2
select PTIMER
select SERIAL
select SSI
+ select UNIMP
config ZAURUS
bool
@@ -430,6 +438,7 @@ config ARMSSE
select TZ_MPC
select TZ_MSC
select TZ_PPC
+ select UNIMP
config ARMSSE_CPUID
bool
diff --git a/hw/microblaze/Kconfig b/hw/microblaze/Kconfig
index c4dc120973..e2697ced9c 100644
--- a/hw/microblaze/Kconfig
+++ b/hw/microblaze/Kconfig
@@ -4,6 +4,7 @@ config PETALOGIX_S3ADSP1800
select XILINX
select XILINX_AXI
select XILINX_ETHLITE
+ select UNIMP
config PETALOGIX_ML605
bool
diff --git a/hw/misc/Kconfig b/hw/misc/Kconfig
index 385e1b0cec..51754bb47c 100644
--- a/hw/misc/Kconfig
+++ b/hw/misc/Kconfig
@@ -117,4 +117,7 @@ config AUX
bool
select I2C
+config UNIMP
+ bool
+
source macio/Kconfig
diff --git a/hw/misc/Makefile.objs b/hw/misc/Makefile.objs
index c71e07ae35..7a0902c76f 100644
--- a/hw/misc/Makefile.objs
+++ b/hw/misc/Makefile.objs
@@ -9,7 +9,7 @@ common-obj-$(CONFIG_PCI_TESTDEV) += pci-testdev.o
common-obj-$(CONFIG_EDU) += edu.o
common-obj-$(CONFIG_PCA9552) += pca9552.o
-common-obj-y += unimp.o
+common-obj-$(CONFIG_UNIMP) += unimp.o
common-obj-$(CONFIG_FW_CFG_DMA) += vmcoreinfo.o
# ARM devices
diff --git a/hw/sparc64/Kconfig b/hw/sparc64/Kconfig
index d4d76a89be..f9f8b0f73a 100644
--- a/hw/sparc64/Kconfig
+++ b/hw/sparc64/Kconfig
@@ -17,3 +17,4 @@ config NIAGARA
bool
select EMPTY_SLOT
select SUN4V_RTC
+ select UNIMP
--
2.21.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [Qemu-arm] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device
2019-05-14 10:00 ` [Qemu-devel] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device Thomas Huth
@ 2019-05-14 10:06 ` Peter Maydell
2019-05-14 10:25 ` Philippe Mathieu-Daudé
2019-05-14 10:35 ` [Qemu-devel] " Philippe Mathieu-Daudé
1 sibling, 1 reply; 17+ messages in thread
From: Peter Maydell @ 2019-05-14 10:06 UTC (permalink / raw)
To: Thomas Huth
Cc: QEMU Trivial, Paolo Bonzini, qemu-arm,
Philippe Mathieu-Daudé,
QEMU Developers
On Tue, 14 May 2019 at 11:00, Thomas Huth <thuth@redhat.com> wrote:
>
> The "or-irq" device is only used by certain machines. Let's add
> a proper config switch for it so that it only gets compiled when we
> really need it.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> hw/arm/Kconfig | 2 ++
> hw/core/Kconfig | 3 +++
> hw/core/Makefile.objs | 2 +-
> hw/pci-host/Kconfig | 3 ++-
> 4 files changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
> index af8cffde9c..0bb3bbe9d3 100644
> --- a/hw/arm/Kconfig
> +++ b/hw/arm/Kconfig
> @@ -277,6 +277,7 @@ config RASPI
> config STM32F205_SOC
> bool
> select ARM_V7M
> + select OR_IRQ
> select STM32F2XX_TIMER
> select STM32F2XX_USART
> select STM32F2XX_SYSCFG
> @@ -424,6 +425,7 @@ config ARMSSE
> select IOTKIT_SECCTL
> select IOTKIT_SYSCTL
> select IOTKIT_SYSINFO
> + select OR_IRQ
> select TZ_MPC
> select TZ_MSC
> select TZ_PPC
In cases like this where a device is used both by
an SoC and also directly by the board code that uses
that SoC, should we put the select OR_IRQ only in
the SoC's config, or also in the board model's config
(ie, in "config MPS2" as well as "config ARMSSE") ?
thanks
-- PMM
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices
2019-05-14 10:00 [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices Thomas Huth
` (3 preceding siblings ...)
2019-05-14 10:00 ` [Qemu-devel] [PATCH 4/4] hw/misc: Add a config switch for the "unimplemented" device Thomas Huth
@ 2019-05-14 10:07 ` Paolo Bonzini
4 siblings, 0 replies; 17+ messages in thread
From: Paolo Bonzini @ 2019-05-14 10:07 UTC (permalink / raw)
To: Thomas Huth, qemu-devel; +Cc: qemu-trivial, qemu-arm, philmd
On 14/05/19 12:00, Thomas Huth wrote:
> Here are some more Kconfig patches that introduce proper config
> switches for some devices in the hw/core and hw/misc directories.
>
> Thomas Huth (4):
> hw/core: Add a config switch for the "register" device
> hw/core: Add a config switch for the "or-irq" device
> hw/core: Add a config switch for the "split-irq" device
> hw/misc: Add a config switch for the "unimplemented" device
>
> hw/arm/Kconfig | 12 ++++++++++++
> hw/core/Kconfig | 9 +++++++++
> hw/core/Makefile.objs | 6 +++---
> hw/dma/Kconfig | 1 +
> hw/microblaze/Kconfig | 1 +
> hw/misc/Kconfig | 3 +++
> hw/misc/Makefile.objs | 2 +-
> hw/pci-host/Kconfig | 3 ++-
> hw/sparc64/Kconfig | 1 +
> hw/timer/Kconfig | 1 +
> 10 files changed, 34 insertions(+), 5 deletions(-)
>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Paolo
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [Qemu-arm] [PATCH 4/4] hw/misc: Add a config switch for the "unimplemented" device
2019-05-14 10:00 ` [Qemu-devel] [PATCH 4/4] hw/misc: Add a config switch for the "unimplemented" device Thomas Huth
@ 2019-05-14 10:08 ` Peter Maydell
2019-05-14 10:43 ` Thomas Huth
0 siblings, 1 reply; 17+ messages in thread
From: Peter Maydell @ 2019-05-14 10:08 UTC (permalink / raw)
To: Thomas Huth
Cc: QEMU Trivial, Paolo Bonzini, qemu-arm,
Philippe Mathieu-Daudé,
QEMU Developers
On Tue, 14 May 2019 at 11:02, Thomas Huth <thuth@redhat.com> wrote:
>
> The device is only used by certain Arm boards. Now that we have
> fine-grained Kconfig for these machines, too, we can enable the
> "unimplemented" devices only for the machines that really need it.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> hw/arm/Kconfig | 9 +++++++++
> hw/microblaze/Kconfig | 1 +
> hw/misc/Kconfig | 3 +++
> hw/misc/Makefile.objs | 2 +-
> hw/sparc64/Kconfig | 1 +
> 5 files changed, 15 insertions(+), 1 deletion(-)
Commit message says "only Arm" but code change changes
sparc and microblaze Kconfigs too :-)
thanks
-- PMM
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [Qemu-arm] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device
2019-05-14 10:06 ` [Qemu-devel] [Qemu-arm] " Peter Maydell
@ 2019-05-14 10:25 ` Philippe Mathieu-Daudé
2019-05-14 10:36 ` Thomas Huth
2019-05-14 10:40 ` Philippe Mathieu-Daudé
0 siblings, 2 replies; 17+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-14 10:25 UTC (permalink / raw)
To: Peter Maydell, Thomas Huth
Cc: QEMU Trivial, Paolo Bonzini, qemu-arm, QEMU Developers
On 5/14/19 12:06 PM, Peter Maydell wrote:
> On Tue, 14 May 2019 at 11:00, Thomas Huth <thuth@redhat.com> wrote:
>>
>> The "or-irq" device is only used by certain machines. Let's add
>> a proper config switch for it so that it only gets compiled when we
>> really need it.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>> hw/arm/Kconfig | 2 ++
>> hw/core/Kconfig | 3 +++
>> hw/core/Makefile.objs | 2 +-
>> hw/pci-host/Kconfig | 3 ++-
>> 4 files changed, 8 insertions(+), 2 deletions(-)
>>
>> diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
>> index af8cffde9c..0bb3bbe9d3 100644
>> --- a/hw/arm/Kconfig
>> +++ b/hw/arm/Kconfig
>> @@ -277,6 +277,7 @@ config RASPI
>> config STM32F205_SOC
>> bool
>> select ARM_V7M
>> + select OR_IRQ
>> select STM32F2XX_TIMER
>> select STM32F2XX_USART
>> select STM32F2XX_SYSCFG
>> @@ -424,6 +425,7 @@ config ARMSSE
>> select IOTKIT_SECCTL
>> select IOTKIT_SYSCTL
>> select IOTKIT_SYSINFO
>> + select OR_IRQ
>> select TZ_MPC
>> select TZ_MSC
>> select TZ_PPC
>
> In cases like this where a device is used both by
> an SoC and also directly by the board code that uses
> that SoC, should we put the select OR_IRQ only in
> the SoC's config, or also in the board model's config
> (ie, in "config MPS2" as well as "config ARMSSE") ?
Someone should be able to work on the board without having to look at
the SoC code/config, so both :) The idea of Kconfig is you only worry
about a specific device, and the qgraph sort the rest out.
So having in both place is safer, and helps to visualize dependencies in
the graph tree (I'm slowly working on this feature to help new-comer to
understand model dependencies).
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 1/4] hw/core: Add a config switch for the "register" device
2019-05-14 10:00 ` [Qemu-devel] [PATCH 1/4] hw/core: Add a config switch for the "register" device Thomas Huth
@ 2019-05-14 10:31 ` Philippe Mathieu-Daudé
2019-05-14 10:38 ` Thomas Huth
0 siblings, 1 reply; 17+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-14 10:31 UTC (permalink / raw)
To: Thomas Huth, Paolo Bonzini, qemu-devel; +Cc: qemu-trivial, qemu-arm
On 5/14/19 12:00 PM, Thomas Huth wrote:
> The "register" device is only used by certain machines. Let's add
> a proper config switch for it so that it only gets compiled when we
> really need it.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> hw/core/Kconfig | 3 +++
> hw/core/Makefile.objs | 2 +-
> hw/dma/Kconfig | 1 +
> hw/timer/Kconfig | 1 +
> 4 files changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/hw/core/Kconfig b/hw/core/Kconfig
> index c2a1ae8122..d11920fcb3 100644
> --- a/hw/core/Kconfig
> +++ b/hw/core/Kconfig
> @@ -9,3 +9,6 @@ config FITLOADER
>
> config PLATFORM_BUS
> bool
> +
> +config REGISTER
> + bool
> diff --git a/hw/core/Makefile.objs b/hw/core/Makefile.objs
> index a799c83815..d493a051ee 100644
> --- a/hw/core/Makefile.objs
> +++ b/hw/core/Makefile.objs
> @@ -16,7 +16,7 @@ common-obj-$(CONFIG_SOFTMMU) += machine.o
> common-obj-$(CONFIG_SOFTMMU) += loader.o
> common-obj-$(CONFIG_FITLOADER) += loader-fit.o
> common-obj-$(CONFIG_SOFTMMU) += qdev-properties-system.o
> -common-obj-$(CONFIG_SOFTMMU) += register.o
> +common-obj-$(CONFIG_REGISTER) += register.o
> common-obj-$(CONFIG_SOFTMMU) += or-irq.o
> common-obj-$(CONFIG_SOFTMMU) += split-irq.o
> common-obj-$(CONFIG_PLATFORM_BUS) += platform-bus.o
> diff --git a/hw/dma/Kconfig b/hw/dma/Kconfig
> index 751dec5426..5c61b67bc0 100644
> --- a/hw/dma/Kconfig
> +++ b/hw/dma/Kconfig
> @@ -16,6 +16,7 @@ config I8257
>
> config ZYNQ_DEVCFG
> bool
> + select REGISTER
>
> config STP2000
> bool
> diff --git a/hw/timer/Kconfig b/hw/timer/Kconfig
> index 51921eb63f..f575481210 100644
> --- a/hw/timer/Kconfig
> +++ b/hw/timer/Kconfig
> @@ -36,6 +36,7 @@ config TWL92230
>
> config XLNX_ZYNQMP
> bool
> + select REGISTER
>
> config ALTERA_TIMER
> bool
>
Annoying, this clashes with "hw/microblaze: Kconfig cleanup" which is
already queued by Paolo:
https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg04669.html
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device
2019-05-14 10:00 ` [Qemu-devel] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device Thomas Huth
2019-05-14 10:06 ` [Qemu-devel] [Qemu-arm] " Peter Maydell
@ 2019-05-14 10:35 ` Philippe Mathieu-Daudé
2019-05-14 10:40 ` Thomas Huth
1 sibling, 1 reply; 17+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-14 10:35 UTC (permalink / raw)
To: Thomas Huth, Paolo Bonzini, qemu-devel; +Cc: qemu-trivial, qemu-arm
On 5/14/19 12:00 PM, Thomas Huth wrote:
> The "or-irq" device is only used by certain machines. Let's add
> a proper config switch for it so that it only gets compiled when we
> really need it.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> hw/arm/Kconfig | 2 ++
> hw/core/Kconfig | 3 +++
> hw/core/Makefile.objs | 2 +-
> hw/pci-host/Kconfig | 3 ++-
> 4 files changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
> index af8cffde9c..0bb3bbe9d3 100644
> --- a/hw/arm/Kconfig
> +++ b/hw/arm/Kconfig
> @@ -277,6 +277,7 @@ config RASPI
> config STM32F205_SOC
> bool
> select ARM_V7M
> + select OR_IRQ
> select STM32F2XX_TIMER
> select STM32F2XX_USART
> select STM32F2XX_SYSCFG
> @@ -424,6 +425,7 @@ config ARMSSE
> select IOTKIT_SECCTL
> select IOTKIT_SYSCTL
> select IOTKIT_SYSINFO
> + select OR_IRQ
> select TZ_MPC
> select TZ_MSC
> select TZ_PPC
You missed the MPS2* boards
> diff --git a/hw/core/Kconfig b/hw/core/Kconfig
> index d11920fcb3..984143456a 100644
> --- a/hw/core/Kconfig
> +++ b/hw/core/Kconfig
> @@ -7,6 +7,9 @@ config PTIMER
> config FITLOADER
> bool
>
> +config OR_IRQ
> + bool
> +
> config PLATFORM_BUS
> bool
>
> diff --git a/hw/core/Makefile.objs b/hw/core/Makefile.objs
> index d493a051ee..dd2c2ca812 100644
> --- a/hw/core/Makefile.objs
> +++ b/hw/core/Makefile.objs
> @@ -17,7 +17,7 @@ common-obj-$(CONFIG_SOFTMMU) += loader.o
> common-obj-$(CONFIG_FITLOADER) += loader-fit.o
> common-obj-$(CONFIG_SOFTMMU) += qdev-properties-system.o
> common-obj-$(CONFIG_REGISTER) += register.o
> -common-obj-$(CONFIG_SOFTMMU) += or-irq.o
> +common-obj-$(CONFIG_OR_IRQ) += or-irq.o
> common-obj-$(CONFIG_SOFTMMU) += split-irq.o
> common-obj-$(CONFIG_PLATFORM_BUS) += platform-bus.o
> common-obj-$(CONFIG_SOFTMMU) += generic-loader.o
> diff --git a/hw/pci-host/Kconfig b/hw/pci-host/Kconfig
> index 8c16d96b3f..1edc1a31d4 100644
> --- a/hw/pci-host/Kconfig
> +++ b/hw/pci-host/Kconfig
> @@ -2,8 +2,9 @@ config PAM
> bool
>
> config PREP_PCI
> - select PCI
> bool
> + select PCI
> + select OR_IRQ
>
> config GRACKLE_PCI
> select PCI
>
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [Qemu-arm] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device
2019-05-14 10:25 ` Philippe Mathieu-Daudé
@ 2019-05-14 10:36 ` Thomas Huth
2019-05-14 10:40 ` Philippe Mathieu-Daudé
1 sibling, 0 replies; 17+ messages in thread
From: Thomas Huth @ 2019-05-14 10:36 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Peter Maydell
Cc: QEMU Trivial, Paolo Bonzini, qemu-arm, QEMU Developers
On 14/05/2019 12.25, Philippe Mathieu-Daudé wrote:
> On 5/14/19 12:06 PM, Peter Maydell wrote:
>> On Tue, 14 May 2019 at 11:00, Thomas Huth <thuth@redhat.com> wrote:
>>>
>>> The "or-irq" device is only used by certain machines. Let's add
>>> a proper config switch for it so that it only gets compiled when we
>>> really need it.
>>>
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>> ---
>>> hw/arm/Kconfig | 2 ++
>>> hw/core/Kconfig | 3 +++
>>> hw/core/Makefile.objs | 2 +-
>>> hw/pci-host/Kconfig | 3 ++-
>>> 4 files changed, 8 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
>>> index af8cffde9c..0bb3bbe9d3 100644
>>> --- a/hw/arm/Kconfig
>>> +++ b/hw/arm/Kconfig
>>> @@ -277,6 +277,7 @@ config RASPI
>>> config STM32F205_SOC
>>> bool
>>> select ARM_V7M
>>> + select OR_IRQ
>>> select STM32F2XX_TIMER
>>> select STM32F2XX_USART
>>> select STM32F2XX_SYSCFG
>>> @@ -424,6 +425,7 @@ config ARMSSE
>>> select IOTKIT_SECCTL
>>> select IOTKIT_SYSCTL
>>> select IOTKIT_SYSINFO
>>> + select OR_IRQ
>>> select TZ_MPC
>>> select TZ_MSC
>>> select TZ_PPC
>>
>> In cases like this where a device is used both by
>> an SoC and also directly by the board code that uses
>> that SoC, should we put the select OR_IRQ only in
>> the SoC's config, or also in the board model's config
>> (ie, in "config MPS2" as well as "config ARMSSE") ?
>
> Someone should be able to work on the board without having to look at
> the SoC code/config, so both :) The idea of Kconfig is you only worry
> about a specific device, and the qgraph sort the rest out.
I don't have a strong opinion here, but likely is safer indeed to put
the switch into both sections in this case - so if one of the two ever
gets changed, the config switch is still there for the other one that
still requires it. I'll send a v2.
Thomas
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 1/4] hw/core: Add a config switch for the "register" device
2019-05-14 10:31 ` Philippe Mathieu-Daudé
@ 2019-05-14 10:38 ` Thomas Huth
2019-05-22 14:24 ` Thomas Huth
0 siblings, 1 reply; 17+ messages in thread
From: Thomas Huth @ 2019-05-14 10:38 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Paolo Bonzini, qemu-devel
Cc: qemu-trivial, qemu-arm
On 14/05/2019 12.31, Philippe Mathieu-Daudé wrote:
> On 5/14/19 12:00 PM, Thomas Huth wrote:
>> The "register" device is only used by certain machines. Let's add
>> a proper config switch for it so that it only gets compiled when we
>> really need it.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>> hw/core/Kconfig | 3 +++
>> hw/core/Makefile.objs | 2 +-
>> hw/dma/Kconfig | 1 +
>> hw/timer/Kconfig | 1 +
>> 4 files changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/hw/core/Kconfig b/hw/core/Kconfig
>> index c2a1ae8122..d11920fcb3 100644
>> --- a/hw/core/Kconfig
>> +++ b/hw/core/Kconfig
>> @@ -9,3 +9,6 @@ config FITLOADER
>>
>> config PLATFORM_BUS
>> bool
>> +
>> +config REGISTER
>> + bool
>> diff --git a/hw/core/Makefile.objs b/hw/core/Makefile.objs
>> index a799c83815..d493a051ee 100644
>> --- a/hw/core/Makefile.objs
>> +++ b/hw/core/Makefile.objs
>> @@ -16,7 +16,7 @@ common-obj-$(CONFIG_SOFTMMU) += machine.o
>> common-obj-$(CONFIG_SOFTMMU) += loader.o
>> common-obj-$(CONFIG_FITLOADER) += loader-fit.o
>> common-obj-$(CONFIG_SOFTMMU) += qdev-properties-system.o
>> -common-obj-$(CONFIG_SOFTMMU) += register.o
>> +common-obj-$(CONFIG_REGISTER) += register.o
>> common-obj-$(CONFIG_SOFTMMU) += or-irq.o
>> common-obj-$(CONFIG_SOFTMMU) += split-irq.o
>> common-obj-$(CONFIG_PLATFORM_BUS) += platform-bus.o
>> diff --git a/hw/dma/Kconfig b/hw/dma/Kconfig
>> index 751dec5426..5c61b67bc0 100644
>> --- a/hw/dma/Kconfig
>> +++ b/hw/dma/Kconfig
>> @@ -16,6 +16,7 @@ config I8257
>>
>> config ZYNQ_DEVCFG
>> bool
>> + select REGISTER
>>
>> config STP2000
>> bool
>> diff --git a/hw/timer/Kconfig b/hw/timer/Kconfig
>> index 51921eb63f..f575481210 100644
>> --- a/hw/timer/Kconfig
>> +++ b/hw/timer/Kconfig
>> @@ -36,6 +36,7 @@ config TWL92230
>>
>> config XLNX_ZYNQMP
>> bool
>> + select REGISTER
>>
>> config ALTERA_TIMER
>> bool
>>
>
> Annoying, this clashes with "hw/microblaze: Kconfig cleanup" which is
> already queued by Paolo:
> https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg04669.html
Ok, I'll wait for the merge of Paolo's next PULL request before sending
the v2 of my series.
Thomas
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [Qemu-arm] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device
2019-05-14 10:25 ` Philippe Mathieu-Daudé
2019-05-14 10:36 ` Thomas Huth
@ 2019-05-14 10:40 ` Philippe Mathieu-Daudé
1 sibling, 0 replies; 17+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-14 10:40 UTC (permalink / raw)
To: Peter Maydell, Thomas Huth
Cc: QEMU Trivial, Paolo Bonzini, qemu-arm, QEMU Developers
On 5/14/19 12:25 PM, Philippe Mathieu-Daudé wrote:
> On 5/14/19 12:06 PM, Peter Maydell wrote:
>> On Tue, 14 May 2019 at 11:00, Thomas Huth <thuth@redhat.com> wrote:
>>>
>>> The "or-irq" device is only used by certain machines. Let's add
>>> a proper config switch for it so that it only gets compiled when we
>>> really need it.
>>>
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>> ---
>>> hw/arm/Kconfig | 2 ++
>>> hw/core/Kconfig | 3 +++
>>> hw/core/Makefile.objs | 2 +-
>>> hw/pci-host/Kconfig | 3 ++-
>>> 4 files changed, 8 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
>>> index af8cffde9c..0bb3bbe9d3 100644
>>> --- a/hw/arm/Kconfig
>>> +++ b/hw/arm/Kconfig
>>> @@ -277,6 +277,7 @@ config RASPI
>>> config STM32F205_SOC
>>> bool
>>> select ARM_V7M
>>> + select OR_IRQ
>>> select STM32F2XX_TIMER
>>> select STM32F2XX_USART
>>> select STM32F2XX_SYSCFG
>>> @@ -424,6 +425,7 @@ config ARMSSE
>>> select IOTKIT_SECCTL
>>> select IOTKIT_SYSCTL
>>> select IOTKIT_SYSINFO
>>> + select OR_IRQ
>>> select TZ_MPC
>>> select TZ_MSC
>>> select TZ_PPC
>>
>> In cases like this where a device is used both by
>> an SoC and also directly by the board code that uses
>> that SoC, should we put the select OR_IRQ only in
>> the SoC's config, or also in the board model's config
>> (ie, in "config MPS2" as well as "config ARMSSE") ?
>
> Someone should be able to work on the board without having to look at
> the SoC code/config, so both :) The idea of Kconfig is you only worry
> about a specific device, and the qgraph sort the rest out.
Hypothetical example if you only use the selector in the SoC config:
If you replace the SoC OR_IRQ by a more complex device or extended one,
the board will lack the OR_IRQ selector.
Using Kconfig selectors in all place a dependency is explicit also ease
backports.
>
> So having in both place is safer, and helps to visualize dependencies in
> the graph tree (I'm slowly working on this feature to help new-comer to
> understand model dependencies).
>
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device
2019-05-14 10:35 ` [Qemu-devel] " Philippe Mathieu-Daudé
@ 2019-05-14 10:40 ` Thomas Huth
0 siblings, 0 replies; 17+ messages in thread
From: Thomas Huth @ 2019-05-14 10:40 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Paolo Bonzini, qemu-devel
Cc: qemu-trivial, qemu-arm
On 14/05/2019 12.35, Philippe Mathieu-Daudé wrote:
> On 5/14/19 12:00 PM, Thomas Huth wrote:
>> The "or-irq" device is only used by certain machines. Let's add
>> a proper config switch for it so that it only gets compiled when we
>> really need it.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>> hw/arm/Kconfig | 2 ++
>> hw/core/Kconfig | 3 +++
>> hw/core/Makefile.objs | 2 +-
>> hw/pci-host/Kconfig | 3 ++-
>> 4 files changed, 8 insertions(+), 2 deletions(-)
>>
>> diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
>> index af8cffde9c..0bb3bbe9d3 100644
>> --- a/hw/arm/Kconfig
>> +++ b/hw/arm/Kconfig
>> @@ -277,6 +277,7 @@ config RASPI
>> config STM32F205_SOC
>> bool
>> select ARM_V7M
>> + select OR_IRQ
>> select STM32F2XX_TIMER
>> select STM32F2XX_USART
>> select STM32F2XX_SYSCFG
>> @@ -424,6 +425,7 @@ config ARMSSE
>> select IOTKIT_SECCTL
>> select IOTKIT_SYSCTL
>> select IOTKIT_SYSINFO
>> + select OR_IRQ
>> select TZ_MPC
>> select TZ_MSC
>> select TZ_PPC
>
> You missed the MPS2* boards
No, the MPS2 boards "select ARMSSE", so this gets added via the above
hunk there. But as mentioned in the reply to Peter, it's likely better
to add it there, too.
Thomas
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [Qemu-arm] [PATCH 4/4] hw/misc: Add a config switch for the "unimplemented" device
2019-05-14 10:08 ` [Qemu-devel] [Qemu-arm] " Peter Maydell
@ 2019-05-14 10:43 ` Thomas Huth
0 siblings, 0 replies; 17+ messages in thread
From: Thomas Huth @ 2019-05-14 10:43 UTC (permalink / raw)
To: Peter Maydell
Cc: QEMU Trivial, Paolo Bonzini, qemu-arm,
Philippe Mathieu-Daudé,
QEMU Developers
On 14/05/2019 12.08, Peter Maydell wrote:
> On Tue, 14 May 2019 at 11:02, Thomas Huth <thuth@redhat.com> wrote:
>>
>> The device is only used by certain Arm boards. Now that we have
>> fine-grained Kconfig for these machines, too, we can enable the
>> "unimplemented" devices only for the machines that really need it.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>> hw/arm/Kconfig | 9 +++++++++
>> hw/microblaze/Kconfig | 1 +
>> hw/misc/Kconfig | 3 +++
>> hw/misc/Makefile.objs | 2 +-
>> hw/sparc64/Kconfig | 1 +
>> 5 files changed, 15 insertions(+), 1 deletion(-)
>
> Commit message says "only Arm" but code change changes
> sparc and microblaze Kconfigs too :-)
D'oh! ... I started with grep'ing for TYPE_UNIMPLEMENTED_DEVICE and only
saw Arm boards there. When I tested my patches, I noticed that I must
also add the machines that use the create_unimplemented_device()
function, but apparently forgot to fix up the commit message
accordingly. I'll fix it in v2.
Thomas
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 1/4] hw/core: Add a config switch for the "register" device
2019-05-14 10:38 ` Thomas Huth
@ 2019-05-22 14:24 ` Thomas Huth
0 siblings, 0 replies; 17+ messages in thread
From: Thomas Huth @ 2019-05-22 14:24 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Paolo Bonzini, qemu-devel
Cc: qemu-trivial, qemu-arm
On 14/05/2019 12.38, Thomas Huth wrote:
> On 14/05/2019 12.31, Philippe Mathieu-Daudé wrote:
>> On 5/14/19 12:00 PM, Thomas Huth wrote:
>>> The "register" device is only used by certain machines. Let's add
>>> a proper config switch for it so that it only gets compiled when we
>>> really need it.
>>>
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>> ---
>>> hw/core/Kconfig | 3 +++
>>> hw/core/Makefile.objs | 2 +-
>>> hw/dma/Kconfig | 1 +
>>> hw/timer/Kconfig | 1 +
>>> 4 files changed, 6 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/hw/core/Kconfig b/hw/core/Kconfig
>>> index c2a1ae8122..d11920fcb3 100644
>>> --- a/hw/core/Kconfig
>>> +++ b/hw/core/Kconfig
>>> @@ -9,3 +9,6 @@ config FITLOADER
>>>
>>> config PLATFORM_BUS
>>> bool
>>> +
>>> +config REGISTER
>>> + bool
>>> diff --git a/hw/core/Makefile.objs b/hw/core/Makefile.objs
>>> index a799c83815..d493a051ee 100644
>>> --- a/hw/core/Makefile.objs
>>> +++ b/hw/core/Makefile.objs
>>> @@ -16,7 +16,7 @@ common-obj-$(CONFIG_SOFTMMU) += machine.o
>>> common-obj-$(CONFIG_SOFTMMU) += loader.o
>>> common-obj-$(CONFIG_FITLOADER) += loader-fit.o
>>> common-obj-$(CONFIG_SOFTMMU) += qdev-properties-system.o
>>> -common-obj-$(CONFIG_SOFTMMU) += register.o
>>> +common-obj-$(CONFIG_REGISTER) += register.o
>>> common-obj-$(CONFIG_SOFTMMU) += or-irq.o
>>> common-obj-$(CONFIG_SOFTMMU) += split-irq.o
>>> common-obj-$(CONFIG_PLATFORM_BUS) += platform-bus.o
>>> diff --git a/hw/dma/Kconfig b/hw/dma/Kconfig
>>> index 751dec5426..5c61b67bc0 100644
>>> --- a/hw/dma/Kconfig
>>> +++ b/hw/dma/Kconfig
>>> @@ -16,6 +16,7 @@ config I8257
>>>
>>> config ZYNQ_DEVCFG
>>> bool
>>> + select REGISTER
>>>
>>> config STP2000
>>> bool
>>> diff --git a/hw/timer/Kconfig b/hw/timer/Kconfig
>>> index 51921eb63f..f575481210 100644
>>> --- a/hw/timer/Kconfig
>>> +++ b/hw/timer/Kconfig
>>> @@ -36,6 +36,7 @@ config TWL92230
>>>
>>> config XLNX_ZYNQMP
>>> bool
>>> + select REGISTER
>>>
>>> config ALTERA_TIMER
>>> bool
>>>
>>
>> Annoying, this clashes with "hw/microblaze: Kconfig cleanup" which is
>> already queued by Paolo:
>> https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg04669.html
>
> Ok, I'll wait for the merge of Paolo's next PULL request before sending
> the v2 of my series.
Looks like your patches were not included in Paolo's last PULL request?
Did the patches fall through the cracks?
Thomas
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2019-05-22 14:25 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-14 10:00 [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices Thomas Huth
2019-05-14 10:00 ` [Qemu-devel] [PATCH 1/4] hw/core: Add a config switch for the "register" device Thomas Huth
2019-05-14 10:31 ` Philippe Mathieu-Daudé
2019-05-14 10:38 ` Thomas Huth
2019-05-22 14:24 ` Thomas Huth
2019-05-14 10:00 ` [Qemu-devel] [PATCH 2/4] hw/core: Add a config switch for the "or-irq" device Thomas Huth
2019-05-14 10:06 ` [Qemu-devel] [Qemu-arm] " Peter Maydell
2019-05-14 10:25 ` Philippe Mathieu-Daudé
2019-05-14 10:36 ` Thomas Huth
2019-05-14 10:40 ` Philippe Mathieu-Daudé
2019-05-14 10:35 ` [Qemu-devel] " Philippe Mathieu-Daudé
2019-05-14 10:40 ` Thomas Huth
2019-05-14 10:00 ` [Qemu-devel] [PATCH 3/4] hw/core: Add a config switch for the "split-irq" device Thomas Huth
2019-05-14 10:00 ` [Qemu-devel] [PATCH 4/4] hw/misc: Add a config switch for the "unimplemented" device Thomas Huth
2019-05-14 10:08 ` [Qemu-devel] [Qemu-arm] " Peter Maydell
2019-05-14 10:43 ` Thomas Huth
2019-05-14 10:07 ` [Qemu-devel] [PATCH 0/4] Kconfig switches for core / misc devices Paolo Bonzini
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).