Attention is currently required from: Ed Sharma, Angel Pons.
Hello Ed Sharma, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/69682
to look at the new patch set (#2).
Change subject: src/commonlib/fsp_relocate.c: Fix cbfstool build on 32 bit host ......................................................................
src/commonlib/fsp_relocate.c: Fix cbfstool build on 32 bit host
On a 32 bit host, uintptr_t is defined as 'unsigned int' instead of 'unsigend long int' like on a 64 bit host. When cbfstool is build on a 32 bit host, the printk format specifier '%lx' expects a 'long int' while new_addr is of type 'uintptr_t', aka 'unsigned int'. This in the end leads to a build error.
To fix this and make it build on both, 32 and 64 bit hosts, cast new_addr to (void *) and use %p as the format specifier.
Test=Build cbfstool on 32 and 64 bit hosts.
Change-Id: Ia917d2ed31778f3a29c0a6c7368f74c15319b099 Signed-off-by: Werner Zeh werner.zeh@siemens.com --- M src/commonlib/fsp_relocate.c 1 file changed, 22 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/82/69682/2