[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 07/10] pseries: do not require CONFIG_USB
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH 07/10] pseries: do not require CONFIG_USB |
Date: |
Fri, 23 Feb 2024 18:47:38 +0100 |
On Fri, Feb 23, 2024 at 6:33 PM Philippe Mathieu-Daudé
<philmd@linaro.org> wrote:
>
> On 23/2/24 13:44, Paolo Bonzini wrote:
> > With --without-default-devices it is possible to build a binary that
> > does not include any USB host controller and therefore that does not
> > include the code guarded by CONFIG_USB. While the simpler creation
> > functions such as usb_create_simple can be inlined, this is not true
> > of usb_bus_find(). Remove it, replacing it with a search of the single
> > USB bus on the machine.
> >
> > Suggested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> > ---
> > hw/ppc/spapr.c | 7 +++----
> > hw/ppc/Kconfig | 1 +
> > 2 files changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> > index 0d72d286d80..44d339982da 100644
> > --- a/hw/ppc/spapr.c
> > +++ b/hw/ppc/spapr.c
> > @@ -3024,10 +3024,9 @@ static void spapr_machine_init(MachineState *machine)
> > }
> >
> > if (has_vga) {
>
> Pre-existing, don't we want defaults_enabled() instead of has_vga here?
Yeah, but I am not sure it can be changed since pseries machine types
are versioned.
Paolo