else shouldn't we go back to waiting for an update or something? Seems wrong to just get stuck here.

Is Noe never going to change these IDs? I think he has already changed DD fpga ID for beta 2.0. If these IDs are not fixed forever, we cannot hardcode them here in the bootloader code.

We can't jump to app if fpga ID is bad? Why not?

Where is this function defined?

Align comments

Why 2 conditions? Are there 2 ways to initiate an update?

And why isn't this check done in the CHECK_FOR_UPDATE_STATE?

TODO

TODO

TODO should be all caps.

TODO should be all caps for consistency (so we know what to search for).

I think it should be restored even if it's not used anywhere yet.

Do we need an end of group marker down here?

Can you capture intent to resolve my comment in a TODO so we don't lose track of it? (General recommendation for all comments that we are postponing resolution for).

Can we put test code inside of #ifdef TEST_CODE_ENABLED or something so that it is easy to remove later?

It will be implemented in the new branch.

Done

Done

Done

Done

Removed the variable.

Removed the TODO.

Removed the interrupt.

Done

The new update request is checked on top of the mode update exec function. If there is a new update, it will request a transition otherwise, this is a terminal state.

It is the update mode re-initialized. The initialization is done using other functions that have their own doxygen comments. Do I need to add more here?

Added the return of a value.

Done

Done