Attention is currently required from: Patrick Rudolph. Paul Menzel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/59057 )
Change subject: soc/intel/common/acpi/pep: Fix type mismatch error ......................................................................
soc/intel/common/acpi/pep: Fix type mismatch error
CC ramstage/soc/intel/common/block/acpi/pep.o CC ramstage/soc/intel/common/fsp_reset.o CC ramstage/soc/intel/common/hda_verb.o src/soc/intel/common/block/acpi/pep.c: In function 'read_pmc_lpm_requirements': src/soc/intel/common/block/acpi/pep.c:57:50: error: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t' {aka 'unsigned int'} [-Werror=format=] 57 | printk(BIOS_ERR, "Failed to retrieve LPM substate registers" | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 58 | "from LPM, substate %lu, reg %lu\n", i, j); | ~ | | | size_t {aka unsigned int} src/soc/intel/common/block/acpi/pep.c:58:62: note: format string is defined here 58 | "from LPM, substate %lu, reg %lu\n", i, j); | ~~^ | | | long unsigned int | %u src/soc/intel/common/block/acpi/pep.c:57:50: error: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' {aka 'unsigned int'} [-Werror=format=] 57 | printk(BIOS_ERR, "Failed to retrieve LPM substate registers" | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 58 | "from LPM, substate %lu, reg %lu\n", i, j); | ~ | | | size_t {aka unsigned int} src/soc/intel/common/block/acpi/pep.c:58:71: note: format string is defined here 58 | "from LPM, substate %lu, reg %lu\n", i, j); | ~~^ | | | long unsigned int | %u
Change-Id: I27bce0a6c62b1c1ebbca732761de2f59b042a5d4 Signed-off-by: Paul Menzel pmenzel@molgen.mpg.de --- M src/soc/intel/common/block/acpi/pep.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/59057/1
diff --git a/src/soc/intel/common/block/acpi/pep.c b/src/soc/intel/common/block/acpi/pep.c index 9774239..8419a33 100644 --- a/src/soc/intel/common/block/acpi/pep.c +++ b/src/soc/intel/common/block/acpi/pep.c @@ -55,7 +55,7 @@ enum cb_err result = pmc_send_ipc_cmd(cmd_reg, &req, &res); if (result != CB_SUCCESS) { printk(BIOS_ERR, "Failed to retrieve LPM substate registers" - "from LPM, substate %lu, reg %lu\n", i, j); + "from LPM, substate %zu, reg %zu\n", i, j); free(reg); return; }