From 179945291ccecab8d5d2c667d1af8f22d6aa8c66 Mon Sep 17 00:00:00 2001 From: Felix Held Date: Fri, 22 Nov 2024 15:33:12 +0100 Subject: [PATCH] soc/amd/common/psp/rpmc: fix printk format string While gcc didn't seem to care about that mismatch, clang didn't like that '%ld' was used in the printk format string to print a size_t variable. Replace the correct '%zu' instead of '%ld' to fix that. Change-Id: I32bc584abe835c9c1d732c12311881345b8f0cdf Signed-off-by: Felix Held Reviewed-on: https://review.coreboot.org/c/coreboot/+/85251 Reviewed-by: Marshall Dawson Tested-by: build bot (Jenkins) --- src/soc/amd/common/block/psp/rpmc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/soc/amd/common/block/psp/rpmc.c b/src/soc/amd/common/block/psp/rpmc.c index dd097c52e3..99e166ad17 100644 --- a/src/soc/amd/common/block/psp/rpmc.c +++ b/src/soc/amd/common/block/psp/rpmc.c @@ -78,12 +78,12 @@ static void print_rpmc_general_status(uint8_t healthy, uint8_t rpmc_protected) static void print_spi_rpmc_usage(uint8_t available, uint8_t used) { for (size_t i = 0; i < SPI_RPMC_COUNTER_COUNT; i++) { - printk(BIOS_SPEW, "SPI flash RPMC counter %ld %s provisioned\n", i, + printk(BIOS_SPEW, "SPI flash RPMC counter %zu %s provisioned\n", i, available & BIT(i) ? "can still be" : "has already been"); } for (size_t i = 0; i < SPI_RPMC_COUNTER_COUNT; i++) { - printk(BIOS_SPEW, "SPI flash RPMC counter %ld is%s in use\n", i, + printk(BIOS_SPEW, "SPI flash RPMC counter %zu is%s in use\n", i, used & BIT(i) ? "" : " not"); } } @@ -97,7 +97,7 @@ static void print_rpmc_rev0_status(union psp_rpmc_caps psp_caps) print_spi_rpmc_usage(psp_caps.r0.spi_rpmc_slots_available, psp_caps.r0.spi_rpmc_slot_used); for (size_t i = 0; i < PSP_RPMC_R0_SLOT_COUNT; i++) { - printk(BIOS_SPEW, "SoC RPMC slot %ld %s provisioned\n", i, + printk(BIOS_SPEW, "SoC RPMC slot %zu %s provisioned\n", i, psp_caps.r0.psp_rpmc_slot_available & BIT(i) ? "can still be" : "has already been"); }