[coreboot-gerrit] Patch set updated for coreboot: commonlib/fsp.h: include sys/types.h for ssize_t
Martin Roth (martinroth@google.com)
gerrit at coreboot.org
Wed Feb 22 22:56:55 CET 2017
Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18417
-gerrit
commit 01e6e0e8939793119119fe963c752d35a12a05a4
Author: Jonathan Neuschäfer <j.neuschaefer at gmx.net>
Date: Tue Feb 21 13:10:05 2017 +0100
commonlib/fsp.h: include sys/types.h for ssize_t
This file reportedly didn't compile on SUSE Linux with gcc 4.3.4:
[...]
> HOSTCC cbfstool/fsp_relocate.o
> In file included from coreboot/src/commonlib/fsp_relocate.c:18:
> coreboot/src/commonlib/include/commonlib/fsp.h:26: error:
> expected '=', ',', ';', 'asm' or '__attribute__' before
> 'fsp_component_relocate'
[...]
According to POSIX-2008[1], sys/types.h defines ssize_t, so include it.
This should not break coreboot code (as opposed to utils code), as we
have a sys/types.h in src/include.
[1]: http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/sys_types.h.html
Change-Id: Id3694dc76c41d800ba09183e4b039b0719ac3d93
Signed-off-by: Jonathan Neuschäfer <j.neuschaefer at gmx.net>
---
src/commonlib/include/commonlib/fsp.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/commonlib/include/commonlib/fsp.h b/src/commonlib/include/commonlib/fsp.h
index 84e2c98..a2a2fae 100644
--- a/src/commonlib/include/commonlib/fsp.h
+++ b/src/commonlib/include/commonlib/fsp.h
@@ -18,6 +18,7 @@
#include <stddef.h>
#include <stdint.h>
+#include <sys/types.h>
/*
* Relocate FSP held within buffer defined by size to new_addr. Returns < 0
More information about the coreboot-gerrit
mailing list