Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33740 )
Change subject: Documentation: Update soc/intel/apollolake ......................................................................
Patch Set 12:
(2 comments)
Sorry for being unresponsive must have missed a message...
https://review.coreboot.org/c/coreboot/+/33740/7/Documentation/soc/intel/apo... File Documentation/soc/intel/apollolake/ifwi.dia:
PS7:
Well, it's explained here (p. 11) https://firmware.intel. […]
Looks like an implementation detail that is not used in practice (afaics). Again, have you ever seen such a layout? I think it's confusing to the reader, and you call it "common" in the text.
https://review.coreboot.org/c/coreboot/+/33740/12/Documentation/soc/intel/ap... File Documentation/soc/intel/apollolake/index.md:
https://review.coreboot.org/c/coreboot/+/33740/12/Documentation/soc/intel/ap... PS12, Line 27: common Can't say that I have seen many IFWIs, but I wouldn't call it common if it's unusual to have a BPDT entry above all S-BPDT entries.