Repository navigation
Add sections on Intel ME dependency - #304
Conversation
|
|
||
| ls /dev/mei*/ | ||
|
|
||
| If no devices are reported, then the user needs to enable the ME. |
There was a problem hiding this comment.
As noted by @fredoh9 , maybe mention here that /dev/mei*/ requires some Kconfig in the first place.
Also mention that firmware authentication itself does NOT require any /dev/mei*/, which could be confusing. Calling `/dev/mei*/ a "debug interface" should avoid this confusion even though it's not just a debug interface.
|
please don't merge for now, the part on /dev/mei is likely to require a significant update. |
Git is not happy without it Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
4724c9a to
7948719
Compare
|
updated (no way to detect ME enablement) and added section on HDaudio amps. |
| capabilities, codec names, and DMI options. While the SOF team and the | ||
| community try to cover all possible cases, errors will happen when the | ||
| wrong file is selected at any of the three layers. No newline at end of file | ||
| wrong file is selected at any of the three layers. |
There was a problem hiding this comment.
Strange, I've seen github display missing new lines much better than this before.
I've been puzzled by this misleading "zero diff" until I looked at the commit message so leaving a comment for others.
|
|
||
| The Intel ME (Management Engine) is responsible for authentication of | ||
| the firmware, be it signed by an Intel production key, a community key | ||
| or an OEM key. If the Intel ME is disabled by an OEM, or disabled by |
There was a problem hiding this comment.
..., be it signed by an Intel or OEM production key (consumer products) or by a community key (development systems and make boards)
There was a problem hiding this comment.
Chromebooks also use community key.
| If the ME is disabled by the OEM or the user, firmware authentication | ||
| will fail without an explicit log provided to the user. In case of any | ||
| authentication failure, the user should verify that the ME is not | ||
| disabled. |
There was a problem hiding this comment.
..., more information about the ME in the "firmware binary" section above.
| *************************** | ||
|
|
||
| If the ME is disabled by the OEM or the user, firmware authentication | ||
| will fail without an explicit log provided to the user. In case of any |
There was a problem hiding this comment.
without any explicit feedback provided to the user
(more generic)
lgirdwood
left a comment
lgirdwood
left a comment
There was a problem hiding this comment.
@deb-intel are you able to fix up the grammar after merge as you usually do ? @marc-hb has pointed out a few grammar changes., but there maybe others. Need to merge for v1.7
|
@lgirdwood, yes, I can make merged grammatical updates. |
This was missed in earlier patches and a key dependency Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Inspired by https://bugzilla.kernel.org/show_bug.cgi?id=210633 Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
7948719 to
381bf45
Compare
|
pushed changes suggested by @marc-hb |
deb-intel
left a comment
deb-intel
left a comment
There was a problem hiding this comment.
Will make grammatical updates in a separate PR.
No description provided.