Re: [PATCH] default-configs: add CONFIG_PARALLEL to hppa-softmmu

[Philippe Mathieu-Daudé]

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 <address@hidden>
> ---
>   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/address@hidden/msg667945.html