Skip to content

feat: add ping() method to verify connectivity and to 'fix' baud rate#165

Open
rayozzie wants to merge 1 commit intomasterfrom
ray/autobaud
Open

feat: add ping() method to verify connectivity and to 'fix' baud rate#165
rayozzie wants to merge 1 commit intomasterfrom
ray/autobaud

Conversation

@rayozzie
Copy link
Copy Markdown
Contributor

In the specific case of using the Notecard in 'req' mode on the AUX port, it can be challenging to deal with baud rate changes robustly.

Specifically, consider the case where the host wants to talk at 460800 and the notecard is sitting at 115200, or vice-versa. How does the host send it the card.aux.serial request if it doesn't know what baud rate it's sitting at?

This change implements a "ping()" request which attempts to verify the ability to communicate with the notecard. If on serial, and if it CANNOT communicate with the notecard, it will attempt to step through the most common baud rates until it can. And regardless, if it's on aux, it will change the baud rate to the one that was specified on begin().

In the specific case of using the Notecard in 'req' mode on the AUX port, it can be challenging to deal with baud rate changes robustly.

Specifically, consider the case where the host wants to talk at 460800 and the notecard is sitting at 115200, or vice-versa.  How does the host send it the card.aux.serial request if it doesn't know what baud rate it's sitting at?

This change implements a "ping()" request which attempts to verify the ability to communicate with the notecard.  If on serial, and if it CANNOT communicate with the notecard, it will attempt to step through the most common baud rates until it can.  And regardless, if it's on aux, it will change the baud rate to the one that was specified on begin().
@rayozzie rayozzie requested a review from zfields April 10, 2026 20:56
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.

1 participant