Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2095
-gerrit
commit d63e5d5e12f8a8223cb4908afaaa6bf808f93079
Author: David Hendricks <dhendrix(a)chromium.org>
Date: Wed Jan 2 17:48:49 2013 -0800
armv7: invoke intermediate build rules
This adds $$(INTERMEDIATE) as a pre-requisite for coreboot.rom on
armv7. It is modeled after the $(obj)/coreboot.rom rule for x86.
Change-Id: I483a88035fa2288829b6e042e51ef932c8c4f23c
Signed-off-by: David Hendricks <dhendrix(a)chromium.org>
---
src/arch/armv7/Makefile.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/arch/armv7/Makefile.inc b/src/arch/armv7/Makefile.inc
index 9e57244..301866c 100644
--- a/src/arch/armv7/Makefile.inc
+++ b/src/arch/armv7/Makefile.inc
@@ -58,7 +58,7 @@ $(obj)/coreboot.pre1: $(CBFSTOOL)
mv $(obj)/coreboot.rom $@
endif
-$(obj)/coreboot.rom: $(obj)/coreboot.pre $(objcbfs)/coreboot_ram.elf $(CBFSTOOL) $(call strip_quotes,$(COREBOOT_ROM_DEPENDENCIES))
+$(obj)/coreboot.rom: $(obj)/coreboot.pre $(objcbfs)/coreboot_ram.elf $(CBFSTOOL) $(call strip_quotes,$(COREBOOT_ROM_DEPENDENCIES)) $$(INTERMEDIATE)
@printf " CBFS $(subst $(obj)/,,$(@))\n"
cp $(obj)/coreboot.pre $@.tmp
if [ -f $(objcbfs)/coreboot_ap.elf ]; \
the following patch was just integrated into master:
commit 31c36137f9519380df0c92ce1a5d2949eb083b91
Author: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Date: Fri Jan 4 10:11:59 2013 -0800
Clean up ARMv7 architecture Kconfig
There was a misuse of bool that would cause the dcache policy to not be
set up correctly, but instead present options "y" and "n" in the Kconfig
menu.
Also, TINY_BOOTBLOCK was removed a while ago, everything is
TINY_BOOTBLOCK now. Hence remove the option.
Change-Id: I5c28ac828955c69614c7bdaf106f79db51e68723
Signed-off-by: Stefan Reinauer <reinauer(a)google.com>
Reviewed-on: http://review.coreboot.org/2101
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com>
Build-Tested: build bot (Jenkins) at Fri Jan 4 19:29:46 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich(a)gmail.com> at Fri Jan 4 19:34:50 2013, giving +2
See http://review.coreboot.org/2101 for details.
-gerrit
the following patch was just integrated into master:
commit c4077d44297c95100cc9eb19df1ab3b16261942f
Author: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Date: Fri Jan 4 10:07:28 2013 -0800
Make PCIe config options depend on PCIe support
Change-Id: I42452a044dc75e35876fcea1736481e538eed663
Signed-off-by: Stefan Reinauer <reinauer(a)google.com>
Reviewed-on: http://review.coreboot.org/2100
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com>
Build-Tested: build bot (Jenkins) at Fri Jan 4 19:19:34 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich(a)gmail.com> at Fri Jan 4 19:35:14 2013, giving +2
See http://review.coreboot.org/2100 for details.
-gerrit
Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2101
-gerrit
commit 555bf7db826824c0166d613753ea77e794964968
Author: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Date: Fri Jan 4 10:11:59 2013 -0800
Clean up ARMv7 architecture Kconfig
There was a misuse of bool that would cause the dcache policy to not be
set up correctly, but instead present options "y" and "n" in the Kconfig
menu.
Also, TINY_BOOTBLOCK was removed a while ago, everything is
TINY_BOOTBLOCK now. Hence remove the option.
Change-Id: I5c28ac828955c69614c7bdaf106f79db51e68723
Signed-off-by: Stefan Reinauer <reinauer(a)google.com>
---
src/arch/armv7/Kconfig | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/arch/armv7/Kconfig b/src/arch/armv7/Kconfig
index f92911d..65de2d5 100644
--- a/src/arch/armv7/Kconfig
+++ b/src/arch/armv7/Kconfig
@@ -34,7 +34,6 @@ config BOOTBLOCK_SOURCE
config UPDATE_IMAGE
bool "Update existing coreboot.rom image"
default n
- depends on TINY_BOOTBLOCK
help
If this option is enabled, no new coreboot.rom file
is created. Instead it is expected that there already
@@ -47,9 +46,11 @@ config BOOTBLOCK_SOC_INIT
# FIXME: Should cache policy be set on a per-CPU basis?
# FIXME(dhendrix): Stefan sayz to make a smart decision and not prompt the user.
config ARM_DCACHE_POLICY_WRITEBACK
- bool y
+ bool
+ default y
config ARM_DCACHE_POLICY_WRITETHROUGH
- bool n
+ bool
+ default n
endmenu
Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2100
-gerrit
commit 4bc29b5e870a382723d23b5f52f1cfe77575f16f
Author: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Date: Fri Jan 4 10:07:28 2013 -0800
Make PCIe config options depend on PCIe support
Change-Id: I42452a044dc75e35876fcea1736481e538eed663
Signed-off-by: Stefan Reinauer <reinauer(a)google.com>
---
src/device/Kconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/device/Kconfig b/src/device/Kconfig
index 849cc15..82583a4 100644
--- a/src/device/Kconfig
+++ b/src/device/Kconfig
@@ -212,6 +212,7 @@ config CARDBUS_PLUGIN_SUPPORT
config PCIEXP_COMMON_CLOCK
prompt "Enable PCIe Common Clock"
bool
+ depends on PCIEXP_PLUGIN_SUPPORT
default n
help
Detect and enable Common Clock on PCIe links.
@@ -219,6 +220,7 @@ config PCIEXP_COMMON_CLOCK
config PCIEXP_ASPM
prompt "Enable PCIe ASPM"
bool
+ depends on PCIEXP_PLUGIN_SUPPORT
default n
help
Detect and enable ASPM on PCIe links.
Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2095
-gerrit
commit c6ba0743b2bdf7c331d43161fd8178d3274dca47
Author: David Hendricks <dhendrix(a)chromium.org>
Date: Wed Jan 2 17:48:49 2013 -0800
armv7: invoke intermediate build rules
This adds $$(INTERMEDIATE) as a pre-requisite for coreboot.rom on
armv7. It is modeled after the $(obj)/coreboot.rom rule for x86.
Change-Id: I483a88035fa2288829b6e042e51ef932c8c4f23c
Signed-off-by: David Hendricks <dhendrix(a)chromium.org>
---
src/arch/armv7/Makefile.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/arch/armv7/Makefile.inc b/src/arch/armv7/Makefile.inc
index 9e57244..301866c 100644
--- a/src/arch/armv7/Makefile.inc
+++ b/src/arch/armv7/Makefile.inc
@@ -58,7 +58,7 @@ $(obj)/coreboot.pre1: $(CBFSTOOL)
mv $(obj)/coreboot.rom $@
endif
-$(obj)/coreboot.rom: $(obj)/coreboot.pre $(objcbfs)/coreboot_ram.elf $(CBFSTOOL) $(call strip_quotes,$(COREBOOT_ROM_DEPENDENCIES))
+$(obj)/coreboot.rom: $(obj)/coreboot.pre $(objcbfs)/coreboot_ram.elf $(CBFSTOOL) $(call strip_quotes,$(COREBOOT_ROM_DEPENDENCIES)) $$(INTERMEDIATE)
@printf " CBFS $(subst $(obj)/,,$(@))\n"
cp $(obj)/coreboot.pre $@.tmp
if [ -f $(objcbfs)/coreboot_ap.elf ]; \
the following patch was just integrated into master:
commit 9027845d65e5e05a45894eb176070fd99e2e073c
Author: Aladyshev Konstantin <aladyshev(a)nicevt.ru>
Date: Tue Dec 18 22:44:01 2012 +0400
rd890: clear IO-APIC before setup
Add function "clear_ioapic" before "setup_ioapic" for RD890 northbridge
like it is done for SB700 and SB800 chipsets ("amd/cimx/sb{7,8}00").
No functionality change is noticed.
Change-Id: I1fd87692d8bf35c166141c9b7a6a1e748c19a636
Signed-off-by: Aladyshev Konstantin <aladyshev(a)nicevt.ru>
Reviewed-on: http://review.coreboot.org/2045
Tested-by: build bot (Jenkins)
Reviewed-by: Zheng Bao <zheng.bao(a)amd.com>
Reviewed-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Build-Tested: build bot (Jenkins) at Wed Dec 19 18:47:57 2012, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer(a)coreboot.org> at Thu Jan 3 23:45:08 2013, giving +2
Reviewed-By: Zheng Bao <zheng.bao(a)amd.com> at Wed Jan 2 11:13:36 2013, giving +1
See http://review.coreboot.org/2045 for details.
-gerrit
the following patch was just integrated into master:
commit 05cbce672e4a41bc55fc09b65aacd0872a756a5b
Author: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Date: Thu Jan 3 14:30:33 2013 -0800
cbmem utility: Use mmap instead of fseek/fread
The kernel on Ubuntu 12.04LTS does not allow to use
fseek/fread to read the coreboot table at the end of
memory but will instead abort cbmem with a "Bad Address"
error.
Whether that is a security feature (some variation of
CONFIG_STRICT_DEVMEM) or a kernel bug is not yet clear,
however using mmap works nicely.
Change-Id: I796b4cd2096fcdcc65c1361ba990cd467f13877e
Signed-off-by: Stefan Reinauer <reinauer(a)google.com>
Reviewed-on: http://review.coreboot.org/2097
Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com>
Tested-by: build bot (Jenkins)
Build-Tested: build bot (Jenkins) at Thu Jan 3 23:55:16 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich(a)gmail.com> at Thu Jan 3 23:53:35 2013, giving +2
See http://review.coreboot.org/2097 for details.
-gerrit