Re: [Qemu-devel] [PATCH 04/10] sun4u: create VGA card after ISA bus

["Hervé Poussineau" <hpoussin@reactos.org>]

Andreas Färber a écrit :
> Am 04.01.2013 22:29, schrieb Hervé Poussineau:
>> Signed-off-by: Hervé Poussineau <hpoussin@reactos.org>
>> ---
>>  hw/sun4u.c |    3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> Doesn't look wrong to me, cc'ing Blue. I figure this is for 07/10,
> which switches to isa_register_portio_list().

Indeed.
ISA bus is created in pci_ebus_init(). VGA card registers some ISA I/O 
ports, so an ISA bus must be present.

Indeed.

> 
> Andreas
> 
>> diff --git a/hw/sun4u.c b/hw/sun4u.c
>> index cbfd217..c9150ad 100644
>> --- a/hw/sun4u.c
>> +++ b/hw/sun4u.c
>> @@ -825,11 +825,12 @@ static void sun4uv_init(MemoryRegion *address_space_mem,
>>      ivec_irqs = qemu_allocate_irqs(cpu_set_ivec_irq, cpu, IVEC_MAX);
>>      pci_bus = pci_apb_init(APB_SPECIAL_BASE, APB_MEM_BASE, ivec_irqs, &pci_bus2,
>>                             &pci_bus3, &pbm_irqs);
>> -    pci_vga_init(pci_bus);
>>  
>>      // XXX Should be pci_bus3
>>      isa_bus = pci_ebus_init(pci_bus, -1, pbm_irqs);
>>  
>> +    pci_vga_init(pci_bus);
>> +
>>      i = 0;
>>      if (hwdef->console_serial_base) {
>>          serial_mm_init(address_space_mem, hwdef->console_serial_base, 0,
>>
> 
>