Re: [PATCH] hw/hppa/Kconfig: Fix building with "configure --without-default-devices"

[Philippe Mathieu-Daudé]

On 9/2/24 20:46, BALATON Zoltan wrote:
> On Fri, 9 Feb 2024, Helge Deller wrote:
> > On 2/9/24 19:55, Thomas Huth wrote:
> > > When running "configure" with "--without-default-devices", building
> > > of qemu-system-hppa currently fails with:
> > >
> > >   /usr/bin/ld: libqemu-hppa-softmmu.fa.p/hw_hppa_machine.c.o: in 
> > > function `machine_HP_common_init_tail':
> > >   hw/hppa/machine.c:399: undefined reference to `usb_bus_find'
> > >   /usr/bin/ld: hw/hppa/machine.c:399: undefined reference to 
> > > `usb_create_simple'
> > >   /usr/bin/ld: hw/hppa/machine.c:400: undefined reference to 
> > > `usb_bus_find'
> > >   /usr/bin/ld: hw/hppa/machine.c:400: undefined reference to 
> > > `usb_create_simple'
> > >   collect2: error: ld returned 1 exit status
> > >   ninja: build stopped: subcommand failed.
> > >   make: *** [Makefile:162: run-ninja] Error 1
> > >
> > > And after fixing this, the qemu-system-hppa binary refuses to run
> > > due to the missing 'pci-ohci' and 'pci-serial' devices. Let's add
> > > the right config switches to fix these problems.
> > >
> > > Signed-off-by: Thomas Huth <thuth@redhat.com>
> > > ---
> > >   hw/hppa/Kconfig | 3 ++-
> > >   1 file changed, 2 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/hw/hppa/Kconfig b/hw/hppa/Kconfig
> > > index ff8528aaa8..124d5e9e47 100644
> > > --- a/hw/hppa/Kconfig
> > > +++ b/hw/hppa/Kconfig
> > > @@ -6,7 +6,7 @@ config HPPA_B160L
> > >       select ASTRO
> > >       select DINO
> > >       select LASI
> > > -    select SERIAL
> > > +    select SERIAL_PCI
> >
> > I think the "SERIAL" is needed too for the B160L machine.
>
> SERIAL_PCI selects SERIAL so I think it should be pulled in without 
> listing it separately but not sure what's the policy for these configs.

We prefer explicit dependencies.

SERIAL is for Lasi/Dino serial_mm_init().

Although pulling in SERIAL, SERIAL_PCI is for pci-serial*.

> Regards,
> BALATON Zoltan
>
> > Other than that,
> >
> > Acked-by: Helge Deller <deller@gmx.de>
> >
> > Thank you!
> > Helge
> >
> > >       select ISA_BUS
> > >       select I8259
> > >       select IDE_CMD646
> > > @@ -16,3 +16,4 @@ config HPPA_B160L
> > >       select LASIPS2
> > >       select PARALLEL
> > >       select ARTIST
> > > +    select USB_OHCI_PCI