Angel Pons 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:
(1 comment)
Patch Set 1:
IMO, this should be made optional. A Kconfig maybe? Not everybody wants to hide errors.
Well, not hiding the MEI device results in long waits because mei_me has to time out. I would appreciate if coreboot would log a message about it, at least.
https://review.coreboot.org/c/coreboot/+/39074/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39074/1//COMMIT_MSG@13 PS1, Line 13: Enable the MEI in device trees of Ibex Peak, Cougar Point and Panther I would prefer that this be done on a separate change.