Skip to content

mimxrt/modmachine.c: Fix settings for the MIMXRT1170 board. #12877

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

Merged
merged 2 commits into from
Nov 4, 2023

Conversation

robert-hh
Copy link
Contributor

These were not changed with commit c0b64a3 for using tools/boardgen.py.

@jimmo
Copy link
Member

jimmo commented Nov 3, 2023

Thanks for this @robert-hh ! The build-all-the-boards script caught this overnight, thanks for the fix. See also #12878

@robert-hh
Copy link
Contributor Author

@jimmo: I just noticed that with PR #12211 the option disappeared in the MIMXRT board to denote a Pin by an integer. That way is used a lot in the documentation. So either one has to be adapted.

These were not changed with commit c0b64a3 for using tools/boardgen.py.

Signed-off-by: robert-hh <robert@hammelrath.com>
This option was removed in PR micropython#12211.

Signed-off-by: robert-hh <robert@hammelrath.com>
@dpgeorge dpgeorge merged commit e63d718 into micropython:master Nov 4, 2023
@robert-hh robert-hh deleted the mimxrt_pin_1170 branch November 5, 2023 06:58
@jimmo
Copy link
Member

jimmo commented Nov 6, 2023

Thanks for updating the documentation @robert-hh !

@robert-hh
Copy link
Contributor Author

Looking into the changed code it would have been complicated (or ugly) to restore the integer variant for naming a Pin. So I went for changing the documentation.

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