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

samd/mcu/samd21/mpconfigmcu.h:Allow user to enable additional options. #15239

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ricksorensen
Copy link
Contributor

Currently for samd21 some features are disable because of limited memory. With the ability to trade firmware and filesystem space, a user may wish to selectively enable some of these features. This change allows them to be enabled in board mpconfigboard.h or on the build command line for example. The selectively enable functions are:

  • MICROPY_PY_FRAMEBUF
  • MICROPY_PY_SELECT
  • MICROPY_PY_ONEWIRE
  • MICROPY_PY_ASYNCIO

@robert-hh
Copy link
Contributor

That's fine. As part of the make process also the setting for MICROPY_HW_CODESIZE has to be adapted accordingly.
Note: The commit message need a space character after the colon. Just change the commit locally with git commit --amend and then push it with git push <your_forks_name> samd_func -f.

Currently for samd21 some features are disable because of limited memory.
With the ability to trade firmware and filesystem space, a user may
wish to selectively enable some of these features.  This change allows
them to be enabled in board mpconfigboard.h or on the build command line
for example.  The selectively enable functions are:

 * MICROPY_PY_FRAMEBUF
 * MICROPY_PY_SELECT
 * MICROPY_PY_ONEWIRE
 * MICROPY_PY_ASYNCIO

Signed-off-by: Rick Sorensen <rick.sorensen@gmail.com>
Copy link

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
     mimxrt:    +0 +0.000% TEENSY40
        rp2:    +0 +0.000% RPI_PICO_W
       samd:    +0 +0.000% ADAFRUIT_ITSYBITSY_M4_EXPRESS

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