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

Electron modem USART and buffer handling fixes #1140

Merged
merged 3 commits into from
Nov 28, 2016

Conversation

avtolstoy
Copy link
Member

@avtolstoy avtolstoy commented Oct 17, 2016

Closes #1138 and #1104.

Fixes several issues:

  • USART_FLAG_TXE was not checked when writing into DR register from system thread which caused an unsent data byte to be overwritten
  • RX flow control was not correctly handled. When an internal ElectronSerialPipe buffer overflows, a data byte should be left in DR register to de-assert RTS line to ask modem to pause transmission for a while
  • ElectronSerialPipe circular buffer was initialized to 1024 bytes, but the implementation allowed only 1023 bytes to be held inside the buffer
  • AT+CLAC-like commands weren't parsed correctly (thanks @technobly)

Relevant tests:

  • MDM_01_socket_writes_with_length_more_than_1023_work_correctly
  • MDM_02_at_commands_with_long_response_are_correctly_parsed_and_flow_controlled

Doneness:

  • Contributor has signed CLA
  • Problem and Solution clearly stated
  • Code peer reviewed
  • API tests compiled
  • Run unit/integration/application tests on device
  • Add documentation (N/A)
  • Add to CHANGELOG.md after merging (add links to docs and issues)

Bugfixes

@technobly
Copy link
Member

@avtolstoy any comment on my question above? Let me know if you need me to verify tests on a particular electron.

@technobly technobly self-assigned this Nov 21, 2016
@technobly technobly merged commit 99abe66 into develop Nov 28, 2016
@technobly technobly deleted the feature/electron-mdm-usart-fixes branch November 28, 2016 21:02
@technobly technobly removed their assignment Nov 29, 2016
@technobly technobly modified the milestones: 0.7.x, 0.6.1 Nov 29, 2016
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.

[Electron] AT commands with long responses not completing
2 participants