[coreboot-gerrit] Change in coreboot[master]: Revert "cbfs: Introduce cbfs_ro_map() and cbfs_ro_load()"

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48324 )

Change subject: Revert "cbfs: Introduce cbfs_ro_map() and cbfs_ro_load()"
......................................................................

Patch Set 1:
> Patch Set 1:
> > Patch Set 1: Code-Review-2
> >
> > Sorry, this revert isn't justified. Instead, consider selecting NO_CBFS_MCACHE in Kconfig for the affected platforms. Thank you for your comprehension.
>
> Although selecting NO_CBFS_MCACHE worked, I'm puzzled why exactly this commit breaks ASUS AM1I-A board - and not CB:38424 which enabled the MCACHE. That probably means something isn't right with this commit - just like my earlier CB:41431 which somehow works for fam15h but doesn't for fam16h. So I still think this commit should be reverted, but I will happy to test any workarounds on top of it.

Did you test with USE_OPTION_TABLE enabled or disabled? It's the only thing that could be causing problems.