Evgeny Zinoviev has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39074 )
Change subject: sb/intel/{bd82x6x,ibexpeak}: hide MEI if ME inoperable ......................................................................
Patch Set 1:
(2 comments)
I hope this change will not be abandoned. My patch for ME Soft Temp Disable depend on it, and I don't want to create duplicating patches.
https://review.coreboot.org/c/coreboot/+/39074/1/src/southbridge/intel/bd82x... File src/southbridge/intel/bd82x6x/me.c:
https://review.coreboot.org/c/coreboot/+/39074/1/src/southbridge/intel/bd82x... PS1, Line 695: ME_ERROR_BIOS_PATH BIOS shouldn't hide ME device by default on error path (me_cleaner is not the only possible reason of error). Please make it at least a Kconfig option, those who want to hide it will use it.
https://review.coreboot.org/c/coreboot/+/39074/1/src/southbridge/intel/bd82x... File src/southbridge/intel/bd82x6x/me_8.x.c:
https://review.coreboot.org/c/coreboot/+/39074/1/src/southbridge/intel/bd82x... PS1, Line 684: ME_ERROR_BIOS_PATH Same as in me.c.