Thanks to visit codestin.com
Credit goes to github.com

Skip to content

stm32/mpconfigport.h: Use the EXTRA feature level. #7958

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

Closed
wants to merge 3 commits into from

Conversation

jimmo
Copy link
Member

@jimmo jimmo commented Nov 1, 2021

Continuing on from #7655, this now updates mpconfig.h to make anything included by default on STM32 enabled at the EXTRA level, updates STM32 to use EXTRA, and removes any redundant config from stm32/mpconfigport.h.

Same for rp2, but with some additional customisations left in rp2/mpconfigport.h (these should be investigated in a future PR).

Note that many of the flags in mpconfig.h that have been set to EXTRA will end up being upgraded to CORE or BASIC (as we figure out what those boards are).

@codecov-commenter
Copy link

codecov-commenter commented Nov 1, 2021

Codecov Report

Merging #7958 (85851d0) into master (3b011d5) will decrease coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 85851d0 differs from pull request most recent head d8a3486. Consider uploading reports for the commit d8a3486 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7958      +/-   ##
==========================================
- Coverage   98.25%   98.24%   -0.01%     
==========================================
  Files         154      154              
  Lines       20007    20007              
==========================================
- Hits        19657    19656       -1     
- Misses        350      351       +1     
Impacted Files Coverage Δ
py/obj.c 97.22% <0.00%> (-0.40%) ⬇️
py/objlist.c 99.23% <0.00%> (-0.39%) ⬇️
py/runtime.c 99.39% <0.00%> (+0.15%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b011d5...d8a3486. Read the comment docs.

@jimmo jimmo force-pushed the port-config-template-stm32 branch 4 times, most recently from e03d9c5 to ebb2a03 Compare November 1, 2021 03:33
jimmo added 3 commits November 1, 2021 14:36
Some of these will later be moved to CORE or BASIC, but EXTRA is a
good starting point based on what STM32 uses.

Signed-off-by: Jim Mussared <[email protected]>
The idea is to make this a no-op change. Future improvements can come from
making individual boards use CORE or BASIC.

Signed-off-by: Jim Mussared <[email protected]>
No-op change to simplify existing config.

Signed-off-by: Jim Mussared <[email protected]>
@jimmo jimmo force-pushed the port-config-template-stm32 branch from ebb2a03 to d8a3486 Compare November 1, 2021 03:37
@dpgeorge
Copy link
Member

dpgeorge commented Nov 1, 2021

Merged in 0e236ee through b1a0ce4

@dpgeorge dpgeorge closed this Nov 1, 2021
tannewt added a commit to tannewt/circuitpython that referenced this pull request May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants