On Wed, Jan 30, 2019 at 09:16:50AM +0100, Igor Mammedov wrote: >On Wed, 30 Jan 2019 11:02:07 +0800 >Wei Yang <richardw.y...@linux.intel.com> wrote: > >> The second argument of build_facs() is not used, just remove it. >> >> Signed-off-by: Wei Yang <richardw.y...@linux.intel.com> > >Reviewed-by: Igor Mammedov <imamm...@redhat.com>
Thanks :-) > >> --- >> hw/i386/acpi-build.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c >> index 236a20eaa8..023729c1f2 100644 >> --- a/hw/i386/acpi-build.c >> +++ b/hw/i386/acpi-build.c >> @@ -292,7 +292,7 @@ static void acpi_align_size(GArray *blob, unsigned align) >> >> /* FACS */ >> static void >> -build_facs(GArray *table_data, BIOSLinker *linker) >> +build_facs(GArray *table_data) >> { >> AcpiFacsDescriptorRev1 *facs = acpi_data_push(table_data, sizeof *facs); >> memcpy(&facs->signature, "FACS", 4); >> @@ -2643,7 +2643,7 @@ void acpi_build(AcpiBuildTables *tables, MachineState >> *machine) >> * requirements. >> */ >> facs = tables_blob->len; >> - build_facs(tables_blob, tables->linker); >> + build_facs(tables_blob); >> >> /* DSDT is pointed to by FADT */ >> dsdt = tables_blob->len; -- Wei Yang Help you, Help me