From patchwork Wed Mar 24 20:24:56 2010
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: remove some warnings
Date: Wed, 24 Mar 2010 20:24:56 -0000
From: Myles Watson <mylesgw@gmail.com>
X-Patchwork-Id: 1139
Message-Id: <2831fecf1003241324s36c4765fgd103742c91b9e27@mail.gmail.com>
To: coreboot@coreboot.org

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

     current = ALIGN(current, 16);


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);