Alex James has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/60228 )
Change subject: util/cbfstool: Remove redundant endian.h include ......................................................................
util/cbfstool: Remove redundant endian.h include
flashmap/fmap.c includes commonlib/bsd/sysincludes.h, which already includes the necessary header for endian(3) functions (endian.h on Linux and sys/endian.h on FreeBSD). This also resolves a compilation error on macOS (tested on 10.5.7), as macOS does not provide endian.h.
Change-Id: I0cb17eacd253605b75db8cf734e71ca3fe24ad6c Signed-off-by: Alex James theracermaster@gmail.com --- M util/cbfstool/flashmap/fmap.c 1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/28/60228/1
diff --git a/util/cbfstool/flashmap/fmap.c b/util/cbfstool/flashmap/fmap.c index cb67b46..7010dbc 100644 --- a/util/cbfstool/flashmap/fmap.c +++ b/util/cbfstool/flashmap/fmap.c @@ -1,7 +1,6 @@ /* SPDX-License-Identifier: BSD-3-Clause or GPL-2.0-only */
#include <ctype.h> -#include <endian.h> #include <stdlib.h> #include <stdio.h> #include <string.h>