Skip to content

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

Merged
dpgeorge merged 2 commits into
micropython:masterfrom
robert-hh:mimxrt_pin_1170
Nov 4, 2023
Merged

mimxrt/modmachine.c: Fix settings for the MIMXRT1170 board.#12877
dpgeorge merged 2 commits into
micropython:masterfrom
robert-hh:mimxrt_pin_1170

Conversation

@robert-hh
Copy link
Copy Markdown
Contributor

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

@jimmo
Copy link
Copy Markdown
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
Copy Markdown
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 <[email protected]>
@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
Copy Markdown
Member

jimmo commented Nov 6, 2023

Thanks for updating the documentation @robert-hh !

@robert-hh
Copy link
Copy Markdown
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