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

SIM7600 Protocol enhancements (Fixes #624) #736

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

imorda
Copy link

@imorda imorda commented Jul 3, 2023

  1. SIM7600 delay in restartImpl() is not enough, a real module takes at least 23 seconds to reboot before it starts to respond to any AT commands.
  2. (Fixes [SIM7600] Get GPS Coordinates locking for multiple seconds #624) The getGPSImpl(...) method expects two extra "," in the response that breaks the iaccuracy variable and also causes it to skip the OK part in the end of the response that leads to the 3-seconds waiting until the timeout occurs. After that the waitResponse() in the last line of that method also does not receive the proper OK and also waits 1 extra second until the timeout.

image

@imorda
Copy link
Author

imorda commented Jul 3, 2023

Don't worry, a small force push to fix the git credentials, branches before and after the force push are identical

jrlanoisTA2023 added a commit to jrlanoisTA2023/TinyGSM that referenced this pull request Jan 12, 2024
jorisvervuurt added a commit to jorisvervuurt/TinyGSM_SIM7600 that referenced this pull request Feb 9, 2024
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.

[SIM7600] Get GPS Coordinates locking for multiple seconds
2 participants