Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31766 )
Change subject: Documentation: Explain FMAP and FMD
......................................................................
Patch Set 5:
Patch Set 5:
- What if someday we want multiple sections being write-protected?
The hardware write-protect mechanism doesn't support that anyway.
The WP mechanism on SPI flash does not support that (today).
For example, on x86 we can set multiple regions as read-only to AP in SI_DESC, just like ME.
Anyway, I've changed the description of all 3 fields to "Not really used today" (hesitate to call that "deprecated" because it's simply we (cros, or coreboot) don't use it. other flashmap users may find it useful). Hope that would work.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/31766
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia389e56c632096d7c905ed221fd4f140dec382e6
Gerrit-Change-Number: 31766
Gerrit-PatchSet: 5
Gerrit-Owner: Hung-Te Lin
hungte@chromium.org
Gerrit-Reviewer: David Hendricks
david.hendricks@gmail.com
Gerrit-Reviewer: Hung-Te Lin
hungte@chromium.org
Gerrit-Reviewer: Philipp Deppenwiese
zaolin.daisuki@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Julius Werner
jwerner@chromium.org
Gerrit-CC: Patrick Georgi
pgeorgi@google.com
Gerrit-Comment-Date: Wed, 06 Mar 2019 04:11:28 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment