Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33740 )
Change subject: Documentation: Update soc/intel/apollolake ......................................................................
Patch Set 3:
(4 comments)
Please resolve all comments.
https://review.coreboot.org/c/coreboot/+/33740/3/Documentation/soc/intel/apo... File Documentation/soc/intel/apollolake/index.md:
https://review.coreboot.org/c/coreboot/+/33740/3/Documentation/soc/intel/apo... PS3, Line 23: For legacy and compatibility reasons, the BIOS partition wasn't renamed to "IFWI". rephrase. It's hard to understand.
https://review.coreboot.org/c/coreboot/+/33740/3/Documentation/soc/intel/apo... PS3, Line 25: which has to be at the beginning of the BIOS region to be compliant with the seems to refer to FMAP, but doesn't say so.
https://review.coreboot.org/c/coreboot/+/33740/3/Documentation/soc/intel/apo... PS3, Line 36: To disable dual partitioning mode, set bit3 in IFD at offset `0x1ff` to 1. 0x11f
https://review.coreboot.org/c/coreboot/+/33740/3/Documentation/soc/intel/apo... PS3, Line 57: **Including components:** Included