Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8197
-gerrit
commit 9c94514ceb638689b7a83aa3d50c2558629337b7
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Tue Jan 13 04:15:29 2015 +1100
Makefile: clang - ramp up some more warnings
Back out '-Wno-unused-variable'.
Change-Id: Icd15b03d3a41f4933c9ae0a4c497ccadcccbefb5
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index 017ca47..8fdd5d3 100644
--- a/Makefile
+++ b/Makefile
@@ -119,7 +119,7 @@ ifeq ($(CONFIG_COMPILER_LLVM_CLANG),y)
# this means the triple is i386-linux-elf instead of i386-none-elf
CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32
# Tone down some clang warnings
-CFLAGS_x86_32 += -Wno-unused-variable -Wno-unused-function -Wno-tautological-compare -Wno-shift-overflow
+CFLAGS_x86_32 += -Wno-unused-function -Wno-tautological-compare -Wno-shift-overflow
CC_x86_32:=clang
HOSTCC := clang
Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4821
-gerrit
commit 7e87866e3a5f82539c6e095dbd1f1f86d4c9e1b9
Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com>
Date: Sun Jan 26 11:18:27 2014 -0600
google/stout: Sanitize cmos.layout
Remove 'baud_rate' and 'hyper_threading' for the same reason described in:
* 74230c3 google/butterfly: Remove unused cmos.layout options
Also add mrc_scrambler_seed_chk, for the same reason as in:
* 655ac24 google/butterfly: Declare mrc_scrambler_seed_chk in cmos.layout
Change-Id: I857d7a52917c6bbb62b53179f2b69d78bc297ea8
Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com>
---
src/mainboard/google/stout/cmos.layout | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/mainboard/google/stout/cmos.layout b/src/mainboard/google/stout/cmos.layout
index 1993be7..da6f114 100644
--- a/src/mainboard/google/stout/cmos.layout
+++ b/src/mainboard/google/stout/cmos.layout
@@ -74,12 +74,14 @@ entries
# -----------------------------------------------------------------
# coreboot config options: console
-392 3 e 5 baud_rate
+# No serial port on this motherboard
+#392 3 e 5 baud_rate
395 4 e 6 debug_level
#399 1 r 0 unused
# coreboot config options: cpu
-400 1 e 2 hyper_threading
+# hyper_threading not supported by the Celeron 847 on this board
+#400 1 e 2 hyper_threading
#401 7 r 0 unused
# coreboot config options: southbridge
@@ -100,6 +102,7 @@ entries
# SandyBridge MRC Scrambler Seed values
896 32 r 0 mrc_scrambler_seed
928 32 r 0 mrc_scrambler_seed_s3
+960 16 r 0 mrc_scrambler_seed_chk
# coreboot config options: check sums
984 16 h 0 check_sum
the following patch was just integrated into master:
commit 29445dc44ee6bfb2617443e0faa985a93d9272f2
Author: Zhuo-Hao Lee <zhuo-hao.lee(a)intel.com>
Date: Wed Dec 24 11:13:34 2014 +0800
device/oprom/realmode/x86: Fix memory corruption
The length of the memcpy is incorrect and this will cause the
destination buffer to corrupt the following 2 bytes of data.
BUG=none
BRANCH=All
TEST=build and boot on rambi, system boot up without error
Change-Id: I96adf2555b01aa35bb38a2e0f221fc2b2e87a41b
Signed-off-by: Zhuo-Hao Lee <zhuo-hao.lee(a)intel.com>
Reviewed-on: https://chromium-review.googlesource.com/237510
Reviewed-by: Ryan Lin <ryan.lin(a)intel.com>
Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org>
[Remove usage of macro `FIELD_SIZEOF(t, f)`.]
Signed-off-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
Reviewed-on: http://review.coreboot.org/8227
Reviewed-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com>
Tested-by: build bot (Jenkins)
See http://review.coreboot.org/8227 for details.
-gerrit