Patrick Georgi merged this change.

View Change

Approvals: build bot (Jenkins): Verified Felix Held: Looks good to me, approved
Documentation: Warn about ME cleaner on Sandy Bridge

Document known issues with 'disabled' ME.

Change-Id: I364f3ed49341523c781eb2f3b41e866f33632a7e
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32889
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
M Documentation/northbridge/intel/sandybridge/index.md
A Documentation/northbridge/intel/sandybridge/me_cleaner.md
2 files changed, 21 insertions(+), 0 deletions(-)

diff --git a/Documentation/northbridge/intel/sandybridge/index.md b/Documentation/northbridge/intel/sandybridge/index.md
index dcb090a..c1d4b99 100644
--- a/Documentation/northbridge/intel/sandybridge/index.md
+++ b/Documentation/northbridge/intel/sandybridge/index.md
@@ -6,3 +6,4 @@

- [Native Ram Initialization](nri.md)
- [RAM initialization feature matrix](nri_features.md)
+- [ME Cleaner](me_cleaner.md)
diff --git a/Documentation/northbridge/intel/sandybridge/me_cleaner.md b/Documentation/northbridge/intel/sandybridge/me_cleaner.md
new file mode 100644
index 0000000..1086e7e
--- /dev/null
+++ b/Documentation/northbridge/intel/sandybridge/me_cleaner.md
@@ -0,0 +1,20 @@
+# ME Cleaner
+It's possible to 'clean' the ME partition within the flash medium as part
+of the build process. While cleaning as much code as possible is removed
+from the ME firmware partition. In this state the ME errors out and doesn't
+operate any more.
+
+**Using a 'cleaned' ME partition may lead to issues and its use should be
+carefully evaulated.**
+
+## Observations with 'cleaned' ME
+
+* Instable LPC bus
+ * SuperIO is malfunctioning
+ * TPM is malfunctioning
+ * Random system shutdowns on high bus activity
+
+## Filing bug reports
+
+Always test with unmodified IFD and ME section before reporting bugs to the
+coreboot project.

To view, visit change 32889. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I364f3ed49341523c781eb2f3b41e866f33632a7e
Gerrit-Change-Number: 32889
Gerrit-PatchSet: 5
Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Nicola Corna <nicola@corna.info>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: merged