Re: [Qemu-devel] [PATCH v4 03/23] hw: acpi: Export the RSDP build API

2018-11-06 Thread Paolo Bonzini
On 02/11/2018 10:56, Philippe Mathieu-Daudé wrote: >>> >>>   } >>>   /* RSDP */ >>> -static GArray * >>> +static void >>>   build_rsdp(GArray *rsdp_table, BIOSLinker *linker, unsigned >>> xsdt_tbl_offset) >>>   { >>>   AcpiRsdpDescriptor *rsdp = acpi_data_push(rsdp_table, sizeof >>> *rsdp); >>

Re: [Qemu-devel] [PATCH v4 03/23] hw: acpi: Export the RSDP build API

2018-11-02 Thread Philippe Mathieu-Daudé
Hi, On 2/11/18 10:20, Shannon Zhao wrote: Hi, On 2018/11/1 18:22, Samuel Ortiz wrote: diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c index f28a2faa53..0ed132b79b 100644 --- a/hw/arm/virt-acpi-build.c +++ b/hw/arm/virt-acpi-build.c @@ -367,7 +367,7 @@ static void

Re: [Qemu-devel] [PATCH v4 03/23] hw: acpi: Export the RSDP build API

2018-11-02 Thread Shannon Zhao
Hi, On 2018/11/1 18:22, Samuel Ortiz wrote: diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c index f28a2faa53..0ed132b79b 100644 --- a/hw/arm/virt-acpi-build.c +++ b/hw/arm/virt-acpi-build.c @@ -367,7 +367,7 @@ static void acpi_dsdt_add_power_button(Aml *scope) } /* RSDP

Re: [Qemu-devel] [PATCH v4 03/23] hw: acpi: Export the RSDP build API

2018-11-01 Thread Philippe Mathieu-Daudé
Hey Samuel, On 1/11/18 11:22, Samuel Ortiz wrote: The hardware-reduced API will need to build RSDP as well, so we should export this routine. While doing so, we also slightly change the function prototype. Since no caller needs it, and to make it more consistent with the rest of the AML build

[Qemu-devel] [PATCH v4 03/23] hw: acpi: Export the RSDP build API

2018-11-01 Thread Samuel Ortiz
The hardware-reduced API will need to build RSDP as well, so we should export this routine. While doing so, we also slightly change the function prototype. Since no caller needs it, and to make it more consistent with the rest of the AML build API, the function no longer returns its RSDP table.