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

boards: rddrone_fmuk66: correct pinmux errors #44780

Merged
merged 1 commit into from
Apr 13, 2022
Merged

boards: rddrone_fmuk66: correct pinmux errors #44780

merged 1 commit into from
Apr 13, 2022

Conversation

danieldegrasse
Copy link
Collaborator

Correct errors in rddrone pinmux. Selection for UART4 RTS line was
incorrect, as was pinmux for red PWM LED.

Fixes #44314

Signed-off-by: Daniel DeGrasse [email protected]

MaureenHelm
MaureenHelm previously approved these changes Apr 12, 2022
dleach02
dleach02 previously approved these changes Apr 12, 2022
Copy link
Member

@dleach02 dleach02 left a comment

Choose a reason for hiding this comment

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

Works on my test setup

@mmahadevan108
Copy link
Collaborator

@danieldegrasse Could you please add some more details to the commit message. There are some more changes in this commit in addition to fixing the UART4 RTS line

Correct the following errors in rddrone pinmux:
- Pin selection for UART4 RTS line was incorrect, should use PTE27
- Pin selection for red PWM LED was incorrect, should use PTD1
- FlexCAN0/1 RX should not have pullup resistors (per prior pinmux.c
settings)
- FTM0_CH1_PTA4 does not require a pullup resistor
- LPUART0_RX_PTD8 requires a pullup resistor for UART RX to work

Fixes #44314

Signed-off-by: Daniel DeGrasse <[email protected]>
@dleach02 dleach02 requested a review from MaureenHelm April 12, 2022 18:07
@mbolivar-nordic mbolivar-nordic merged commit e4b76ef into zephyrproject-rtos:main Apr 13, 2022
@dleach02 dleach02 deleted the rddrone-pinmux-fixes branch July 17, 2024 01:51
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.

rddrone_fmuk66: fatal error upon running basic samples
5 participants