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.

Sign up for GitHub

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

Jump to bottom

Command Timeouts (IDFGH-8185) #129

Closed
AshUK opened this issue Aug 30, 2022 · 1 comment · Fixed by #139
Closed

Command Timeouts (IDFGH-8185) #129

AshUK opened this issue Aug 30, 2022 · 1 comment · Fixed by #139
Labels
Resolution: Done Status: Done

Comments

@AshUK
Copy link

AshUK commented Aug 30, 2022

This is somewhat related to #66

From testing and from the BG96/95 modem data sheet require longer timeouts.

  • set_radio_state/AT+CFUN= upto 15s
  • set_pdp_context/AT+CGACT= upto 150s

I have from testing been seeing both of these commands timing out when using the default 300ms.

It would also be useful if the esp_modem_at function had a custom timeout parameter.

See BG95&BG77_AT_Commands_Manual.pdf pages 27 and 134.
@AshUK AshUK changed the title Custom Timeouts Command Timeouts Aug 30, 2022
@github-actions github-actions bot changed the title Command Timeouts Command Timeouts (IDFGH-8185) Aug 30, 2022
david-cermak added a commit to david-cermak/esp-protocols that referenced this issue Sep 9, 2022
@david-cermak
fix(esp_modem): Correct timeouts for certain commands
1029078 
And adds an explicit timeout parameter to the esp_modem_at()

Closes espressif#129
@david-cermak david-cermak mentioned this issue Sep 9, 2022
Merged
@david-cermak
Copy link
Collaborator

Timeouts updated in #139, also added the timeout parameter to the at() command.
Thanks for reporting!

euripedesrocha pushed a commit to euripedesrocha/esp-protocols that referenced this issue Oct 17, 2022
@david-cermak @euripedesrocha
fix(esp_modem): Correct timeouts for certain commands
d0c4c8d 
And adds an explicit timeout parameter to the esp_modem_at()

Closes espressif#129
gabsuren pushed a commit to gabsuren/esp-protocols-1 that referenced this issue Oct 31, 2022
@david-cermak @suren-gabrielyan-espressif
fix(esp_modem): Correct timeouts for certain commands
fed0b36 
And adds an explicit timeout parameter to the esp_modem_at()

Closes espressif#129
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Resolution: Done Status: Done
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(esp_modem): Make get_operator_name() return also ACT value david-cermak/esp-protocols
3 participants
@AshUK @david-cermak @espressif-bot