Attention is currently required from: Nico Huber, Angel Pons. Evgeny Zinoviev has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37115 )
Change subject: sb/intel/bd82x6x: Support ME Soft Temporary Disable Mode ......................................................................
Patch Set 28:
(3 comments)
File src/southbridge/intel/bd82x6x/me.h:
https://review.coreboot.org/c/coreboot/+/37115/comment/e8dadd00_fba716b0 PS25, Line 188: u32
Actually, this may be a problem... […]
Fixed, tested, works.
File src/southbridge/intel/bd82x6x/me.c:
https://review.coreboot.org/c/coreboot/+/37115/comment/4a902811_4573a715 PS25, Line 322: /* : * ME starts loading firmware immediately after writing to H_GS, : * but Lenovo BIOS performs a reboot after bringing ME back to : * Normal mode. Assume that global reset is needed. : */
Update: my usbdebug ft2232h+pl2303 combo doesn't work for some reason (tried on two laptops, worked […]
Here's what I found.
Tested it multiple times on a corebooted MacBook Pro 10,1, ME FW version 8.0.1441.4. It doesn't work as we would expect after reading ME10 docs: ME returns some weird value of 0x12cc678c in MEI header. No idea what it means but if you google it you'll find this https://github.com/corna/me_cleaner/issues/53#issuecomment-325455687
My theory is that it's just the first message in a multi message response (the completed bit is 0 here), and coreboot doesn't support them, it just returns with an error if it sees a non-complete bit in MEI header. I'll try supporting multi packet responses but it's out of scope of this patch.
File src/southbridge/intel/ibexpeak/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/37115/comment/b787ac96_09214dac PS25, Line 33: romstage-y +=../bd82x6x/me_common.c
aaaaaargh! can we avoid this?
Done