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

Memory allocation is now using heap_caps API #2107

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

josesimoes
Copy link
Member

Description

  • Memory allocation is now using heap_caps API and always requesting memory from internal RAM.

Motivation and Context

  • This is required to make use that malloc and free call the correct API for targets with and without PSRAM.
  • Also this ensures that there are no cache or other issues in memory allocated dynamically in targets with PSRAM.

How Has This Been Tested?

Screenshots

Types of changes

  • Improvement (non-breaking change that improves a feature, code or algorithm)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Config and build (change in the configuration and build system, has no impact on code or features)
  • Dependencies (update dependencies and changes associated, has no impact on code or features)
  • Unit Tests (work on Unit Tests, has no impact on code or features)
  • Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

- This is required to make use that malloc and free call the correct API for targets with and without PSRAM.
@josesimoes josesimoes added the Series: ESP32 Everything related specifically with ESP32 series targets label Oct 19, 2021
@josesimoes josesimoes merged commit 0c62a06 into nanoframework:develop Oct 19, 2021
@josesimoes josesimoes deleted the fix-platform-malloc branch October 19, 2021 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Series: ESP32 Everything related specifically with ESP32 series targets Type: enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants