[coreboot-gerrit] Change in coreboot[master]: Documentation: Update soc/intel/apollolake

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33740 )

Change subject: Documentation: Update soc/intel/apollolake
......................................................................

Patch Set 7:

(2 comments)

https://review.coreboot.org/c/coreboot/+/33740/7/Documentation/soc/intel/apo... 
File Documentation/soc/intel/apollolake/ifwi.dia:

PS7: 
I haven't ever seen a BPDT that points behind the S-BPDT sub partitions.
Have you?

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 47: from IFWI
> OBB is part of an LBP, which is part of the IFWI. It doesn't start after […]

Looking at an IFWI, it would be rather "usually spans". It seems there
can be empty space after it.