Skip to content

Increase available memory on the Feather M0#320

Merged
nitin710 merged 1 commit intomasterfrom
fix-FeatherM0FreeMemory
Aug 29, 2024
Merged

Increase available memory on the Feather M0#320
nitin710 merged 1 commit intomasterfrom
fix-FeatherM0FreeMemory

Conversation

@nitin710
Copy link
Collaborator

Description

  • Moved FileTransferManager definitions to live under ESP board definitions. Since FTP is available only on ESP32, it does not affect Feather M0.
  • This adds to the free memory available on the Feather M0, adding stability.

Requirements

  • None

Issues Referenced

  • None

Documentation update

None

Notes for Reviewer

  • Add detailed notes explaining code changes, algorithms or any other information that can help the reviewer understand the PR better.

List of new distributable files (For software PR only)

  • Name all new files added with this PR that need to be distributed with the release package. This includes files like .json, .xml settings files etc.

Testing

Results

  • ✔️ Repeated connect/disconnect
  • ✔️ Repeated Record begin/record end tests

Feature Tests

Shared files

  • Firmware binary: [Link to firmware binary]
  • Other files.

Checklist to allow merge

  • All dependent repositories used were on branch master
  • Firmware
    • Set testingMode to TestingMode::NONE
    • Set const bool DIGITAL_WRITE_DEBUG = false (if set true while testing)
    • Update version in EmotiBit.h
    • Update library.properties to the correct version (should match EmotiBit.h)
  • doxygen style comments included for new code snippets
  • Required documentation updated

Screenshots:

… is anyway only available on ESP32. This adds to the pool of free memory on the Feather M0
@nitin710 nitin710 merged commit da2a46a into master Aug 29, 2024
@nitin710 nitin710 deleted the fix-FeatherM0FreeMemory branch August 22, 2025 20:23
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.

1 participant