[coreboot-gerrit] New patch to review for coreboot: cbfstool: don't use endian to fix BSD hosts

Aaron Durbin (adurbin@chromium.org) gerrit at coreboot.org
Mon Sep 21 18:12:19 CET 2015


Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11695

-gerrit

commit 28ea67f1f8993a7c3f875d75888d3b220fef2bd2
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Mon Sep 21 13:10:13 2015 -0500

    cbfstool: don't use endian to fix BSD hosts
    
    endian.h lives in under sys on the BSDs. Replace htole32() with
    swab32(htonl(..)) as a proxy for little endian operations.
    
    Change-Id: I84a88f6882b6c8f14fb089e4b629e916386afe4d
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 util/cbfstool/cbfstool.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/util/cbfstool/cbfstool.c b/util/cbfstool/cbfstool.c
index ed6e898..5194061 100644
--- a/util/cbfstool/cbfstool.c
+++ b/util/cbfstool/cbfstool.c
@@ -19,7 +19,6 @@
  * Foundation, Inc.
  */
 
-#include <endian.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -272,7 +271,7 @@ static int cbfs_add_master_header(void)
 	// TODO: when we have a BE target, we'll need to store this as BE
 	*(uint32_t *)(buffer_get(&image.buffer) +
 		buffer_size(&image.buffer) - 4) =
-		htole32(header_offset);
+		swab32(htonl(header_offset));
 
 	ret = 0;
 



More information about the coreboot-gerrit mailing list