On 2015/12/10 7:41, Igor Mammedov wrote:
> static void acpi_dsdt_add_virtio(Aml *scope,
> const MemMapEntry *virtio_mmio_memmap,
> - int mmio_irq, int num)
> + uint32_t mmio_irq, int num)
> {
> hwaddr base = virtio_mmio_memmap->base;
> hwaddr size = virtio_mmio_memmap->size;
> - int irq = mmio_irq;
> + uint32_t irq = mmio_irq;
> int i;
>
> for (i = 0; i < num; i++) {
> @@ -152,15 +152,15 @@ static void acpi_dsdt_add_virtio(Aml *scope,
> aml_append(crs, aml_memory32_fixed(base, size, AML_READ_WRITE));
> aml_append(crs,
> aml_interrupt(AML_CONSUMER, AML_LEVEL, AML_ACTIVE_HIGH,
> - AML_EXCLUSIVE, irq + i));
> + AML_EXCLUSIVE, &irq, 1));
This looks not right. You could move uint32_t irq = mmio_irq; into the
for loop and make it as uint32_t irq = mmio_irq + i;
Thanks,
--
Shannon
