Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33740 )
Change subject: Documentation: Update soc/intel/apollolake ......................................................................
Patch Set 11:
(4 comments)
https://review.coreboot.org/c/coreboot/+/33740/1/Documentation/soc/intel/apo... File Documentation/soc/intel/apollolake/index.md:
https://review.coreboot.org/c/coreboot/+/33740/1/Documentation/soc/intel/apo... PS1, Line 19: * Each LBP has it's own IFWI and OBB
AIUI, IFWI is the whole thing, LBPs are part of the IFWI, and OBB is part of […]
Done
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.
Done
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.
Done
https://review.coreboot.org/c/coreboot/+/33740/3/Documentation/soc/intel/apo... PS3, Line 58: * Intel TXE : * Power Management Controller (PMC) and P-Unit
Those two are firmware as well
Done