Skip to content

Add 38400 GPS baud option - #364

Open
FoxKyong wants to merge 1 commit into
jgauchia:masterfrom
FoxKyong:add-gps-baud-38400
Open

Add 38400 GPS baud option#364
FoxKyong wants to merge 1 commit into
jgauchia:masterfrom
FoxKyong:add-gps-baud-38400

Conversation

@FoxKyong

Copy link
Copy Markdown

Summary

This PR adds 38400 as a selectable GPS baud rate.

Some T-Deck Plus units with u-blox GNSS modules use 38400 baud by default. The current fixed baud rate list only contains 4800, 9600 and 19200, with AUTO at index 3, so 38400 cannot be selected from the GUI.

Changes

  • Add 38400 to GPS_BAUD[]
  • Move AUTO baud index from 3 to 4
  • Add 38400 to the GPS Speed dropdown
  • Set gpsBaudDetected when a fixed baud rate is selected, so the info command reports the active baud rate correctly
  • Keep AUTO as the default value

Tested on

  • Device: LilyGO T-Deck Plus
  • Environment: TDECK_ESP32S3
  • GPS baud: 38400
  • GPS update rate: 1 Hz

Test result

With 38400 selected:

  • NMEA parsing works
  • Parse OK increases continuously
  • CRC Errors remains 0
  • GNSS gets a 3D fix

Index mapping after this change

0 = 4800
1 = 9600
2 = 19200
3 = 38400
4 = AUTO

Comment thread lib/gps/gps.cpp Outdated
Comment thread lib/gps/gps.hpp
static const unsigned long GPS_BAUD[] = {4800, 9600, 19200, 0}; /**< Supported GPS baud rates. */
static const char *GPS_BAUD_PCAS[] = {"$PCAS01,0*1C\r\n", "$PCAS01,1*1D\r\n", "$PCAS01,2*1E\r\n"}; /**< NMEA command strings to set baud rate for PCAS modules. */
static const unsigned long GPS_BAUD[] = {4800, 9600, 19200, 38400, 0}; /**< Supported GPS baud rates. */
static const char *GPS_BAUD_PCAS[] = {"$PCAS01,0*1C\r\n", "$PCAS01,1*1D\r\n", "$PCAS01,2*1E\r\n", "$PCAS01,3*1F\r\n"}; /**< NMEA command strings to set baud rate for PCAS modules. */

@jgauchia jgauchia Jun 18, 2026

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I need to review GPS_BAUD_PCAS , because this only applies to AT6558D modules and maybe 38400 baud isn't available

@FoxKyong
FoxKyong force-pushed the add-gps-baud-38400 branch from 064b913 to 5258902 Compare June 18, 2026 11:15
@jgauchia jgauchia self-assigned this Jun 18, 2026
@jgauchia

Copy link
Copy Markdown
Owner

I'll include this PR in the next release , planned to be at end of June.

@jgauchia jgauchia added enhancement New feature or request Pre-release labels Jun 19, 2026
@FoxKyong

Copy link
Copy Markdown
Author

Thanks

@jgauchia

Copy link
Copy Markdown
Owner

Thanks

Thanks to you!,
I hope that my little project you like it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Pre-release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants