[PATCH v2 12/15] Firmware upgrade for OSTC3
Anton Lundin
glance at acc.umu.se
Mon Dec 22 14:07:57 PST 2014
On 21 December, 2014 - Jef Driesen wrote:
> On 20-12-14 12:13, Anton Lundin wrote:
> >On 20 December, 2014 - Anton Lundin wrote:
> >>I still like my way of parsing the firmware files better than yours, but
> >>if you want it that way, fine.
>
> It's a bit less efficient then your version (e.g. a few more fread calls),
> but I prefer to use the same pattern throughout the entire codebase. We
> already had this code for the ostc, so it doesn't make much sense to
> introduce another variant for the ostc3.
>
> >I did do a round of cleanup, and to test how things looked with the
> >hw_ostc3_device_init-code and they are all how cleaned and fixed on
> >github.
> >
> >If you would like to see them, i can send them out, or you can look at:
> >https://github.com/glance-/libdivecomputer/tree/ostc3_fw
>
> I did a few minor style cleanups here and there, and pushed everything to master.
>
> Thanks for your work!
>
Thank you for doing a proper review and keeping my code honest.
//Anton
--
Anton Lundin +46702-161604
More information about the devel
mailing list