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

Add deprecation warning to flashmem_get_address() #1927

Merged
merged 2 commits into from
Nov 3, 2019

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Oct 31, 2019

Added in #1490 but not currently used within the framework.

This function maps a PROGMEM address to the corresponding flash location, but doesn't work if program space has been remapped by rBoot.

I will fix this properly in a separate upgrade to Host flash memory handling and FlashStrings.

@mikee47
Copy link
Contributor Author

mikee47 commented Nov 3, 2019

@slaff Any thoughts on this patch?

@mikee47
Copy link
Contributor Author

mikee47 commented Nov 3, 2019

Since we don't use this in the framework (at present) but will shortly, I'll revert this patch but add a deprecation warning just in case anyone's used it.

@mikee47 mikee47 force-pushed the fix/flashmem_get_address branch from 4c6697f to b4f975f Compare November 3, 2019 09:39
@mikee47 mikee47 changed the title Fix flashmem_get_address() to account for rBoot mapping Add deprecation warning to flashmem_get_address() Nov 3, 2019
@mikee47 mikee47 force-pushed the fix/flashmem_get_address branch from b4f975f to cbdef55 Compare November 3, 2019 09:45
@slaff slaff added this to the 4.0.0 milestone Nov 3, 2019
@slaff slaff merged commit ff4e615 into SmingHub:develop Nov 3, 2019
@mikee47 mikee47 deleted the fix/flashmem_get_address branch November 3, 2019 17:00
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.

2 participants