Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5317
-gerrit
commit 6a9a5dab086a9b720b8a6344e773184d89a75064
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Mon Mar 3 02:23:18 2014 +1100
Trivial: Append .gitignore with .ccwrap.
Change-Id: I5757023b844e965d797e6b1a7e6955940f6d3363
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
.gitignore | 1 +
1 file changed, 1 insertion(+)
diff --git a/.gitignore b/.gitignore
index 0ad15ef..96af4a6 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,6 +1,7 @@
.config
.config.old
.xcompile
+.ccwrap
build/
coreboot-builds/
payloads/external/FILO/filo/
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5314
-gerrit
commit 033958f02d1bb600163b333168d360ba047399fc
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Sat Mar 1 09:27:37 2014 +1100
lib/selfboot: s_srcaddr is uninitialized.
s_srcaddr is uninitialized in the BSS section, leading to a
garbage valued operand on the LHS of a '<' on line 383.
Change-Id: Ie4fec91b09c70fb1d91ad3918ac3f60653fa1d83
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/lib/selfboot.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/lib/selfboot.c b/src/lib/selfboot.c
index 222eae2..98ed66d 100644
--- a/src/lib/selfboot.c
+++ b/src/lib/selfboot.c
@@ -352,6 +352,9 @@ static int build_self_segment_list(
ntohl(segment->mem_len));
new = malloc(sizeof(*new));
new->s_filesz = 0;
+ new->s_srcaddr = (uintptr_t)
+ ((unsigned char *)first_segment)
+ + ntohl(segment->offset);
new->s_dstaddr = ntohll(segment->load_addr);
new->s_memsz = ntohl(segment->mem_len);
break;
the following patch was just integrated into master:
commit 44208ea58ca04c9e2ce434932a8398ea7ab934d6
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Fri Feb 21 12:43:07 2014 +1100
vendorcode/amd/agesa/f*: Improve gcccar.inc assembler compatibility.
A comparison with a two's complement in gcccar.inc has dubious
GAS/AT&T notation. Clang miss-parses 0x-1 as an invalid hexadecimal
number.
Change-Id: I88baa5c2513f062ff309df05916a3832b9bd9bb1
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
See http://review.coreboot.org/5277 for details.
-gerrit
the following patch was just integrated into master:
commit 13f65f0e4ef0075193314abb3b70d26497a0f5eb
Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com>
Date: Wed Jan 29 16:50:40 2014 -0600
cbfstool/lzma: Remove dead code under #ifdefs
Remove a bunch of dead code which depends either on commented out
#defines, or compiler definitions. Use this opportunity to remove the
need for "-D_7ZIP_ST" in the compiler flags.
Change-Id: Ib6629002be7bf4cee6d95d7baa724893b5e8ba32
Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com>
See http://review.coreboot.org/5083 for details.
-gerrit