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

ports/nxp: Plan to Integrate Micropython and NXP's MCU #5028

Closed
wants to merge 1 commit into from
Closed

ports/nxp: Plan to Integrate Micropython and NXP's MCU #5028

wants to merge 1 commit into from

Conversation

Albort12138
Copy link
Contributor

-Add nxp MIMXRT1064 port.
-Enable sdcard lcd lpi2c pinmux led

@Albort12138
Copy link
Contributor Author

This PR tries to integrate Micropython and MIMXRT1064 MCU. The new submodule named as 'nxplib' was added to contain the copy the NXP MCUxpresso SDK. Currently, some basic feature was enabled(eg pinmux sdcard lpi2c littlevgl led). Since I am a new one to contribute to the community there are some mistakes. I will be grateful if some of you guys can give me some suggestions.

 -Add nxp MIMXRT1064 port.
 -Enable sdcard lcd lpi2c pinmux led
@dpgeorge
Copy link
Member

Thanks for making the PR cleaner with the submodule.

I guess the first question is: what is the scope of this port, what variants of MCUs could it potentially target? Is it just the MIMXRT1064, or all MIMXRTxxxx's, or also i.MX6/8? The name of the port should reflect the MCUs that it supports, eg maybe this port should be called mimxrt?

@Albort12138
Copy link
Contributor Author

Thanks for making the PR cleaner with the submodule.

I guess the first question is: what is the scope of this port, what variants of MCUs could it potentially target? Is it just the MIMXRT1064, or all MIMXRTxxxx's, or also i.MX6/8? The name of the port should reflect the MCUs that it supports, eg maybe this port should be called mimxrt?

Thanks for the Reply. We have the plan to enable the majority of NXP's MCUs( if they can) including Kinetis series LPC series and i.mx rt series. Currently, we take mimxrt1064 as the try and enable little of IP module. If this PR was approved. We will enable more MCUs. What more, mimxrt1050 mimxrt1060 mimxrt1020 have been enabled(have not been updated). Through this PR, we hope to get suggestions from you guys.

@Albort12138
Copy link
Contributor Author

Hi @dpgeorge , please help me review this PR?. We hope to accelerate the progress of community support.

@ezequielgarcia
Copy link

Hi @yanyao12138, have you noticed this PR from @martinribelotta (from 2017) #3400 that adds support for LPC43xx family? Perhaps you can start from that, or reuse some of the effort. You will see that the port is really clean, currently consisting in just 700 LoC.

Copy link
Sponsor

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, is it possible to remove FreeRTOS from this? Most MicroPython ports don't use an RTOS. The boiler plate from the board directory would be better shared between boards.

LittleVGL is also large on its own and would be better added as a second PR.

Thanks for this first PR!

@Albort12138
Copy link
Contributor Author

Hi, is it possible to remove FreeRTOS from this? Most MicroPython ports don't use an RTOS. The boiler plate from the board directory would be better shared between boards.

LittleVGL is also large on its own and would be better added as a second PR.

Thanks for this first PR!

Hi Scott, thanks for the reply. I will remove the FrreeRTos and LittleVGl.

@Albort12138 Albort12138 deleted the NXP_PORT_MIMXRT1064 branch April 22, 2020 09:14
tannewt added a commit to tannewt/circuitpython that referenced this pull request Jul 23, 2021
ulab: move documentation to the right location
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

4 participants