GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

amd/fam*/northbridge.c: Fix unused hest variable 

The variable actually makes to code look a lot better.

TESTED: BUILD_TIMELESS=1 results in identical binaries

Change-Id: Ie9104e4736a3c30b7592bb0e79a8ddc6af579800
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63041
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Elyes Haouas <ehaouas@noos.fr>
This commit is contained in:
Arthur Heymans 2022-03-23 21:33:15 +01:00 committed by Felix Held
parent a24bcce321
commit cc66ff3043
3 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/northbridge/amd/agesa/family14/northbridge.c
View File

@ -714,9 +714,9 @@ static unsigned long agesa_write_acpi_tables(const struct device *device,
/* HEST */ /* HEST */
current = ALIGN(current, 8); current = ALIGN(current, 8);
hest = (acpi_hest_t *)current; hest = (acpi_hest_t *)current;
acpi_write_hest((void *)current, acpi_fill_hest); acpi_write_hest(hest, acpi_fill_hest);
acpi_add_table(rsdp, (void *)current); acpi_add_table(rsdp, hest);
current += ((acpi_header_t *)current)->length; current += hest->header.length;
/* SRAT */ /* SRAT */
current = ALIGN(current, 8); current = ALIGN(current, 8);

6
src/northbridge/amd/agesa/family15tn/northbridge.c
View File

@ -475,9 +475,9 @@ static unsigned long agesa_write_acpi_tables(const struct device *device,
/* HEST */ /* HEST */
current = ALIGN(current, 8); current = ALIGN(current, 8);
hest = (acpi_hest_t *)current; hest = (acpi_hest_t *)current;
acpi_write_hest((void *)current, acpi_fill_hest); acpi_write_hest(hest, acpi_fill_hest);
acpi_add_table(rsdp, (void *)current); acpi_add_table(rsdp, hest);
current += ((acpi_header_t *)current)->length; current += hest->header.length;
current = ALIGN(current, 8); current = ALIGN(current, 8);
printk(BIOS_DEBUG, "ACPI: * IVRS at %lx\n", current); printk(BIOS_DEBUG, "ACPI: * IVRS at %lx\n", current);

6
src/northbridge/amd/agesa/family16kb/northbridge.c
View File

@ -469,9 +469,9 @@ static unsigned long agesa_write_acpi_tables(const struct device *device,
/* HEST */ /* HEST */
current = ALIGN(current, 8); current = ALIGN(current, 8);
hest = (acpi_hest_t *)current; hest = (acpi_hest_t *)current;
acpi_write_hest((void *)current, acpi_fill_hest); acpi_write_hest(hest, acpi_fill_hest);
acpi_add_table(rsdp, (void *)current); acpi_add_table(rsdp, hest);
current += ((acpi_header_t *)current)->length; current += hest->header.length;
current = ALIGN(current, 8); current = ALIGN(current, 8);
printk(BIOS_DEBUG, "ACPI: * IVRS at %lx\n", current); printk(BIOS_DEBUG, "ACPI: * IVRS at %lx\n", current);