Hi all,

This seems to be already in SVN, but breaks the build, because on other boards
acpi_fill_mcfg is not defined.

Following patch solves that.

I declared it weak symbol, so it works even if it is not there. Its only compile
tested for VIA epia-m and ASUS a8v-e se.

Perhaps this function should be only a wrapper in mb code for some kind of
chipset specific mmconfig fill function.

Signed-off-by: Rudolf Marek <[EMAIL PROTECTED]>


Rudolf
Index: src/arch/i386/boot/acpi.c
===================================================================
--- src/arch/i386/boot/acpi.c	(revision 2939)
+++ src/arch/i386/boot/acpi.c	(working copy)
@@ -161,6 +161,10 @@
 	acpi_header_t *header=&(mcfg->header);
 	unsigned long current=(unsigned long)mcfg+sizeof(acpi_mcfg_t);
 	
+	/* have no such function defined, do nothing */
+	if (!acpi_fill_mcfg)
+		return;
+	
 	memset((void *)mcfg, 0, sizeof(acpi_mcfg_t));
 	
 	/* fill out header fields */
Index: src/arch/i386/include/arch/acpi.h
===================================================================
--- src/arch/i386/include/arch/acpi.h	(revision 2939)
+++ src/arch/i386/include/arch/acpi.h	(working copy)
@@ -291,7 +291,7 @@
 /* These are implemented by the target port */
 unsigned long write_acpi_tables(unsigned long addr);
 unsigned long acpi_fill_madt(unsigned long current);
-unsigned long acpi_fill_mcfg(unsigned long current);
+extern unsigned long acpi_fill_mcfg(unsigned long current) __attribute__ ((weak));
 unsigned long acpi_fill_srat(unsigned long current); 
 void acpi_create_fadt(acpi_fadt_t *fadt,acpi_facs_t *facs,void *dsdt);
 
-- 
linuxbios mailing list
linuxbios@linuxbios.org
http://www.linuxbios.org/mailman/listinfo/linuxbios

Reply via email to