Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52879 )
Change subject: arch/x86/walkcbfs.S: Use FMAP instead of "cbfs master header"
......................................................................
Patch Set 6:
(1 comment)
Patchset:
PS6:
FYI, this change can break platforms with FMD file containing a statement like "FLASH 8M", because on x86 flash is memory mapped under 4GB. It results in fmap_config.h with "#define FMAP_SECTION_FLASH_START 0" and all offsets to regions relative to zero. There is difference bewteen COREBOOT let's say at 0x400000 and at 0xffc00000. So the correct statement in FMD would be "FLASH@<4GB - flash_size> <flash_size>" like "FLASH@0xff800000 8M". If one does not use any FMD file, the autogenerated fmap_config will contain absolute addresses and it will work with this patch. Just found it out with my Dell Optiplex 9010 and a FMD file with "FLASH 12M" statement.
Thanks for finding this: fixed in CB:55452
--
To view, visit
https://review.coreboot.org/c/coreboot/+/52879
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ibab657cc40cd5c09c3a73c54950b98ac45a98dbf
Gerrit-Change-Number: 52879
Gerrit-PatchSet: 6
Gerrit-Owner: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: 9elements QA
hardwaretestrobot@gmail.com
Gerrit-CC: Michał Żygowski
michal.zygowski@3mdeb.com
Gerrit-CC: Patrick Rudolph
siro@das-labor.org
Gerrit-Comment-Date: Sat, 12 Jun 2021 20:39:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Michał Żygowski
michal.zygowski@3mdeb.com
Gerrit-MessageType: comment