qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] default-configs: add CONFIG_PARALLEL to hppa-softmmu
@ 2020-01-29 18:28 Alex Bennée
  2020-01-29 18:59 ` Philippe Mathieu-Daudé
  0 siblings, 1 reply; 2+ messages in thread
From: Alex Bennée @ 2020-01-29 18:28 UTC (permalink / raw)
  To: qemu-devel; +Cc: Alex Bennée, richard.henderson

The PARISC Lasi chipset emulation requires some of the common parallel
support and fails to build on a --without-default-devices. I am
assuming the chipset emulation is non-optional for the HPPA case.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 default-configs/hppa-softmmu.mak | 1 +
 1 file changed, 1 insertion(+)

diff --git a/default-configs/hppa-softmmu.mak b/default-configs/hppa-softmmu.mak
index b64c5eb3ff..9083ce8634 100644
--- a/default-configs/hppa-softmmu.mak
+++ b/default-configs/hppa-softmmu.mak
@@ -3,6 +3,7 @@
 # Uncomment the following lines to disable these optional devices:
 #
 #CONFIG_PCI_DEVICES=n
+CONFIG_PARALLEL=y
 
 # Boards:
 #
-- 
2.20.1



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] default-configs: add CONFIG_PARALLEL to hppa-softmmu
  2020-01-29 18:28 [PATCH] default-configs: add CONFIG_PARALLEL to hppa-softmmu Alex Bennée
@ 2020-01-29 18:59 ` Philippe Mathieu-Daudé
  0 siblings, 0 replies; 2+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-01-29 18:59 UTC (permalink / raw)
  To: Alex Bennée, qemu-devel; +Cc: richard.henderson

Hi Alex,

On 1/29/20 7:28 PM, Alex Bennée wrote:
> The PARISC Lasi chipset emulation requires some of the common parallel
> support and fails to build on a --without-default-devices. I am
> assuming the chipset emulation is non-optional for the HPPA case.
 >

Fixes: 376b851909d

> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>   default-configs/hppa-softmmu.mak | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/default-configs/hppa-softmmu.mak b/default-configs/hppa-softmmu.mak
> index b64c5eb3ff..9083ce8634 100644
> --- a/default-configs/hppa-softmmu.mak
> +++ b/default-configs/hppa-softmmu.mak
> @@ -3,6 +3,7 @@
>   # Uncomment the following lines to disable these optional devices:
>   #
>   #CONFIG_PCI_DEVICES=n
> +CONFIG_PARALLEL=y

We now use Kconfig :)

CONFIG_DINO=y pulls from hw/hppa/Kconfig:

config DINO
     bool
     imply PCI_DEVICES
     imply E1000_PCI
     imply VIRTIO_VGA
     select PCI
     select SERIAL
     select ISA_BUS
     select I8259
     select IDE_CMD646
     select MC146818RTC
     select LSI_SCSI_PCI
     select LASI_82596
     select LASIPS2
     select ARTIST

The fix is:

-- >8 --
diff --git a/hw/hppa/Kconfig b/hw/hppa/Kconfig
index 82178c7dcb..22948db025 100644
--- a/hw/hppa/Kconfig
+++ b/hw/hppa/Kconfig
@@ -12,4 +12,5 @@ config DINO
      select LSI_SCSI_PCI
      select LASI_82596
      select LASIPS2
+    select PARALLEL
      select ARTIST
---

Btw I suggested Sven to use a different config for LASI:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg667945.html



^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-01-29 19:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-01-29 18:28 [PATCH] default-configs: add CONFIG_PARALLEL to hppa-softmmu Alex Bennée
2020-01-29 18:59 ` Philippe Mathieu-Daudé

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).