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

Improve emulated EEPROM reload handling #179

Closed
wants to merge 1 commit into from

Conversation

nakarlsson
Copy link
Contributor

Add variable to control if ESC support read size 4 or 8 Byte. Depends on ESC.

Add variable and function to set user function
to perform emulated EEPROM reload.

Add EEPROM error codes that can be used to indicatë crc error on reload.

Fixes #178

Copy link
Contributor

@ArthurKetels ArthurKetels left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional approved, but needs some documentation why 4byte reloads do not have full functionality without a custom implementation. Also what should a minimal custom reload function look like.

Add variable to control if ESC support read size 4
or 8 Byte. Depends on ESC.

Add variable and function to set user function
to perform emulated EEPROM reload.

Add EEPROM error codes that can be used to indicatë
crc error on reload.

Include custom reload samples
@nakarlsson nakarlsson force-pushed the feature/improve_eeprom_reload_handling branch from 0c7becc to 4f0704d Compare April 8, 2024 12:56
@nakarlsson
Copy link
Contributor Author

When writing samples I got a idea of an alternative implementation.

Second option is to pass eep_stat_t stat; as pointer to the custom reload. The custom reload will via that know the address. It can update the crc error directly...
Eg. it can be a function, void (*reload_ptr) (eep_stat_t * stat).

What do you think?

@nakarlsson nakarlsson closed this Apr 9, 2024
@nakarlsson
Copy link
Contributor Author

Go for alternative 2 #180

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

Successfully merging this pull request may close these issues.

Improve emulated EEPROM reload handling
2 participants