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

fix(kakuteh7.default.px4board): remove CONFIG_BOARD_SERIAL_TEL2 #23553

Closed
wants to merge 1 commit into from

Conversation

iftahnaf
Copy link

Solved Problem

From the documentation, Kakute H7 does not have the TELEM2 option. It does appear in the default.px4board file:

CONFIG_BOARD_SERIAL_TEL2="/dev/ttyS1"

Solution

The simple solution is to remove the line from the file so it won't appear as an option.

Alternatives

We could also disable serial console and use R3/T3 pins as TELEM2:

  1. Disable the serial console:

  2. Change the CONFIG_BOARD_SERIAL_TEL2 in the default.px4board to the correct UART (from the documentation:

CONFIG_BOARD_SERIAL_TEL2="/dev/ttyS2"

@vincentpoont2 I would appreciate it if you could review this.

@beniaminopozzan
Copy link
Member

side question: is /dev/ttyS1 the one referred to as UART 2 in the Holybro documentation?

@iftahnaf
Copy link
Author

I'm not sure, but I guess so; I couldn't get a clear answer from Holybro's tech support.

@vincentpoont2
Copy link
Member

@iftahnaf Thanks for the suggestion.
KakuteH7's Telem2 is connected to the UART of the Bluetooth chip via a resistor inside the board, but it has no exposed solder pad.
image

Although it is not exposed by default. There are workaround, so I think it's better to keep this line of config.
The workaround is as follow:

image

There might a version in the future that doesn't have bluetooth and exposed the UART2.

@vincentpoont2
Copy link
Member

side question: is /dev/ttyS1 the one referred to as UART 2 in the Holybro documentation?

-----Yes, dev/ttys1 corresponds to UART2
-----The UART correspondence of PX4 is ttys0->UART1, ttys1->UART2, ttys2->UART3, and so on

@iftahnaf
Copy link
Author

@vincentpoont2 Awesome, thanks for the clarification.
I'll add this to the PX4 Kakute H7 docs.

@iftahnaf iftahnaf closed this Aug 16, 2024
@iftahnaf iftahnaf deleted the fix/remove-telem2-from-kakute-h7 branch August 16, 2024 13:27
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.

3 participants