Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement check of firmware version and flash format procedure #76

Open
oscgonfer opened this issue Nov 20, 2024 · 2 comments
Open

Implement check of firmware version and flash format procedure #76

oscgonfer opened this issue Nov 20, 2024 · 2 comments
Assignees
Labels

Comments

@oscgonfer
Copy link
Contributor

Newer versions of the firmware are weirdly interpreting previous flash storage data with the wrong IDs. This is creating new components in the platform. Currently, the rootcause is not understood and not logic, but upon testing on various devices it's a consistent issue.

As a precaution, we should implement a firmware check based on a flag on EEPROM for the current flash information firmware. This would mean that the kit would check its new configuration against it and it would potentially flash the memory in case of inconsistency.

@oscgonfer oscgonfer self-assigned this Nov 20, 2024
@oscgonfer
Copy link
Contributor Author

oscgonfer commented Nov 20, 2024

UPDATE: the rootcause is now clear.

The way the flash memory management stores the sensor IDs is via a static cast of the SensorType enum here:

This was never considered to change, but it did in two commits for SCK2.1:

01b42f2
4f4713a

With these changes, the ENUM casting also changes, and previous mappings done in the flash memory no longer match to those originally stored. Reverting this commit would provoke all devices currently deployed with newer firmware versions to suffer from the same problem once there is a firmware upgrade.

Therefore, the only "clean" option is to perform a flash format in case there is no agreement in terms of firmware versions with the current version and the one that the flash memory has stored its data with.

@oscgonfer
Copy link
Contributor Author

Check #50 if working on flash issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant