diff options
author | Evan Green <evgreen@chromium.org> | 2019-09-23 11:02:41 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-10-05 00:47:44 +0000 |
commit | e34fca3e01d75552ad8d712879c3ccd6a6168584 (patch) | |
tree | e2a71c2fa4dd52ffe459d10156276fbb0b5aa597 /include | |
parent | a1216326c5d58af300b7c6f24c8597a232ced131 (diff) | |
download | chrome-ec-e34fca3e01d75552ad8d712879c3ccd6a6168584.tar.gz |
builtin: Introduce and use inttypes.h
In order to pass the right printf format specifiers for certain types
that are compiled both in 32-bit EC and 64-bit host environments,
standard macros PRIx64 and PRId64 must be introduced. These specify
the correct printf format specifier in the given compilation environment
for printing a 64-bit value.
On the host, inttypes.h already exists. Add an inttypes.h for the EC
codebase so that these macros can be used where they're needed.
BUG=chromium:984041
TEST=make -j buildall
BRANCH=none
Change-Id: I76e3bdc88aef7da6e5234d5b86b595f7138ea9a1
Signed-off-by: Evan Green <evgreen@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1819642
Reviewed-by: caveh jalali <caveh@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/common.h | 1 | ||||
-rw-r--r-- | include/host_command.h | 6 |
2 files changed, 5 insertions, 2 deletions
diff --git a/include/common.h b/include/common.h index 7bf4b39215..ee95bf5f15 100644 --- a/include/common.h +++ b/include/common.h @@ -9,6 +9,7 @@ #define __CROS_EC_COMMON_H #include <stdint.h> +#include <inttypes.h> #include "compile_time_macros.h" /* diff --git a/include/host_command.h b/include/host_command.h index 2be94cdb83..10d7fc37ea 100644 --- a/include/host_command.h +++ b/include/host_command.h @@ -125,8 +125,10 @@ struct host_command { #ifdef CONFIG_HOST_EVENT64 typedef uint64_t host_event_t; -#define HOST_EVENT_CPRINTS(str, e) CPRINTS("%s 0x%016lx", str, e) -#define HOST_EVENT_CCPRINTF(str, e) ccprintf("%s 0x%016lx\n", str, e) +#define HOST_EVENT_CPRINTS(str, e) CPRINTS("%s 0x%016" PRIx64, str, e) +#define HOST_EVENT_CCPRINTF(str, e) \ + ccprintf("%s 0x%016" PRIx64 "\n", str, e) + #else typedef uint32_t host_event_t; #define HOST_EVENT_CPRINTS(str, e) CPRINTS("%s 0x%08x", str, e) |