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

Add additional iMX RT support #7921

Merged
merged 5 commits into from
May 2, 2023
Merged

Add additional iMX RT support #7921

merged 5 commits into from
May 2, 2023

Conversation

tannewt
Copy link
Member

@tannewt tannewt commented Apr 28, 2023

This adds a script to generate the peripherals files (except clock).

It adds support for the 1015, 1020, 1040, and 1050 EVKs.

Some work was started on 1176 but it isn't working. So, the board def is in a separate branch.

Fixes #3521. Fixes #2477.

This adds a script to generate the peripherals files (except clock).

It adds support for the 1015, 1020, 1040, and 1050 EVKs.

Some work was started on 1176 but it isn't working. So, the board
def is in a separate branch.

Fixes micropython#3521. Fixes micropython#2477.
@tannewt tannewt added the mimxrt10xx iMX RT based boards such as Teensy 4.x label Apr 28, 2023
@tannewt tannewt requested a review from jepler April 28, 2023 18:09
Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

Glad you got these working. A few minor things.

I think it would be better to call these boards nxp_mimx... instead of just mimx.... I know the first one was not named that. I would really like to see the manufacturer included in the board name. I know we haven't been consistent, and it's going to be a job to rename them all. So if you don't want to do it now and we do it in a big bang later, that's OK.

ports/mimxrt10xx/boards/imxrt1015_evk/mpconfigboard.mk Outdated Show resolved Hide resolved
ports/mimxrt10xx/boards/imxrt1015_evk/pins.c Show resolved Hide resolved
ports/mimxrt10xx/boards/imxrt1040_evk/mpconfigboard.mk Outdated Show resolved Hide resolved
ports/mimxrt10xx/boards/imxrt1040_evk/pins.c Show resolved Hide resolved
ports/mimxrt10xx/boards/imxrt1050_evkb/mpconfigboard.mk Outdated Show resolved Hide resolved
ports/mimxrt10xx/boards/imxrt1050_evkb/pins.c Show resolved Hide resolved
ports/mimxrt10xx/common-hal/microcontroller/Processor.c Outdated Show resolved Hide resolved
Copy link
Member

@jepler jepler left a comment

Choose a reason for hiding this comment

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

Thanks -- I have some questions though.

lib/tinyusb Outdated Show resolved Hide resolved
ports/mimxrt10xx/tools/gen_peripherals_data.py Outdated Show resolved Hide resolved
@dhalbert dhalbert dismissed their stale review April 29, 2023 02:22

Points addressed

@tannewt tannewt requested a review from jepler May 1, 2023 21:49
Copy link
Member

@jepler jepler left a comment

Choose a reason for hiding this comment

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

thanks for addressing my concern!

@jepler jepler merged commit 80e1418 into adafruit:main May 2, 2023
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mimxrt10xx iMX RT based boards such as Teensy 4.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

iMX RT 1020 EVK doesn't work mimxrt10xx: Support the rest of the EVKs
3 participants