tinyfpga / TinyFPGA-Bootloader

An open source USB bootloader for FPGAs
Apache License 2.0
357 stars 94 forks source link

Add retrying of sector writes if they fail #55

Open mithro opened 5 years ago

mithro commented 5 years ago

As @ewenmcneill suggested.

ewenmcneill commented 5 years ago

These two commits from an old PR give the general code structure, but probably need reimplementing to fit with the much rearranged code in master now:

Ewen

ewenmcneill commented 5 years ago

FTR, another user having problems, supposedly with the "latest" tinyprog (installed from git). I've asked whether they installed it before or after the changes merged this past weekend (2019-07-12). But either way, retrying the failed sectors might have helped them.

https://discourse.tinyfpga.com/t/issues-with-tinyprog-u-on-bx/628/18

(and this is another reason for wanting to actually do a release to PyPI...)

Ewen

ewenmcneill commented 5 years ago

FTR, another user having problems, supposedly with the "latest" tinyprog (installed from git). I've asked whether they installed it before or after the changes merged this past weekend (2019-07-12).

Apparently they installed just before the fix was merged, and it's been working better for them after updating to the current installed version -- https://discourse.tinyfpga.com/t/issues-with-tinyprog-u-on-bx/628/20.

They also suggest that the PyPI version be updated... which I also agree would be a good idea.

Ewen