Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32889 )
Change subject: Documentation: Warn about ME cleaner on Sandy Bridge
......................................................................
Patch Set 1:
Patch Set 1:
maybe add a sentence what me_cleaner does? from the text it isn't obvious what "cleaning" the me image does, so i'd add that this will remove as much code as it can from the me image
I'd also add that the ME errors when said code is removed, which is an abnormal state.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/32889
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I364f3ed49341523c781eb2f3b41e866f33632a7e
Gerrit-Change-Number: 32889
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: Nicola Corna
nicola@corna.info
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Felix Held
felix-coreboot@felixheld.de
Gerrit-CC: Patrick Rudolph
siro@das-labor.org
Gerrit-Comment-Date: Mon, 20 May 2019 10:26:01 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment