Attention is currently required from: laforge.
1 comment:
Patchset:
ok, so I think we agree this is not neccessary after all? Given that all loads/stores are atomic, a […]
AFAIU: Pau asked if this assignment itself is atomic (last_dfu_state = dfu_state) and if not would need a CRITICAL section, where we agreed, the assignment itself is atomic.
About this assignment, I want to make sure the mainloop would loop over and only does one thing based based on the dfu_state.
Trying to keep the code path simple and keep away from side effects.
Otherwise a loop run could run through different states and does multiple things
To view, visit change 42170. To unsubscribe, or for help writing mail filters, visit settings.