[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/5] hw/isa/Kconfig: PIIX4 southbridge requir
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/5] hw/isa/Kconfig: PIIX4 southbridge requires USB UHCI |
Date: |
Mon, 25 Mar 2019 05:55:20 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.0 |
On 25/03/2019 05.49, Thomas Huth wrote:
> On 16/03/2019 23.39, Philippe Mathieu-Daudé wrote:
>> This fixes when configuring with --without-default-devices:
>>
>> $ qemu-system-mips64 -M malta --nodefaults -bios /dev/null
>> qemu-system-mips64: Unknown device 'piix4-usb-uhci' for bus 'PCI'
>> Aborted (core dumped)
>>
>> (gdb) bt
>> #0 0x00007ffff5a4353f in __GI_raise (address@hidden) at
>> ../sysdeps/unix/sysv/linux/raise.c:50
>> #1 0x00007ffff5a2d895 in __GI_abort () at abort.c:79
>> #2 0x00005555558745c3 in qdev_create (address@hidden, address@hidden
>> "piix4-usb-uhci") at hw/core/qdev.c:131
>> #3 0x00005555558cb3e1 in pci_create_multifunction (address@hidden,
>> address@hidden, address@hidden, address@hidden "piix4-usb-uhci") at
>> hw/pci/pci.c:2104
>> #4 0x00005555558cb88b in pci_create_simple_multifunction (address@hidden
>> "piix4-usb-uhci", multifunction=false, address@hidden, address@hidden) at
>> hw/pci/pci.c:2126
>> #5 0x00005555558cb88b in pci_create_simple (address@hidden,
>> address@hidden, address@hidden "piix4-usb-uhci") at hw/pci/pci.c:2126
>> #6 0x000055555575ea38 in mips_malta_init (machine=0x555555ffe430) at
>> hw/mips/mips_malta.c:1392
>> #7 0x000055555587bf2b in machine_run_board_init (machine=0x555555ffe430)
>> at hw/core/machine.c:1030
>>
>> Fixes: 7c28b925b7e
>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>> ---
>> hw/isa/Kconfig | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/hw/isa/Kconfig b/hw/isa/Kconfig
>> index 57e09a0cb8..c942b47d03 100644
>> --- a/hw/isa/Kconfig
>> +++ b/hw/isa/Kconfig
>> @@ -29,6 +29,7 @@ config PIIX4
>> # For historical reasons, SuperIO devices are created in the board
>> # for PIIX4.
>> select ISA_BUS
>> + select USB_UHCI
>
> The UHCI controller is not instantiated by hw/isa/piix4.c, but by
> hw/mips/mips_malta.c as far as I can see. And you can use the "pc"
> machine also without uhci. So I think this patch is wrong, the select
> should be done by "MALTA" instead.
Ah, never mind, I mixed up piix3-usb-uhci (used by "pc") and
piix4-usb-uhci (used by "malta"). ... So this UHCI is part of the
chipset? Then I think your patch is fine.
Thomas
- [Qemu-devel] [PATCH v2 0/5] hw/Kconfig: Fixes when running ./configure --without-default-devices, Philippe Mathieu-Daudé, 2019/03/16
- [Qemu-devel] [PATCH v2 1/5] hw/isa/Kconfig: PIIX4 southbridge requires USB UHCI, Philippe Mathieu-Daudé, 2019/03/16
- [Qemu-devel] [PATCH v2 2/5] hw/isa/Kconfig: VT82C686 southbridge requires USB UHCI, Philippe Mathieu-Daudé, 2019/03/16
- [Qemu-devel] [PATCH v2 3/5] hw/isa/Kconfig: i82378 SuperIO requires PC speaker device, Philippe Mathieu-Daudé, 2019/03/16
- [Qemu-devel] [PATCH v2 4/5] hw/mips/Kconfig: Fulong 2e board requires ati-vga display device, Philippe Mathieu-Daudé, 2019/03/16
- [Qemu-devel] [PATCH v2 5/5] hw/sparc/Kconfig: SPARCstation machine requires the TCX display, Philippe Mathieu-Daudé, 2019/03/16