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

[msom] Ignore CIMI errors on bg95s5 with no esim profile loaded #2866

Merged
merged 2 commits into from
Mar 3, 2025

Conversation

scott-brust
Copy link
Member

@scott-brust scott-brust commented Feb 19, 2025

Problem

QuectelNcpClient::connect() for bg95-s5 fails when no esim profile is loaded due to IMSI AT command failing.
DVOS will reset the modem continually each time connect() fails. This is causing problems at the factory as they require a stable modem USB connection for testing.

Solution

Ignore AT+CIMI errors for BG95S5 modems. We will assume it has no profile loaded

Steps to Test

Run DVOS on M635E device, it should idle and not reset modem

Example App

Tinker

References

Links to the Community, Docs, Other Issues, etc..


Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA (Info here)
  • Problem and Solution clearly stated
  • Run unit/integration/application tests on device
  • Added documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)

@scott-brust scott-brust added this to the 6.3.1 milestone Feb 19, 2025
@scott-brust scott-brust changed the title Ignore CIMI errors on bg95s5 with no esim profile loaded [msom] Ignore CIMI errors on bg95s5 with no esim profile loaded Feb 19, 2025
@scott-brust scott-brust force-pushed the fix/bg95s5-no-profile-resets branch from be9bc36 to 53aa028 Compare February 25, 2025 04:08
@scott-brust scott-brust merged commit 507ab15 into develop Mar 3, 2025
14 checks passed
@scott-brust scott-brust deleted the fix/bg95s5-no-profile-resets branch March 3, 2025 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants