HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43756 )
Change subject: src: cast *cbmem_top_chipset() function to uintptr_t ......................................................................
Patch Set 18:
(3 comments)
https://review.coreboot.org/c/coreboot/+/43756/18/src/include/cbmem.h File src/include/cbmem.h:
https://review.coreboot.org/c/coreboot/+/43756/18/src/include/cbmem.h@60 PS18, Line 60: uintptr_t cbmem_top(void) maybe in an other changes?
https://review.coreboot.org/c/coreboot/+/43756/18/src/northbridge/intel/x4x/... File src/northbridge/intel/x4x/memmap.c:
https://review.coreboot.org/c/coreboot/+/43756/18/src/northbridge/intel/x4x/... PS18, Line 5: #include <stdint.h> duplicated
https://review.coreboot.org/c/coreboot/+/43756/18/src/northbridge/intel/x4x/... PS18, Line 8: <stdint.h> here is the 2nd