Change in coreboot[master]: soc/intel/xeon_sp: Fix size_t type mismatch in print statement

[Paul Fagerburg (Code Review)]

Paul Fagerburg has submitted this change. ( https://review.coreboot.org/c/coreboot/+/59054 )

Change subject: soc/intel/xeon_sp: Fix size_t type mismatch in print statement
......................................................................

soc/intel/xeon_sp: Fix size_t type mismatch in print statement

The 64-bit compiler x86_64-linux-gnu-gcc-10 aborts the build with the format
warning below:

CC         romstage/soc/intel/xeon_sp/memmap.o
    src/soc/intel/xeon_sp/memmap.c: In function 'fill_postcar_frame':
    src/soc/intel/xeon_sp/memmap.c:39:62: error: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' {aka 'unsigned int'} [-Werror=format=]
       39 |         printk(BIOS_DEBUG, "cbmem base_ptr: 0x%lx, size: 0x%lx\n", cbmem_base, cbmem_size);
          |                                                            ~~^                 ~~~~~~~~~~
          |                                                              |                 |
          |                                                              long unsigned int size_t {aka unsigned int}
          |                                                            %x

As `cbmem_size` is of type `size_t` use the appropriate length modifier
`z`.

Change-Id: I1ca77de1ce33ce1e97d7c8895c6e75424f0769f5
Found-by: gcc (Debian 11.2.0-10) 11.2.0
Signed-off-by: Paul Menzel 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/59054
Tested-by: build bot (Jenkins) 
Reviewed-by: Felix Held 
Reviewed-by: Lance Zhao
---
M src/soc/intel/xeon_sp/memmap.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Felix Held: Looks good to me, approved
  Lance Zhao: Looks good to me, approved

diff --git a/src/soc/intel/xeon_sp/memmap.c b/src/soc/intel/xeon_sp/memmap.c
index 0af0ad2..b640c2d 100644
--- a/src/soc/intel/xeon_sp/memmap.c
+++ b/src/soc/intel/xeon_sp/memmap.c
@@ -36,7 +36,7 @@
    /* Try account for the CBMEM region currently used and for future use */
    cbmem_get_region((void **)&cbmem_base, &cbmem_size);
    printk(BIOS_DEBUG, "top_of_ram = 0x%lx\n", top_of_ram);
-	printk(BIOS_DEBUG, "cbmem base_ptr: 0x%lx, size: 0x%lx\n", cbmem_base, cbmem_size);
+	printk(BIOS_DEBUG, "cbmem base_ptr: 0x%lx, size: 0x%zx\n", cbmem_base, cbmem_size);
    /* Assume 4MiB will be enough for future cbmem objects (FSP-S, ramstage, ...) */
    cbmem_base -= 4 * MiB;
    cbmem_base = ALIGN_DOWN(cbmem_base, 4 * MiB);

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.