Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/51793 )
Change subject: soc/amd/common/block/i2c: Fix printf format specifiers ......................................................................
soc/amd/common/block/i2c: Fix printf format specifiers
The correct printf format specifier for an `unsigned int` is `%u`.
Change-Id: Iaf780eb366f8c3493b89beb9a5643fa285e7825d Signed-off-by: Angel Pons th3fanbus@gmail.com --- M src/soc/amd/common/block/i2c/i2c.c 1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/51793/1
diff --git a/src/soc/amd/common/block/i2c/i2c.c b/src/soc/amd/common/block/i2c/i2c.c index ec05e49..30fa6e7 100644 --- a/src/soc/amd/common/block/i2c/i2c.c +++ b/src/soc/amd/common/block/i2c/i2c.c @@ -21,7 +21,7 @@ const struct soc_i2c_ctrlr_info *ctrlr = soc_get_i2c_ctrlr_info(&num_ctrlrs);
if (bus >= num_ctrlrs) { - printk(BIOS_ERR, "Bus ID %d is >= number of I2C controllers %zu\n", + printk(BIOS_ERR, "Bus ID %u is >= number of I2C controllers %zu\n", bus, num_ctrlrs); return 0; } @@ -35,7 +35,7 @@ const struct dw_i2c_bus_config *cfg = soc_get_i2c_bus_config(&num_buses);
if (bus >= num_buses) { - printk(BIOS_ERR, "Bus ID %d is >= number of I2C buses %zu\n", bus, num_buses); + printk(BIOS_ERR, "Bus ID %u is >= number of I2C buses %zu\n", bus, num_buses); return NULL; }
@@ -101,7 +101,7 @@ continue;
if (dw_i2c_init(bus, cfg)) { - printk(BIOS_ERR, "Failed to init i2c bus %d\n", bus); + printk(BIOS_ERR, "Failed to init i2c bus %u\n", bus); continue; }