Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33740 )
Change subject: Documentation: Update soc/intel/apollolake ......................................................................
Patch Set 10:
(4 comments)
https://review.coreboot.org/c/coreboot/+/33740/7/Documentation/soc/intel/apo... File Documentation/soc/intel/apollolake/flash_layout2.svg:
PS7:
Usage of IFWI is misleading here. IFWI includes both LBPs.
Done
https://review.coreboot.org/c/coreboot/+/33740/7/Documentation/soc/intel/apo... File Documentation/soc/intel/apollolake/index.md:
https://review.coreboot.org/c/coreboot/+/33740/7/Documentation/soc/intel/apo... PS7, Line 44: (bootblock)
They are all called bootblock somehow. Are you referring to the […]
Done
https://review.coreboot.org/c/coreboot/+/33740/7/Documentation/soc/intel/apo... PS7, Line 47: from IFWI
Looking at an IFWI, it would be rather "usually spans". It seems there […]
Done
https://review.coreboot.org/c/coreboot/+/33740/7/Documentation/soc/intel/apo... PS7, Line 52: Instead we call the partition with the firmware components as "IFWI", : which has to be at the beginning of the BIOS region to be compliant with the : IFWI specification.
"we" being who exactly? There is no consent in the coreboot community that […]
Done