Attention is currently required from: Felix Singer, Nico Huber, Paul Menzel, Angel Pons, Michael Niewöhner.
6 comments:
Commit Message:
Patch Set #1, Line 7: board_enable.c: Add ME unlock function for TUXEDO laptops
Still working on that, because after rebase something broke... […]
Okay, the bug was in the acpi_ec patch and now it works correctly.
Commit Message:
Patch Set #2, Line 9: baord
board
Done
File board_enable.c:
Patch Set #2, Line 2310: tuxedo
btw. the initial discussion was about the function name and that is still valid. […]
Done
> > Read again what I wrote. […]
Not naming it at all is good as well for now. Although it probably means Win dows Flash or something like that...
Patch Set #2, Line 2335: power off the system and power it on
Tested with AC plugged. The simple shutdown through APCI is enough, EC FW takes care of the rest.
After a second thought I think we shall leave the decision to user to do the power off. Added an msg_pinfo with a message what to do to take effect. Also this board enable is going to be used with fwupd, it would be bad if suddenly the laptop will power off. fwupd could probably handle it more gracefully.
Patch Set #2, Line 2339: return 0;
The reason to set the strap is when someone wants to write IFD and ME region. […]
If we unlock the ME successfully we return -1 so that the flashrom will not proceed with flashing and user could power cycle the device first. Of course the user must explicitly pass the programmer option for flashrom to take this path, so I think it might be good as it is now.
To view, visit change 56023. To unsubscribe, or for help writing mail filters, visit settings.