[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 7:

(1 comment)

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 53: (CONFIG(CACHE_MRC_SETTINGS)
> This check was located earlier. […]

So I tried to keep the assignment and the check in the same line as before, but coreboot has some pre-commit checks that prevent that type of code from going in (for good reason, since it's hard to read).  The logic should be the same.  We just split it into two lines now (one for the assignment and one for the condition).  We need the assignment though, unlike the previous mrc_cache_get_current where we could just check the return value.