[coreboot-gerrit] [M] Change in coreboot[master]: cbmem_top_chipset: Change the return value to uintptr_t

[Jakub Czapiga (Code Review)]

Attention is currently required from: Hung-Te Lin, Philipp Hug, Matt DeVillier, Angel Pons, Julius Werner, Andrey Petrov, ron minnich, Elyes Haouas, Felix Held.

Jakub Czapiga has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69078 )

Change subject: cbmem_top_chipset: Change the return value to uintptr_t
......................................................................

Patch Set 4:

(2 comments)

File src/lib/imd_cbmem.c:

https://review.coreboot.org/c/coreboot/+/69078/comment/6d9411c0_6168ae4b 
PS4, Line 21: 		uintptr_t top;
It should be static

File src/northbridge/intel/i440bx/memmap.c:

https://review.coreboot.org/c/coreboot/+/69078/comment/05a1a3fe_da22ef3d 
PS1, Line 48: int tseg_size = 128 * KiB * (1 << (tseg >> 1));
            : 		tom -= tseg_size;
> If anything, this belongs to a follow-up.

I assume it works with ints here, so there should be no problems.
I agree a potential fix should be in follow-up patch.