-
Notifications
You must be signed in to change notification settings - Fork 1.2k
TinyUSB stack ported to MCXA and MXCN freedom boards. #2466
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
Conversation
frdm-mcxn947 folder created. Led and button pin modified compared to mcxn947brk board
BOARD_InitPins pinlist modification
Bunny brain board created, but not modified yet
MCXN and MCXA have different ARM Cortex M33 core, can't be on the same family
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much for your PR. It looks great, though there is some change request below:
MCX A and MCX N families created under hw/bsp folder. Due to differing compiler options required by different versions of ARM Cortex M-33 for MCX A and MCX N, they were not grouped together in the same family.
besides the DSP/FPU option, is there any other difference, we can solve this by either using the MCU_VARIANT check in family.mk/cmake or just set the correct CPU_CORE
in the board.mk/cmake instead. That would both A & N series under one mcx directory
Thank you a lot for your feedback. |
Hello Thach! |
I try to resolve conflict with master, and also want to make changes to the PR. Please edit PR to "allow maintainer to edit this pull request" |
Hi @hathach |
resolved the conflict, I don't why, but your PR contains also lots of unrelated changes to other files such as cdc_msc_freertos, nrf etc.. probably the git branch is messed up somehow. I have reverted those changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you for your PR. Everything work perfecty now, I made some changes to get it built with all examples. Currenlty ci failed on hil test, but that is a totally different issue by local pi4 setup. Will fix that in master.
TinyUSB stack ported to MCX A and MXC N freedom boards.
Notes
MCX A and MCX N families created under hw/bsp folder. Due to differing compiler options required by different versions of ARM Cortex M-33 for MCX A and MCX N, they were not grouped together in the same family.
Build code for make and cmake tools added under example/build_system to support the ARM Cortex M-33 core with no DSP and no FPU.
Since the NXP SDK repository is not up to date yet, the tools/get_deps.py file was modified to get the dependencies for MCX A, MCX N, etc. from our fork https://github.com/wavenumber-eng/mcux-sdk. We manually added the data from the MCUXpresso SDK generator to support MCX A. In a few weeks NXP will update their SDK repository, so no github fork will be necessary.
The building code was implemented and tested to work with make and cmake. The following examples have been tested on the MCX A and MCX N boards: