Attention is currently required from: lynxis lazus, pespin.
3 comments:
Patchset:
honestly I can't quite see the improvement because now we have dfu_flash_done and usb_dfu_status on top of the already existing dfu_state and dfu_status ... a rewrite should make it less confusing instead of even more contrived.
The patch mostly changes "things" but I'm not aware of any actual issues that are being solved here, we only have one repeatable apparently hardware related flash write issue as far as I am aware, and I'd rather not start touching a bootloader that (at least until now) did a fairly good job.
File usb/class/dfu/device/dfudf.c:
Patch Set #4, Line 175: dfu_state = USB_DFU_STATE_DFU_DNBUSY;
at least move that assignment to case USB_DFU_STATE_DFU_DNLOAD_SYNC and don't just drop old comments..
Patch Set #4, Line 272: break;
why do we suddenly need superfluous break statements within else if chains?
To view, visit change 42174. To unsubscribe, or for help writing mail filters, visit settings.