Re: [PATCH v3 10/46] hw/hppa: use pci_init_nic_devices()

[Thomas Huth]

On 08/01/2024 21.26, David Woodhouse wrote:
> From: David Woodhouse <dwmw@amazon.co.uk>
>
> Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
> ---
>   hw/hppa/machine.c | 7 ++-----
>   1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/hw/hppa/machine.c b/hw/hppa/machine.c
> index c8da7c18d5..19d477105e 100644
> --- a/hw/hppa/machine.c
> +++ b/hw/hppa/machine.c
> @@ -338,7 +338,6 @@ static void machine_HP_common_init_tail(MachineState 
> *machine, PCIBus *pci_bus,
>       uint64_t kernel_entry = 0, kernel_low, kernel_high;
>       MemoryRegion *addr_space = get_system_memory();
>       MemoryRegion *rom_region;
> -    long i;
>       unsigned int smp_cpus = machine->smp.cpus;
>       SysBusDevice *s;
>   
> @@ -362,10 +361,8 @@ static void machine_HP_common_init_tail(MachineState *machine, PCIBus *pci_bus,
>                           qdev_get_gpio_in(lasi_dev, LASI_IRQ_LAN_HPA));
>       }
>   
> -    for (i = 0; i < nb_nics; i++) {
> -        if (!enable_lasi_lan()) {
> -            pci_nic_init_nofail(&nd_table[i], pci_bus, mc->default_nic, NULL);
> -        }
> +    if (!enable_lasi_lan()) {
> +        pci_init_nic_devices(pci_bus, mc->default_nic);
>       }
>   
>       /* BMC board: HP Powerbar SP2 Diva (with console only) */

Reviewed-by: Thomas Huth <thuth@redhat.com>