[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