[coreboot] [Patch] remove some warnings

Myles Watson mylesgw at gmail.com
Wed Mar 24 21:24:56 CET 2010


Ping.

I've inlined a representative snippet.  Not only does it get rid of the
warnings, it gets rid of a lot of casting.

Thanks,
Myles

Index: svn/src/mainboard/tyan/s2895/acpi_tables.c
===================================================================
--- svn.orig/src/mainboard/tyan/s2895/acpi_tables.c
+++ svn/src/mainboard/tyan/s2895/acpi_tables.c
@@ -20,7 +20,7 @@
 #include <../../../northbridge/amd/amdk8/amdk8_acpi.h>
 #include <cpu/amd/model_fxx_powernow.h>

-extern unsigned char AmlCode[];
+extern acpi_header_t AmlCode;

 unsigned long acpi_fill_mcfg(unsigned long current)
 {
@@ -164,8 +164,8 @@ unsigned long write_acpi_tables(unsigned
     current = ALIGN(current, 16);
     dsdt = (acpi_header_t *) current;
     printk_debug("ACPI:    * DSDT %p\n", dsdt);
-    current += ((acpi_header_t *) AmlCode)->length;
-    memcpy((void*) dsdt, (void*)AmlCode,
((acpi_header_t*)AmlCode)->length);
+    current += AmlCode.length;
+    memcpy((void*) dsdt, &AmlCode, AmlCode.length);
     printk_debug("ACPI:    * DSDT @ %p Length %x\n",dsdt,dsdt->length);

     current = ALIGN(current, 16);
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100324/49ce67eb/attachment.html>


More information about the coreboot mailing list