On Thu, Oct 26, 2023 at 7:23 AM Sunil V L <suni...@ventanamicro.com> wrote: > > With common function to add virtio in DSDT created now, update microvm > code also to use it instead of duplicate code. > > Suggested-by: Andrew Jones <ajo...@ventanamicro.com> > Signed-off-by: Sunil V L <suni...@ventanamicro.com>
Acked-by: Alistair Francis <alistair.fran...@wdc.com> Alistair > --- > hw/i386/acpi-microvm.c | 15 ++------------- > 1 file changed, 2 insertions(+), 13 deletions(-) > > diff --git a/hw/i386/acpi-microvm.c b/hw/i386/acpi-microvm.c > index 2909a73933..279da6b4aa 100644 > --- a/hw/i386/acpi-microvm.c > +++ b/hw/i386/acpi-microvm.c > @@ -37,6 +37,7 @@ > #include "hw/pci/pci.h" > #include "hw/pci/pcie_host.h" > #include "hw/usb/xhci.h" > +#include "hw/virtio/virtio-acpi.h" > #include "hw/virtio/virtio-mmio.h" > #include "hw/input/i8042.h" > > @@ -77,19 +78,7 @@ static void acpi_dsdt_add_virtio(Aml *scope, > uint32_t irq = mms->virtio_irq_base + index; > hwaddr base = VIRTIO_MMIO_BASE + index * 512; > hwaddr size = 512; > - > - Aml *dev = aml_device("VR%02u", (unsigned)index); > - aml_append(dev, aml_name_decl("_HID", aml_string("LNRO0005"))); > - aml_append(dev, aml_name_decl("_UID", aml_int(index))); > - aml_append(dev, aml_name_decl("_CCA", aml_int(1))); > - > - Aml *crs = aml_resource_template(); > - 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, 1)); > - aml_append(dev, aml_name_decl("_CRS", crs)); > - aml_append(scope, dev); > + virtio_acpi_dsdt_add(scope, base, size, irq, index, 1); > } > } > } > -- > 2.39.2 > >