[coreboot-gerrit] Patch set updated for coreboot: util/cbfstool: Fix to build with latest llvm

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Nov 28 12:15:33 CET 2016


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17568

-gerrit

commit 203956165e4257c4cc04da2cc879288169d0b6a0
Author: Manoj Gupta <manojgupta at google.com>
Date:   Mon Nov 14 11:19:30 2016 -0800

    util/cbfstool: Fix to build with latest llvm
    
    cbfs-payload-linux.c:255:43: note: add parentheses around left
    hand side expression to silence this warning
    
    if ((hdr->protocol_version >= 0x200) && (!hdr->loadflags & 1)) {
    
    BUG=chromium:665657
    TEST=coreboot-utils builds
    
    Change-Id: I025c784330885cce8ae43c44f9d938394af30ed5
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 35c4935f2a89c3d3b45213372bcf0474a60eda43
    Original-Change-Id: I8758e7d158ca32e87107797f2a33b9d9a0e4676f
    Original-Reviewed-on: https://chromium-review.googlesource.com/411335
    Original-Commit-Ready: Manoj Gupta <manojgupta at chromium.org>
    Original-Tested-by: Manoj Gupta <manojgupta at chromium.org>
    Original-Reviewed-by: Mike Frysinger <vapier at chromium.org>
---
 util/cbfstool/cbfs-payload-linux.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/cbfstool/cbfs-payload-linux.c b/util/cbfstool/cbfs-payload-linux.c
index afb0f3a..ac7e729 100644
--- a/util/cbfstool/cbfs-payload-linux.c
+++ b/util/cbfstool/cbfs-payload-linux.c
@@ -252,7 +252,7 @@ int parse_bzImage_to_payload(const struct buffer *input,
 	}
 
 	unsigned long kernel_base = 0x100000;
-	if ((hdr->protocol_version >= 0x200) && (!hdr->loadflags & 1)) {
+	if ((hdr->protocol_version >= 0x200) && (!hdr->loadflags)) {
 		kernel_base = 0x1000; /* zImage kernel */
 	}
 	/* kernel prefers an address, so listen */



More information about the coreboot-gerrit mailing list