erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

soc/amd/common/psp/rpmc: fix printk format string

Browse source 
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 <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/85251
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Felix Held 2024-11-22 15:33:12 +01:00
commit 179945291c
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/soc/amd/common/block/psp/rpmc.c
View file

@ -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");
}