Idwer Vollering has submitted this change. ( https://review.coreboot.org/c/coreboot/+/48977 )
Change subject: util/amdfwtool: portability fixes for FreeBSD ......................................................................
util/amdfwtool: portability fixes for FreeBSD
Add the stdint.h header, and drop the GLIBC section from amdfwtool.h to build this tool on FreeBSD as well as Linux.
Signed-off-by: Idwer Vollering vidwer@gmail.com Change-Id: I295fd308b0f5e2902931f02c9455823a614976de Reviewed-on: https://review.coreboot.org/c/coreboot/+/48977 Reviewed-by: Felix Held felix-coreboot@felixheld.de Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M util/amdfwtool/amdfwtool.c M util/amdfwtool/amdfwtool.h M util/amdfwtool/data_parse.c 3 files changed, 2 insertions(+), 8 deletions(-)
Approvals: build bot (Jenkins): Verified Felix Held: Looks good to me, approved
diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c index f167e5e..8b54876 100644 --- a/util/amdfwtool/amdfwtool.c +++ b/util/amdfwtool/amdfwtool.c @@ -57,6 +57,7 @@ #include <stdlib.h> #include <getopt.h> #include <libgen.h> +#include <stdint.h>
#include "amdfwtool.h"
diff --git a/util/amdfwtool/amdfwtool.h b/util/amdfwtool/amdfwtool.h index 6ae4675..b12b989 100644 --- a/util/amdfwtool/amdfwtool.h +++ b/util/amdfwtool/amdfwtool.h @@ -3,14 +3,6 @@ #ifndef _AMD_FW_TOOL_H_ #define _AMD_FW_TOOL_H_
-#if defined(__GLIBC__) -typedef unsigned long long int uint64_t; -typedef unsigned int uint32_t; -typedef unsigned char uint8_t; -typedef unsigned short uint16_t; -#endif - - typedef enum _amd_fw_type { AMD_FW_PSP_PUBKEY = 0, AMD_FW_PSP_BOOTLOADER = 1, diff --git a/util/amdfwtool/data_parse.c b/util/amdfwtool/data_parse.c index 017c689..b6e14a8 100644 --- a/util/amdfwtool/data_parse.c +++ b/util/amdfwtool/data_parse.c @@ -2,6 +2,7 @@ #include <regex.h> #include <string.h> #include <stdlib.h> +#include <stdint.h>
#include "amdfwtool.h"