[coreboot] Patch set updated for coreboot: b161e56 Intel and GFXUMA: drop redundant use of lb_add_memory_range()
Kyösti Mälkki (kyosti.malkki@gmail.com)
gerrit at coreboot.org
Fri Jul 27 14:02:07 CEST 2012
Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1238
-gerrit
commit b161e5614ce6660b14a10d39d9a673715b88160c
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date: Wed Jul 18 14:38:54 2012 +0300
Intel and GFXUMA: drop redundant use of lb_add_memory_range()
Use of uma_resource() in northbridge code created a memory
resource marked as reserved. Such resources are removed
from system memory in write_coreboot_table().
Change-Id: I14bfd560140d8d30ec156562f23072bfae747bde
Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
src/mainboard/asus/mew-am/mainboard.c | 2 +-
src/mainboard/asus/mew-vm/mainboard.c | 2 +-
src/mainboard/ecs/p6iwp-fe/mainboard.c | 2 +-
src/mainboard/hp/e_vectra_p2706t/mainboard.c | 2 +-
src/mainboard/intel/d810e2cb/mainboard.c | 2 +-
src/mainboard/intel/emeraldlake2/mainboard.c | 2 +-
src/mainboard/mitac/6513wu/mainboard.c | 2 +-
src/mainboard/msi/ms6178/mainboard.c | 2 +-
src/mainboard/nec/powermate2000/mainboard.c | 2 +-
src/mainboard/rca/rm4100/mainboard.c | 2 +-
src/mainboard/samsung/lumpy/mainboard.c | 2 +-
src/mainboard/samsung/stumpy/mainboard.c | 2 +-
src/mainboard/thomson/ip1000/mainboard.c | 2 +-
src/northbridge/intel/i82810/northbridge.c | 9 ---------
src/northbridge/intel/i82830/northbridge.c | 9 ---------
src/northbridge/intel/sandybridge/northbridge.c | 5 -----
16 files changed, 13 insertions(+), 36 deletions(-)
diff --git a/src/mainboard/asus/mew-am/mainboard.c b/src/mainboard/asus/mew-am/mainboard.c
index 8887218..6409dab 100644
--- a/src/mainboard/asus/mew-am/mainboard.c
+++ b/src/mainboard/asus/mew-am/mainboard.c
@@ -23,7 +23,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/asus/mew-vm/mainboard.c b/src/mainboard/asus/mew-vm/mainboard.c
index e278565..2e43be2 100644
--- a/src/mainboard/asus/mew-vm/mainboard.c
+++ b/src/mainboard/asus/mew-vm/mainboard.c
@@ -3,7 +3,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/ecs/p6iwp-fe/mainboard.c b/src/mainboard/ecs/p6iwp-fe/mainboard.c
index 9031ac7..667ac5e 100644
--- a/src/mainboard/ecs/p6iwp-fe/mainboard.c
+++ b/src/mainboard/ecs/p6iwp-fe/mainboard.c
@@ -23,7 +23,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/hp/e_vectra_p2706t/mainboard.c b/src/mainboard/hp/e_vectra_p2706t/mainboard.c
index 0f9b75e..80e6f61 100644
--- a/src/mainboard/hp/e_vectra_p2706t/mainboard.c
+++ b/src/mainboard/hp/e_vectra_p2706t/mainboard.c
@@ -23,7 +23,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/intel/d810e2cb/mainboard.c b/src/mainboard/intel/d810e2cb/mainboard.c
index 7589d6e..8455377 100644
--- a/src/mainboard/intel/d810e2cb/mainboard.c
+++ b/src/mainboard/intel/d810e2cb/mainboard.c
@@ -22,7 +22,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/intel/emeraldlake2/mainboard.c b/src/mainboard/intel/emeraldlake2/mainboard.c
index 5717edf..796f71e 100644
--- a/src/mainboard/intel/emeraldlake2/mainboard.c
+++ b/src/mainboard/intel/emeraldlake2/mainboard.c
@@ -44,7 +44,7 @@ void mainboard_suspend_resume(void)
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
#if defined(CONFIG_PCI_OPTION_ROM_RUN_REALMODE) && CONFIG_PCI_OPTION_ROM_RUN_REALMODE
diff --git a/src/mainboard/mitac/6513wu/mainboard.c b/src/mainboard/mitac/6513wu/mainboard.c
index 968b9e9..04af449 100644
--- a/src/mainboard/mitac/6513wu/mainboard.c
+++ b/src/mainboard/mitac/6513wu/mainboard.c
@@ -23,7 +23,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/msi/ms6178/mainboard.c b/src/mainboard/msi/ms6178/mainboard.c
index 0aeedb8..76c8c04 100644
--- a/src/mainboard/msi/ms6178/mainboard.c
+++ b/src/mainboard/msi/ms6178/mainboard.c
@@ -23,7 +23,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/nec/powermate2000/mainboard.c b/src/mainboard/nec/powermate2000/mainboard.c
index 3b29c7c..3bbaa11 100644
--- a/src/mainboard/nec/powermate2000/mainboard.c
+++ b/src/mainboard/nec/powermate2000/mainboard.c
@@ -23,7 +23,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/rca/rm4100/mainboard.c b/src/mainboard/rca/rm4100/mainboard.c
index 613c1d7..ff98977 100644
--- a/src/mainboard/rca/rm4100/mainboard.c
+++ b/src/mainboard/rca/rm4100/mainboard.c
@@ -24,7 +24,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
static void mainboard_init(device_t dev)
diff --git a/src/mainboard/samsung/lumpy/mainboard.c b/src/mainboard/samsung/lumpy/mainboard.c
index 20b59b7..e83516a 100644
--- a/src/mainboard/samsung/lumpy/mainboard.c
+++ b/src/mainboard/samsung/lumpy/mainboard.c
@@ -51,7 +51,7 @@ void mainboard_suspend_resume(void)
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
#if defined(CONFIG_PCI_OPTION_ROM_RUN_REALMODE) && CONFIG_PCI_OPTION_ROM_RUN_REALMODE
diff --git a/src/mainboard/samsung/stumpy/mainboard.c b/src/mainboard/samsung/stumpy/mainboard.c
index 2456e70..246b261 100644
--- a/src/mainboard/samsung/stumpy/mainboard.c
+++ b/src/mainboard/samsung/stumpy/mainboard.c
@@ -44,7 +44,7 @@ void mainboard_suspend_resume(void)
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
#if defined(CONFIG_PCI_OPTION_ROM_RUN_REALMODE) && CONFIG_PCI_OPTION_ROM_RUN_REALMODE
diff --git a/src/mainboard/thomson/ip1000/mainboard.c b/src/mainboard/thomson/ip1000/mainboard.c
index e5c3ea1..ae5872d 100644
--- a/src/mainboard/thomson/ip1000/mainboard.c
+++ b/src/mainboard/thomson/ip1000/mainboard.c
@@ -30,7 +30,7 @@
int add_mainboard_resources(struct lb_memory *mem)
{
- return add_northbridge_resources(mem);
+ return 0;
}
// setting the bit disables the led.
diff --git a/src/northbridge/intel/i82810/northbridge.c b/src/northbridge/intel/i82810/northbridge.c
index f6d14e0..c738de7 100644
--- a/src/northbridge/intel/i82810/northbridge.c
+++ b/src/northbridge/intel/i82810/northbridge.c
@@ -62,15 +62,6 @@ static const struct pci_driver i810e_northbridge_driver __pci_driver = {
.device = 0x7124,
};
-int add_northbridge_resources(struct lb_memory *mem)
-{
- printk(BIOS_DEBUG, "Adding IGD UMA memory area\n");
- lb_add_memory_range(mem, LB_MEM_RESERVED,
- uma_memory_base, uma_memory_size);
-
- return 0;
-}
-
/* Table which returns the RAM size in MB when fed the DRP[7:4] or [3:0] value.
* Note that 2 is a value which the DRP should never be programmed to.
* Some size values appear twice, due to single-sided vs dual-sided banks.
diff --git a/src/northbridge/intel/i82830/northbridge.c b/src/northbridge/intel/i82830/northbridge.c
index 649a203..4951c4f 100644
--- a/src/northbridge/intel/i82830/northbridge.c
+++ b/src/northbridge/intel/i82830/northbridge.c
@@ -52,15 +52,6 @@ static const struct pci_driver northbridge_driver __pci_driver = {
.device = 0x3575,
};
-int add_northbridge_resources(struct lb_memory *mem)
-{
- printk(BIOS_DEBUG, "Adding IGD UMA memory area\n");
- lb_add_memory_range(mem, LB_MEM_RESERVED,
- uma_memory_base, uma_memory_size);
-
- return 0;
-}
-
#if CONFIG_WRITE_HIGH_TABLES
#include <cbmem.h>
#endif
diff --git a/src/northbridge/intel/sandybridge/northbridge.c b/src/northbridge/intel/sandybridge/northbridge.c
index 5a9388e..eec49dd 100644
--- a/src/northbridge/intel/sandybridge/northbridge.c
+++ b/src/northbridge/intel/sandybridge/northbridge.c
@@ -61,11 +61,6 @@ int bridge_silicon_revision(void)
static const int legacy_hole_base_k = 0xa0000 / 1024;
static const int legacy_hole_size_k = 384;
-int add_northbridge_resources(struct lb_memory *mem)
-{
- return 0;
-}
-
void cbmem_post_handling(void)
{
update_mrc_cache();
More information about the coreboot
mailing list