Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/36986 )
Change subject: mysteries_intel: Add a section about SMM_BWP ......................................................................
Patch Set 2: Code-Review+1
(2 comments)
Maybe mention that the "protection" is that a SMI gets triggered when trying to change the write protection status, and the handler just enables it again. I think it worked like that.
https://review.coreboot.org/c/flashrom/+/36986/2/Documentation/mysteries_int... File Documentation/mysteries_intel.txt:
https://review.coreboot.org/c/flashrom/+/36986/2/Documentation/mysteries_int... PS2, Line 9: Sometimes a hardware vendor will enable "SMM BIOS Write Protect" (SMM_BWP). Maybe mention this is done in firmware? I read this paragraph, and asked myself "How, though?"
https://review.coreboot.org/c/flashrom/+/36986/2/Documentation/mysteries_int... PS2, Line 17: Flashrom "flashrom" should be in lowercase, like on line 32