[M] Change in coreboot[master]: cbmem_top: Change the return value to uintptr_t

[Elyes Haouas (Code Review)]

Attention is currently required from: Raul Rangel, Jeff Daly, Jakub Czapiga, Jonathan Zhang, Matt DeVillier, Julius Werner, Angel Pons, Arthur Heymans, Andrey Petrov, Lance Zhao, Jason Glenesk, Johnny Lin, Tim Wawrzynczak, Christian Walter, Vanessa Eusebio, Fred Reitberger, Felix Held, Tim Chu.

Elyes Haouas has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69163 )

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

Patch Set 9:

(6 comments)

File src/drivers/mrc_cache/mrc_cache.c:

https://review.coreboot.org/c/coreboot/+/69163/comment/d22eb485_e83e73c8 
PS5, Line 663: (void *)cbmem_entry_start(to_be_updated) +
> Misaligned code

Done

File src/include/imd.h:

https://review.coreboot.org/c/coreboot/+/69163/comment/833dac48_e9d01e06 
PS7, Line 112: uintptr_t imd_entry_at(const struct imd *imd, const struct imd_entry *entry);
> It should not be changed as it returns `pointer`. `uintptr_t` is not appropriate here.

Done

File src/lib/imd.c:

https://review.coreboot.org/c/coreboot/+/69163/comment/cc54ff55_1e46ee25 
PS7, Line 14: uintptr_t relative_pointer(void *base, ssize_t offset)
> I think it should remain `void *`. This function has `pointer` in it's name.

Done

https://review.coreboot.org/c/coreboot/+/69163/comment/bff56412_7261dba4 
PS7, Line 267: relative_pointer(imdr_root(imdr), e->start_offset);
> `(uintptr_t)` will be needed here after reverting above.

Done

https://review.coreboot.org/c/coreboot/+/69163/comment/06191aa5_66e397ba 
PS7, Line 568: uintptr_t imd_entry_at(const struct imd *imd, const struct imd_entry *entry)
> As in header file

Done

File src/lib/imd_cbmem.c:

https://review.coreboot.org/c/coreboot/+/69163/comment/90ef91bb_0f602aa7 
PS7, Line 143: uintptr_t cbmem_add(u32 id, u64 size)
> This also should remain `void *`

Done