[coreboot] unifying `acpi_tables.c` (was: [PATCH] workaround for "An unexpected error (805262864) occurred at line 1768 of d:\xpclient\base\boot\setup\arcdisp.c")
Paul Menzel
paulepanter at users.sourceforge.net
Thu May 19 00:14:49 CEST 2011
Dear Scott and coreboot folks,
Am Mittwoch, den 18.05.2011, 01:40 -0500 schrieb Scott Duplichan:
[…]
looking at the change
> Index: src/mainboard/amd/bimini_fam10/acpi_tables.c
> ===================================================================
> --- src/mainboard/amd/bimini_fam10/acpi_tables.c (revision 6599)
> +++ src/mainboard/amd/bimini_fam10/acpi_tables.c (working copy)
> @@ -127,6 +127,31 @@
> acpi_write_rsdp(rsdp, rsdt, NULL);
> acpi_write_rsdt(rsdt);
>
> + /* DSDT */
> + current = ( current + 0x07) & -0x08;
> + printk(BIOS_DEBUG, "ACPI: * DSDT at %lx\n", current);
> + dsdt = (acpi_header_t *)current; // it will used by fadt
> + memcpy(dsdt, &AmlCode, sizeof(acpi_header_t));
> + current += dsdt->length;
> + memcpy(dsdt, &AmlCode, dsdt->length);
> + printk(BIOS_DEBUG, "ACPI: * DSDT @ %p Length %x\n",dsdt,dsdt->length);
> +
> + /* FACS */ // it needs 64 bit alignment
> + current = ( current + 0x07) & -0x08;
> + printk(BIOS_DEBUG, "ACPI: * FACS at %lx\n", current);
> + facs = (acpi_facs_t *) current; // it will be used by fadt
> + current += sizeof(acpi_facs_t);
> + acpi_create_facs(facs);
> +
> + /* FADT */
> + current = ( current + 0x07) & -0x08;
> + printk(BIOS_DEBUG, "ACPI: * FADT at %lx\n", current);
> + fadt = (acpi_fadt_t *) current;
> + current += sizeof(acpi_fadt_t);
> +
> + acpi_create_fadt(fadt, facs, dsdt);
> + acpi_add_table(rsdp, fadt);
[…]
the contents seems to very similar. Besides some comments even
identical.
Can `acpi_tables.c` be shared between certain families or even all new
boards?
If yes, maybe that could be put on a todo list.
Thanks,
Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20110519/2dc0da63/attachment.sig>
More information about the coreboot
mailing list