On Mon, 15 Jul 2024 22:41:22 +0530 Sunil V L <suni...@ventanamicro.com> wrote:
> The requirement ACPI_060 in the RISC-V BRS specification [1], requires > NS16550 compatible UART to have the HID RSCV0003. So, update the HID for > the UART. > > [1] - https://github.com/riscv-non-isa/riscv-brs/commits/main/acpi.adoc this should point to text like in previous patch and not to commit > (commit: 7bfa87e86ad5658283731207dbfc8ab3744d3265) > > Signed-off-by: Sunil V L <suni...@ventanamicro.com> > Acked-by: Alistair Francis <alistair.fran...@wdc.com> with above fixed: Reviewed-by: Igor Mammedov <imamm...@redhat.com> > --- > hw/riscv/virt-acpi-build.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/riscv/virt-acpi-build.c b/hw/riscv/virt-acpi-build.c > index 5f5082a35b..36d6a3a412 100644 > --- a/hw/riscv/virt-acpi-build.c > +++ b/hw/riscv/virt-acpi-build.c > @@ -170,7 +170,7 @@ acpi_dsdt_add_uart(Aml *scope, const MemMapEntry > *uart_memmap, > uint32_t uart_irq) > { > Aml *dev = aml_device("COM0"); > - aml_append(dev, aml_name_decl("_HID", aml_string("PNP0501"))); > + aml_append(dev, aml_name_decl("_HID", aml_string("RSCV0003"))); > aml_append(dev, aml_name_decl("_UID", aml_int(0))); > > Aml *crs = aml_resource_template();
