build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43756 )
Change subject: cast all void *cbmem_top_chipset() function to uintptr_t ......................................................................
Patch Set 2:
(6 comments)
https://review.coreboot.org/c/coreboot/+/43756/2/src/mainboard/emulation/qem... File src/mainboard/emulation/qemu-aarch64/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/2/src/mainboard/emulation/qem... PS2, Line 14: return (uintptr_t)(_dram + (probe_ramsize((uintptr_t)_dram, CONFIG_DRAM_SIZE_MB) * MiB)); line over 96 characters
https://review.coreboot.org/c/coreboot/+/43756/2/src/mainboard/emulation/qem... File src/mainboard/emulation/qemu-armv7/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/2/src/mainboard/emulation/qem... PS2, Line 10: return (uintptr_t)(_dram + (probe_ramsize((uintptr_t)_dram, CONFIG_DRAM_SIZE_MB) * MiB)); line over 96 characters
https://review.coreboot.org/c/coreboot/+/43756/2/src/soc/mediatek/common/cbm... File src/soc/mediatek/common/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/2/src/soc/mediatek/common/cbm... PS2, Line 13: return (uintptr_t )(MIN((uintptr_t)_dram + sdram_size(), MAX_DRAM_ADDRESS)); space prohibited before that close parenthesis ')'
https://review.coreboot.org/c/coreboot/+/43756/2/src/soc/nvidia/tegra124/cbm... File src/soc/nvidia/tegra124/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/2/src/soc/nvidia/tegra124/cbm... PS2, Line 11: } adding a line without newline at end of file
https://review.coreboot.org/c/coreboot/+/43756/2/src/soc/rockchip/common/cbm... File src/soc/rockchip/common/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/2/src/soc/rockchip/common/cbm... PS2, Line 10: uintptr_t cbmem_top_chipset(void) please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/43756/2/src/soc/ucb/riscv/cbmem.c File src/soc/ucb/riscv/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/2/src/soc/ucb/riscv/cbmem.c@1... PS2, Line 10: return (uintptr_t)(_dram + (probe_ramsize((uintptr_t)_dram, CONFIG_DRAM_SIZE_MB) * MiB)); line over 96 characters