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

Cleanup for Air530z GPS #4344

Merged
merged 2 commits into from
Jul 31, 2024
Merged

Conversation

fifieldt
Copy link
Contributor

It turns out that the Air530z is a GNSS_MODEL_MTK. Our existing code detects and configures it properly. This patch removes a couple of AIROHA ifdefs since they are not required for a working GPS.

fifieldt and others added 2 commits July 29, 2024 13:02
It turns out that the Air530z is a GNSS_MODEL_MTK. Our existing
code detects and configures it properly. This patch removes a
couple of AIROHA ifdefs since they are not required for a
working GPS.
@thebentern thebentern merged commit ce1eb14 into meshtastic:master Jul 31, 2024
98 checks passed
@markbirss
Copy link
Contributor

@fifieldt This change also affects Tracker T1000-e GPS

@fifieldt
Copy link
Contributor Author

fifieldt commented Aug 2, 2024

yes @markbirss , I believe setting #define GPS_BAUDRATE 115200 in the T1000e variant.h is incorrect: the Air530z defaults to 9600. Try removing that line?

@markbirss
Copy link
Contributor

yes @markbirss , I believe setting #define GPS_BAUDRATE 115200 in the T1000e variant.h is incorrect: the Air530z defaults to 9600. Try removing that line?

Does not seem to help.

@fifieldt
Copy link
Contributor Author

fifieldt commented Aug 2, 2024

What's breaking? What is it detecting as? Can you confirm if the Air530 in the T1000E is the same as the Air530 available for sale from the Seeed store (I.e. an Air530z)?

@markbirss
Copy link
Contributor

@failed "Failed to find UBlox & MTK GNSS Module using baudrate"

I cannot confirm only have T1000-E

@Dylanliacc can you confirm ?

@fifieldt
Copy link
Contributor Author

fifieldt commented Aug 2, 2024

Thanks for looking at it. If it's not too much trouble, what are the baud rates it tries?

@markbirss
Copy link
Contributor

markbirss commented Aug 2, 2024

It does try couple 9600,4800, 57600 and 115200 but still fails

DEBUG | ??:??:?? 2 [GPS] Probing for GPS at 9600                                 
WARN  | ??:??:?? 6 [GPS] Failed to find UBlox & MTK GNSS Module using baudrate 9
600

@fifieldt
Copy link
Contributor Author

fifieldt commented Aug 4, 2024

Working on something I think will fix t1000e

master...fifieldt:meshtastic-firmware:l89-2

@fifieldt fifieldt deleted the add-air530z-support branch August 5, 2024 12:42
@fifieldt
Copy link
Contributor Author

fifieldt commented Aug 5, 2024

Working on the fix in #4394

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