Check EEPROM version and CRC on load

[amitchone]

No description provided.

[ethan-duckett-brill]

I think this is a good shout, but my only worry is that someone without shell access could for example change a parameter in BrAn, the NVM saves and then they lose all of their previous defaults. Probably unlikely since we hopefully won't have to update EEPROM versions too often in the field, but I thought it was worth raising. Reckon this is enough of an issue to worry about?

[amitchone]

I think this is a good shout, but my only worry is that someone without shell access could for example change a parameter in BrAn, the NVM saves and then they lose all of their previous defaults. Probably unlikely since we hopefully won't have to update EEPROM versions too often in the field, but I thought it was worth raising. Reckon this is enough of an issue to worry about?

I suppose this was the driver behind the "backups" feature we had in BrAn for B62E...

That aside, though, that's application-level behaviour and this is separate library code. IMO that is something we manage externally to this and shouldn't be a blocker if you're happy with this implementation and behaviour; BPUX-behaviour-notwithstanding

[ethan-duckett-brill]

Yeah fair enough, we can deal with that separately. I'm happy for this to go in!