[coreboot-gerrit] [S] Change in coreboot[main]: treewide: Assume FMAP_SECTION_FLASH_START = 0

[Maximilian Brune (Code Review)]

Attention is currently required from: Ana Carolina Cabral, Erik van den Bogaert, Felix Held, Frans Hendriks, Fred Reitberger, Intel coreboot Reviewers, Jakub "Kuba" Czapiga, Jason Glenesk, Julius Werner, Jérémy Compostella, Martin L Roth, Matt DeVillier, Nick Kochlowski, Shuo Liu.

Maximilian Brune has posted comments on this change by Maximilian Brune. ( https://review.coreboot.org/c/coreboot/+/86771?usp=email )

Change subject: treewide: Assume FMAP_SECTION_FLASH_START = 0
......................................................................

Patch Set 3:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/86771/comment/d6c802c5_686f1946?usp... :
PS2, Line 8:
> Before the changes, if we would like to get the host address of flash start we can check the fmap_co […]

SOC and mainboard code should always know the flash mapping either way.

common code can access the flash through `boot_device_ro()` or the relevant FMAP and CBFS functions (if you want to access FMAP regions or CBFS files). But In general common code should not need to know the actual MMIO address, besides being able to read and write the flash.

cbfstool also should have knowledge about the mmap using the "--mmap" option (although I am not sure if all platforms actually use this one).