Author: oxygene
Date: 2009-04-28 14:57:25 +0200 (Tue, 28 Apr 2009)
New Revision: 4223
Modified:
trunk/coreboot-v2/src/arch/i386/boot/coreboot_table.c
trunk/coreboot-v2/src/mainboard/asus/m2v-mx_se/mainboard.c
trunk/coreboot-v2/src/mainboard/kontron/986lcd-m/mainboard.c
trunk/coreboot-v2/src/mainboard/supermicro/h8dme/Options.lb
trunk/coreboot-v2/src/mainboard/supermicro/h8dme/mainboard.c
trunk/coreboot-v2/src/mainboard/tyan/s2891/Options.lb
trunk/coreboot-v2/src/mainboard/tyan/s2891/mainboard.c
trunk/coreboot-v2/src/mainboard/tyan/s2892/Options.lb
trunk/coreboot-v2/src/mainboard/tyan/s2892/mainboard.c
trunk/coreboot-v2/src/mainboard/tyan/s2895/Options.lb
trunk/coreboot-v2/src/mainboard/tyan/s2895/mainboard.c
trunk/coreboot-v2/src/mainboard/via/vt8454c/Options.lb
trunk/coreboot-v2/src/mainboard/via/vt8454c/mainboard.c
Log:
add_mainboard_resources is necessary for some boards (eg. kontron), but
this generic code could be added to the caller of
add_mainboard_resources (wrapped in HAVE_HIGH_TABLES, of course).
That way, boards that really need it (for other things) can use this
function, while others don't have to do anything to use
HAVE_HIGH_TABLES.
Signed-off-by: Patrick Georgi <patrick.georgi(a)coresystems.de>
Acked-by: Stefan Reinauer <stepan(a)coresystems.de>
Acked-by: Peter Stuge <peter(a)stuge.se>
Modified: trunk/coreboot-v2/src/arch/i386/boot/coreboot_table.c
===================================================================
--- trunk/coreboot-v2/src/arch/i386/boot/coreboot_table.c 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/arch/i386/boot/coreboot_table.c 2009-04-28 12:57:25 UTC (rev 4223)
@@ -413,6 +413,10 @@
return mem;
}
+#if HAVE_HIGH_TABLES == 1
+extern uint64_t high_tables_base, high_tables_size;
+#endif
+
unsigned long write_coreboot_table(
unsigned long low_table_start, unsigned long low_table_end,
unsigned long rom_table_start, unsigned long rom_table_end)
@@ -479,6 +483,12 @@
lb_add_memory_range(mem, LB_MEM_TABLE,
rom_table_start, rom_table_end-rom_table_start);
+#if HAVE_HIGH_TABLES == 1
+ printk_debug("Adding high table area\n");
+ lb_add_memory_range(mem, LB_MEM_TABLE,
+ high_tables_base, high_tables_size);
+#endif
+
#if (HAVE_MAINBOARD_RESOURCES == 1)
add_mainboard_resources(mem);
#endif
Modified: trunk/coreboot-v2/src/mainboard/asus/m2v-mx_se/mainboard.c
===================================================================
--- trunk/coreboot-v2/src/mainboard/asus/m2v-mx_se/mainboard.c 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/asus/m2v-mx_se/mainboard.c 2009-04-28 12:57:25 UTC (rev 4223)
@@ -17,23 +17,14 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
-#include <console/console.h>
#include <device/device.h>
#include <device/pci.h>
#include <device/pci_ids.h>
#include <boot/tables.h>
#include "chip.h"
-/* in arch/i386/boot/tables.c */
-extern uint64_t high_tables_base, high_tables_size;
-
int add_mainboard_resources(struct lb_memory *mem)
{
-#if HAVE_HIGH_TABLES == 1
- printk_debug("Adding high table area\n");
- lb_add_memory_range(mem, LB_MEM_TABLE,
- high_tables_base, high_tables_size);
-#endif
#if HAVE_ACPI_RESUME == 1
lb_add_memory_range(mem, LB_MEM_RESERVED,
_RAMBASE, ((CONFIG_LB_MEM_TOPK<<10) - _RAMBASE));
Modified: trunk/coreboot-v2/src/mainboard/kontron/986lcd-m/mainboard.c
===================================================================
--- trunk/coreboot-v2/src/mainboard/kontron/986lcd-m/mainboard.c 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/kontron/986lcd-m/mainboard.c 2009-04-28 12:57:25 UTC (rev 4223)
@@ -25,19 +25,11 @@
#include <boot/tables.h>
#include "chip.h"
-/* in arch/i386/boot/tables.c */
-extern uint64_t high_tables_base, high_tables_size;
-
/* in northbridge/intel/i945/northbridge.c */
extern uint64_t uma_memory_base, uma_memory_size;
int add_mainboard_resources(struct lb_memory *mem)
{
-#if HAVE_HIGH_TABLES == 1
- printk_debug("Adding high table area\n");
- lb_add_memory_range(mem, LB_MEM_TABLE,
- high_tables_base, high_tables_size);
-#endif
printk_debug("Adding UMA memory area\n");
lb_add_memory_range(mem, LB_MEM_RESERVED,
uma_memory_base, uma_memory_size);
Modified: trunk/coreboot-v2/src/mainboard/supermicro/h8dme/Options.lb
===================================================================
--- trunk/coreboot-v2/src/mainboard/supermicro/h8dme/Options.lb 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/supermicro/h8dme/Options.lb 2009-04-28 12:57:25 UTC (rev 4223)
@@ -56,7 +56,6 @@
uses STACK_SIZE
uses HEAP_SIZE
uses USE_OPTION_TABLE
-uses HAVE_MAINBOARD_RESOURCES
uses HAVE_HIGH_TABLES
uses HAVE_LOW_TABLES
uses CONFIG_MULTIBOOT
@@ -129,7 +128,6 @@
#default ROM_SIZE=524288
default ROM_SIZE=0x100000
-default HAVE_MAINBOARD_RESOURCES = 1
default HAVE_HIGH_TABLES = 1
default HAVE_LOW_TABLES = 0
default CONFIG_MULTIBOOT=0
Modified: trunk/coreboot-v2/src/mainboard/supermicro/h8dme/mainboard.c
===================================================================
--- trunk/coreboot-v2/src/mainboard/supermicro/h8dme/mainboard.c 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/supermicro/h8dme/mainboard.c 2009-04-28 12:57:25 UTC (rev 4223)
@@ -17,22 +17,8 @@
*/
#include <device/device.h>
-#include <console/console.h>
-#include <boot/tables.h>
#include "chip.h"
-/* in arch/i386/boot/tables.c */
-extern uint64_t high_tables_base, high_tables_size;
-
-int add_mainboard_resources(struct lb_memory *mem)
-{
-#if HAVE_HIGH_TABLES == 1
- lb_add_memory_range(mem, LB_MEM_TABLE,
- high_tables_base, high_tables_size);
-#endif
- return 0;
-}
-
struct chip_operations mainboard_ops = {
CHIP_NAME("Supermicro H8DME Mainboard")
};
Modified: trunk/coreboot-v2/src/mainboard/tyan/s2891/Options.lb
===================================================================
--- trunk/coreboot-v2/src/mainboard/tyan/s2891/Options.lb 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/tyan/s2891/Options.lb 2009-04-28 12:57:25 UTC (rev 4223)
@@ -33,7 +33,6 @@
uses LB_CKS_LOC
uses HAVE_ACPI_TABLES
uses HAVE_ACPI_RESUME
-uses HAVE_MAINBOARD_RESOURCES
uses HAVE_HIGH_TABLES
uses HAVE_LOW_TABLES
uses CONFIG_MULTIBOOT
@@ -130,8 +129,7 @@
##
default HAVE_ACPI_TABLES=1
default HAVE_LOW_TABLES=1
-default HAVE_MAINBOARD_RESOURCES=1
-default HAVE_HIGH_TABLES=0
+default HAVE_HIGH_TABLES=1
default CONFIG_MULTIBOOT=0
##
Modified: trunk/coreboot-v2/src/mainboard/tyan/s2891/mainboard.c
===================================================================
--- trunk/coreboot-v2/src/mainboard/tyan/s2891/mainboard.c 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/tyan/s2891/mainboard.c 2009-04-28 12:57:25 UTC (rev 4223)
@@ -1,22 +1,6 @@
#include <device/device.h>
-#include <console/console.h>
-#include <boot/tables.h>
#include "chip.h"
-/* in arch/i386/boot/tables.c */
-extern uint64_t high_tables_base, high_tables_size;
-
-int add_mainboard_resources(struct lb_memory *mem)
-{
-#if HAVE_HIGH_TABLES == 1
- printk_debug("Adding high table area\n");
- lb_add_memory_range(mem, LB_MEM_TABLE,
- high_tables_base, high_tables_size);
-#endif
- return 0;
-}
-
-
struct chip_operations mainboard_ops = {
CHIP_NAME("Tyan S2891 Mainboard")
};
Modified: trunk/coreboot-v2/src/mainboard/tyan/s2892/Options.lb
===================================================================
--- trunk/coreboot-v2/src/mainboard/tyan/s2892/Options.lb 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/tyan/s2892/Options.lb 2009-04-28 12:57:25 UTC (rev 4223)
@@ -33,7 +33,6 @@
uses LB_CKS_LOC
uses HAVE_ACPI_TABLES
uses HAVE_ACPI_RESUME
-uses HAVE_MAINBOARD_RESOURCES
uses HAVE_HIGH_TABLES
uses HAVE_LOW_TABLES
uses CONFIG_MULTIBOOT
@@ -124,8 +123,7 @@
##
default HAVE_ACPI_TABLES=1
default HAVE_LOW_TABLES=1
-default HAVE_MAINBOARD_RESOURCES=1
-default HAVE_HIGH_TABLES=0
+default HAVE_HIGH_TABLES=1
default CONFIG_MULTIBOOT=0
##
Modified: trunk/coreboot-v2/src/mainboard/tyan/s2892/mainboard.c
===================================================================
--- trunk/coreboot-v2/src/mainboard/tyan/s2892/mainboard.c 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/tyan/s2892/mainboard.c 2009-04-28 12:57:25 UTC (rev 4223)
@@ -1,22 +1,6 @@
#include <device/device.h>
-#include <console/console.h>
-#include <boot/tables.h>
#include "chip.h"
-/* in arch/i386/boot/tables.c */
-extern uint64_t high_tables_base, high_tables_size;
-
-int add_mainboard_resources(struct lb_memory *mem)
-{
-#if HAVE_HIGH_TABLES == 1
- printk_debug("Adding high table area\n");
- lb_add_memory_range(mem, LB_MEM_TABLE,
- high_tables_base, high_tables_size);
-#endif
- return 0;
-}
-
-
struct chip_operations mainboard_ops = {
CHIP_NAME("Tyan S2892 Mainboard")
};
Modified: trunk/coreboot-v2/src/mainboard/tyan/s2895/Options.lb
===================================================================
--- trunk/coreboot-v2/src/mainboard/tyan/s2895/Options.lb 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/tyan/s2895/Options.lb 2009-04-28 12:57:25 UTC (rev 4223)
@@ -36,7 +36,6 @@
uses LB_CKS_LOC
uses HAVE_ACPI_TABLES
uses HAVE_ACPI_RESUME
-uses HAVE_MAINBOARD_RESOURCES
uses HAVE_HIGH_TABLES
uses HAVE_LOW_TABLES
uses CONFIG_MULTIBOOT
@@ -137,8 +136,7 @@
##
default HAVE_ACPI_TABLES=1
default HAVE_LOW_TABLES=1
-default HAVE_MAINBOARD_RESOURCES=1
-default HAVE_HIGH_TABLES=0
+default HAVE_HIGH_TABLES=1
default CONFIG_MULTIBOOT=0
##
Modified: trunk/coreboot-v2/src/mainboard/tyan/s2895/mainboard.c
===================================================================
--- trunk/coreboot-v2/src/mainboard/tyan/s2895/mainboard.c 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/tyan/s2895/mainboard.c 2009-04-28 12:57:25 UTC (rev 4223)
@@ -1,22 +1,6 @@
#include <device/device.h>
-#include <console/console.h>
-#include <boot/tables.h>
#include "chip.h"
-/* in arch/i386/boot/tables.c */
-extern uint64_t high_tables_base, high_tables_size;
-
-int add_mainboard_resources(struct lb_memory *mem)
-{
-#if HAVE_HIGH_TABLES == 1
- printk_debug("Adding high table area\n");
- lb_add_memory_range(mem, LB_MEM_TABLE,
- high_tables_base, high_tables_size);
-#endif
- return 0;
-}
-
-
struct chip_operations mainboard_ops = {
CHIP_NAME("Tyan S2895 Mainboard")
};
Modified: trunk/coreboot-v2/src/mainboard/via/vt8454c/Options.lb
===================================================================
--- trunk/coreboot-v2/src/mainboard/via/vt8454c/Options.lb 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/via/vt8454c/Options.lb 2009-04-28 12:57:25 UTC (rev 4223)
@@ -27,7 +27,6 @@
uses USE_OPTION_TABLE
uses HAVE_LOW_TABLES
uses HAVE_HIGH_TABLES
-uses HAVE_MAINBOARD_RESOURCES
uses USE_FALLBACK_IMAGE
uses HAVE_FALLBACK_BOOT
@@ -147,7 +146,6 @@
##
default HAVE_LOW_TABLES=1
default HAVE_HIGH_TABLES=1
-default HAVE_MAINBOARD_RESOURCES=1
##
Modified: trunk/coreboot-v2/src/mainboard/via/vt8454c/mainboard.c
===================================================================
--- trunk/coreboot-v2/src/mainboard/via/vt8454c/mainboard.c 2009-04-27 20:19:48 UTC (rev 4222)
+++ trunk/coreboot-v2/src/mainboard/via/vt8454c/mainboard.c 2009-04-28 12:57:25 UTC (rev 4223)
@@ -20,23 +20,8 @@
*/
#include <device/device.h>
-#include <boot/tables.h>
-#include <console/console.h>
#include "chip.h"
-/* in arch/i386/boot/tables.c */
-extern uint64_t high_tables_base, high_tables_size;
-
-int add_mainboard_resources(struct lb_memory *mem)
-{
-#if HAVE_HIGH_TABLES == 1
- printk_debug("Adding high table area\n");
- lb_add_memory_range(mem, LB_MEM_TABLE,
- high_tables_base, high_tables_size);
-#endif
- return 0;
-}
-
struct chip_operations mainboard_ops = {
CHIP_NAME("VIA VT8454c Mainboard")
};