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

[Elyes Haouas (Code Review)]

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

Elyes Haouas 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:

(4 comments)

Patchset:

PS4: 
Thank you.

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

https://review.coreboot.org/c/coreboot/+/69078/comment/091e3b03_e97ca0a1 
PS1, Line 76: uintptr_t top_of_ram = ALIGN_DOWN(northbridge_get_tseg_base(), 4*MiB);
            : 	return top_of_ram;
> +1, there are other occurrences in other Intel platforms. Could be done in a follow-up.

Done

File src/soc/nvidia/tegra124/cbmem.c:

https://review.coreboot.org/c/coreboot/+/69078/comment/082426b9_d91d2a13 
PS1, Line 9: 	return ((sdram_max_addressable_mb() - FB_SIZE_MB) << 20UL);
> nit: Outer set of parentheses is now unnecessary

Done

File src/soc/sifive/fu540/cbmem.c:

https://review.coreboot.org/c/coreboot/+/69078/comment/d9a9261f_5e5eea76 
PS1, Line 11: return MIN((uintptr_t)_dram + sdram_size_mb() * MiB,
            : 			   FU540_MAXDRAM);
> nit: Can it fit in one line?

Done