Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54343 )
Change subject: cpu/x86/smm: Fix u32 type mismatch in print statement ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1: Hmm, worked on my system:
``` CC ramstage/cpu/x86/smm/smm_module_loader.o src/cpu/x86/smm/smm_module_loader.c: In function 'smm_module_setup_stub': src/cpu/x86/smm/smm_module_loader.c:415:21: error: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'long unsigned int' [-Werror=format=] printk(BIOS_DEBUG, "%s: stack_end = 0x%" PRIu32 "\n", ^~~~~~~~~~~~~~~~~~~~~ __func__, stub_params->stack_top - total_stack_size); ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/cpu/x86/smm/smm_module_loader.c:4: src/include/inttypes.h:27:18: note: format string is defined here #define PRIu32 "u" src/cpu/x86/smm/smm_module_loader.c: At top level: cc1: error: unrecognized command line option '-Wno-address-of-packed-member' [-Werror] ```