From 312d455a93ca6918c9127eb8e64c2e1d953d0e3a Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Tue, 28 Oct 2025 12:59:39 +0100 Subject: [PATCH] soc/intel/{adl,mtl}/romstage/fsp_params.c: Fix printf specifier `cfg[i].clk_req` is a `uint8_t` so use `%u` instead of `%d`. Change-Id: I6c7a6ecbd2f5b917d44923d0ad6cb331d9bb054c Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/89789 Reviewed-by: Matt DeVillier Tested-by: build bot (Jenkins) Reviewed-by: Kilian Krause Reviewed-by: Paul Menzel Reviewed-by: Mario Scheithauer --- src/soc/intel/alderlake/romstage/fsp_params.c | 2 +- src/soc/intel/meteorlake/romstage/fsp_params.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/soc/intel/alderlake/romstage/fsp_params.c b/src/soc/intel/alderlake/romstage/fsp_params.c index 6ab8bb3e50..30ec714b50 100644 --- a/src/soc/intel/alderlake/romstage/fsp_params.c +++ b/src/soc/intel/alderlake/romstage/fsp_params.c @@ -79,7 +79,7 @@ static void pcie_rp_init(FSP_M_CONFIG *m_cfg, uint32_t en_mask, enum pcie_rp_typ if (!(cfg[i].flags & PCIE_RP_CLK_REQ_UNUSED)) { if (clk_req_mapping & (1 << cfg[i].clk_req)) printk(BIOS_WARNING, - "Found overlapped clkreq assignment on clk req %d\n", + "Found overlapped clkreq assignment on clk req %u\n", cfg[i].clk_req); m_cfg->PcieClkSrcClkReq[cfg[i].clk_src] = cfg[i].clk_req; clk_req_mapping |= 1 << cfg[i].clk_req; diff --git a/src/soc/intel/meteorlake/romstage/fsp_params.c b/src/soc/intel/meteorlake/romstage/fsp_params.c index 414c270dcd..0543106442 100644 --- a/src/soc/intel/meteorlake/romstage/fsp_params.c +++ b/src/soc/intel/meteorlake/romstage/fsp_params.c @@ -59,7 +59,7 @@ static void pcie_rp_init(FSP_M_CONFIG *m_cfg, uint32_t en_mask, if (!(cfg[i].flags & PCIE_RP_CLK_REQ_UNUSED)) { if (clk_req_mapping & (1 << cfg[i].clk_req)) printk(BIOS_WARNING, - "Found overlapped clkreq assignment on clk req %d\n", + "Found overlapped clkreq assignment on clk req %u\n", cfg[i].clk_req); m_cfg->PcieClkSrcClkReq[cfg[i].clk_src] = cfg[i].clk_req; clk_req_mapping |= 1 << cfg[i].clk_req;