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

[Furquan Shaikh (Code Review)]

Furquan Shaikh 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 9: Code-Review+2

(2 comments)

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_cache_load_current() and line 43 as it applies to mrc_cache_stash_data() as well.

https://review.coreboot.org/c/coreboot/+/44006/8/src/include/mrc_cache.h@43 
PS8, Line 43: data_size
> I thought that we had agreed to return the void * in the past?  Furquan, do you not agree with this  […]

I think this is fine. Makes sense to keep it consistent with the underlying calls.