erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions
coreboot/src/soc/intel/common
History
Paul Menzel 0594bf87c1 soc/intel/common: irq: Use correct size_t length modifier 
Building an image for the Purism Mini v2 with `x86_64-linux-gnu-gcc-11`
fails with the format warning below as the size of size_t differs
between 32-bit and 64-bit.

        CC         ramstage/soc/intel/common/block/irq/irq.o
    src/soc/intel/common/block/irq/irq.c: In function 'assign_fixed_pirqs':
    src/soc/intel/common/block/irq/irq.c:186:90: error: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'size_t' {aka 'unsigned int'} [-Werror=format=]
      186 |                         printk(BIOS_ERR, "ERROR: Slot %u, pirq %u, no pin for function %lu\n",
          |                                                                                        ~~^
          |                                                                                          |
          |                                                                                          long unsigned int
          |                                                                                        %u
      187 |                                constraints->slot, fixed_pirq, i);
          |                                                               ~
          |                                                               |
          |                                                               size_t {aka unsigned int}
        CC         ramstage/soc/intel/common/block/gspi/gspi.o
        CC         ramstage/soc/intel/common/block/graphics/graphics.o
        CC         ramstage/soc/intel/common/block/gpio/gpio.o
        CC         ramstage/soc/intel/common/block/gpio/gpio_dev.o

The variable `i` is of type size_t, so use the corresponding length
modifier `z`.

Fixes: b59980b54e ("soc/intel/common: Add new IRQ module")
Change-Id: I09f4a8d22a2964471344f5dcf971dfa801555f4a
Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/59056
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
2022-01-04 14:54:33 +00:00
..
acpi soc/intel: generate SSDT instead of using GNVS for SGX 2021-11-09 16:02:19 +00:00
basecode
block soc/intel/common: irq: Use correct size_t length modifier 2022-01-04 14:54:33 +00:00
pch soc/intel/common/pch: Fix return value documentation for CHIPSET_LOCKDOWN 2021-12-08 22:29:52 +00:00
fsp_reset.c
hda_verb.c soc/intel/common/hda_verb.c: Fix up comment style 2021-03-22 12:59:57 +00:00
hda_verb.h
Kconfig.common soc/intel/common: Do not trigger crashlog on all resets by default 2021-12-20 17:49:53 +00:00
Makefile.inc src/soc to src/superio: Fix spelling errors 2021-10-05 18:07:08 +00:00
mma.c intel: mma: Use new CBFS API 2021-04-14 01:03:33 +00:00
mma.h intel: mma: Use new CBFS API 2021-04-14 01:03:33 +00:00
nhlt.c
reset.c
reset.h
smbios.c arch/x86: Refactor the SMBIOS type 17 write function 2021-11-11 09:10:10 +00:00
smbios.h
tpm_tis.c
vbt.c
vbt.h