[coreboot-gerrit] Change in coreboot[master]: mrc_cache: mrc_cache fetch functions to support non-x86 platforms

[Shelley Chen (Code Review)]

Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44006 )

Change subject: mrc_cache: mrc_cache fetch functions to support non-x86 platforms
......................................................................

Patch Set 11:

(2 comments)

https://review.coreboot.org/c/coreboot/+/44006/7/src/drivers/intel/fsp1_1/ro... 
File src/drivers/intel/fsp1_1/romstage.c:

https://review.coreboot.org/c/coreboot/+/44006/7/src/drivers/intel/fsp1_1/ro... 
PS7, Line 292: __weak void *mrc_cache_current_mmap_leak(int type, uint32_t version,
> If you remove it and Jenkins doesn't complain, then we aren't using it.

Done

Okay, let's see if Jenkins complains.

https://review.coreboot.org/c/coreboot/+/44006/8/src/include/mrc_cache.h 
File src/include/mrc_cache.h:

https://review.coreboot.org/c/coreboot/+/44006/8/src/include/mrc_cache.h@24 
PS8, Line 24:  The functions
            :  * return < 0 on error, 0 on success.
> Instead of dropping this, can we move it to the comment on line 30 since it still applies to mrc_cac […]

Done