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 8:
(4 comments)
https://review.coreboot.org/c/coreboot/+/43756/8/src/mainboard/emulation/qem... File src/mainboard/emulation/qemu-aarch64/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/8/src/mainboard/emulation/qem... PS8, 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/8/src/mainboard/emulation/qem... File src/mainboard/emulation/qemu-armv7/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/8/src/mainboard/emulation/qem... PS8, 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/8/src/soc/nvidia/tegra124/cbm... File src/soc/nvidia/tegra124/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/8/src/soc/nvidia/tegra124/cbm... PS8, Line 11: } adding a line without newline at end of file
https://review.coreboot.org/c/coreboot/+/43756/8/src/soc/rockchip/common/cbm... File src/soc/rockchip/common/cbmem.c:
https://review.coreboot.org/c/coreboot/+/43756/8/src/soc/rockchip/common/cbm... PS8, Line 10: uintptr_t cbmem_top_chipset(void) please, no spaces at the start of a line