40 UI nitropad does not reboot as state in the UI message after oem factory resetfirmware upgrade

Nitrokey / heads

A minimal Linux that runs as a coreboot or LinuxBoot ROM payload to provide a secure, flexible boot environment for laptops and servers.

http://osresearch.net/

GNU General Public License v2.0

15 stars 1 forks source link

40 UI nitropad does not reboot as state in the UI message after oem factory resetfirmware upgrade #43

Open alexgithublab opened 7 months ago

alexgithublab commented 7 months ago

This is for https://github.com/Nitrokey/heads/issues/40

tlaurion commented 7 months ago

@nestire how is this normal?

tlaurion commented 7 months ago

There are two scripts. Shutdown and reboot. Seems like what is passed to EC in reboot is buggy.

Since the goal is to fix things and not add more workaround, is there bug reports upstream so that ec/coreboot port is fixed?

alexgithublab commented 7 months ago

If we remove the "if" from this commit the rebooting script is working.

not sure why @daringer did this

daringer commented 7 months ago

this essentially was a safety measure as I can not reliably say, which portions of the flash have to be overwritten to fail during the next power cycle so the safety measure was to always shutdown via the EC. During testing I often flash from within HEADS using flashrom directly and overwriting the entire flash, then EC shutdown is needed ... using the UI this seems not necessary as only the bios region is written.