4 comments:
File src/security/vboot/ec_sync.c:
Patch Set #1, Line 69: vboot_reboot();
This seems a bit odd for error handling? If you do want to reboot, you should at least vb2api_fail() […]
Agreed, that really shouldn't happen, I'll change to just print on an error and halt() regardless.
Patch Set #1, Line 71: poweroff();
poweroff() is not available on all coreboot platforms, so putting this here would prevent us from us […]
Ack
Patch Set #1, Line 86: poweroff();
This has the same problem with poweroff(). […]
Ah yes, that's from battery cutoff, not needed in just this flow. Removed.
Patch Set #1, Line 95: if (retval != VB2_SUCCESS) {
nit: maybe just put a […]
Done
To view, visit change 37562. To unsubscribe, or for help writing mail filters, visit settings.