David Hendricks has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/42485 )
Change subject: mysteries_intel: Add a section for software vs hardware sequencing ......................................................................
Patch Set 3:
(4 comments)
https://review.coreboot.org/c/flashrom/+/42485/2/Documentation/mysteries_int... File Documentation/mysteries_intel.txt:
https://review.coreboot.org/c/flashrom/+/42485/2/Documentation/mysteries_int... PS2, Line 20: be therefore be
duplicated `be`
Done
https://review.coreboot.org/c/flashrom/+/42485/2/Documentation/mysteries_int... PS2, Line 26: SPI Programming Guide
Reference [SPI_PROG] as an example?
I'm not sure what you mean by [SPI_PROG]... I also may have used confusing terminology. Here I am specifically referring to application notes from Intel that they title "SPI Programming Guide" for each PCH. These are usually referenced from the PCH datasheet, for example in https://www.intel.com/content/www/us/en/products/docs/chipsets/300-series-ch....
I'll try to clarify that.
https://review.coreboot.org/c/flashrom/+/42485/2/Documentation/mysteries_int... PS2, Line 31: and know
nor
Done
https://review.coreboot.org/c/flashrom/+/42485/2/Documentation/mysteries_int... PS2, Line 35: will not necessarily even read : the chip's ID
note that sometimes we can't do that at all, because there are multiple chips or because some import […]
I prefer to avoid mentioning software sequencing in this paragraph since it might be confusing, however I used your example of multiple chips to clarify that we might not even be able to read the chip ID.