[coreboot-gerrit] Change in coreboot[master]: include, lib: Add <inttypes.h> printf macros

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33823 )

Change subject: include, lib: Add <inttypes.h> printf macros
......................................................................

Patch Set 14:

(2 comments)

https://review.coreboot.org/c/coreboot/+/33823/14//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/33823/14//COMMIT_MSG@11 
PS14, Line 11: useful for portably printing them. Of these definitions, coreboot so far has
             : only used PRIu64 (in one place), which isn't needed anymore since we know
overlong lines btw.

https://review.coreboot.org/c/coreboot/+/33823/14/src/lib/timestamp.c 
File src/lib/timestamp.c:

https://review.coreboot.org/c/coreboot/+/33823/14/src/lib/timestamp.c@184 
PS14, Line 184: 				timestamp_name(id), ts_time);
> Mostly to be consistent with the rest of the code base, which doesn't use PRIu64 anywhere else (or a […]

Well, I would see PRIu64 as a courtesy to the reviewer, so they don't
have to know our uint64_t implementation details. Consistency is a good
argument, too. Feel free to remove the line break, though.