search

FactbirdHQ / atat

no_std crate for parsing AT commands
Apache License 2.0
109 stars 29 forks source link

Recover from unexpected prompt `@\n@` #121

Closed simonvandel closed 2 years ago

simonvandel commented 2 years ago

It does not fix the root cause of the extra \n@ though

MathiasKoch commented 2 years ago

I think it is okay to create a temporary OTA version with this to hotfix our current issue, bit I am not sure I think it belongs in the master of this repo? As you say it is a temp fix around an issue that has nothing to do with atat, and are more likely introduced somewhere else in our devices? Either uart config, DMA or similar.

Thoughts?

simonvandel commented 2 years ago

I agree with that, yeah. The core issue is likely elsewhere. I'll close the PR, but cherry-pick the commit for the temporary OTA until we find a more lasting solution.