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

No-WiFi flash fixes #2353

Merged
merged 2 commits into from Jul 20, 2021
Merged

No-WiFi flash fixes #2353

merged 2 commits into from Jul 20, 2021

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Jul 19, 2021

Fixes the following issues:

  • Call to spi_flash_get_id() hangs in application code
  • ROMs must be located below 1M

Both due to SDK implementation of Cache_Read_Enable_New() which requires a variable to be set to 0 at startup.
This variable is always located at offset -8 from the function address so it can be reliably patched during init code.

Also, system_os_post must be in IRAM.

Fixes the following issues:

- Call to `spi_flash_get_id()` hangs in application code
- ROMs must be located below 1M

Both due to SDK implementation of `Cache_Read_Enable_New()` which requires a variable to be set to 0 at startup.

Also, `system_os_post` must be in IRAM.
@slaff slaff added this to the 4.4.0 milestone Jul 20, 2021
@slaff slaff merged commit 0a448d1 into SmingHub:develop Jul 20, 2021
@slaff slaff mentioned this pull request Aug 12, 2021
5 tasks
@mikee47 mikee47 deleted the fix/no-wifi-flash branch September 11, 2021 08:04
slaff pushed a commit that referenced this pull request Sep 27, 2021
Fixes the following issues:

- Call to `spi_flash_get_id()` hangs in application code
- ROMs must be located below 1M

Both due to SDK implementation of `Cache_Read_Enable_New()` which requires a variable to be set to 0 at startup.

Also, `system_os_post` must be in IRAM.

* Update README
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.

None yet

2 participants