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

Serial callback problem #1214

Closed
kwis2 opened this issue Aug 1, 2017 · 2 comments
Closed

Serial callback problem #1214

kwis2 opened this issue Aug 1, 2017 · 2 comments
Assignees

Comments

@kwis2
Copy link

kwis2 commented Aug 1, 2017

When I send garbage data to serial module work unstable. I have tested sample project and there is the same behaviour. For instance with Basic_rBoot example it is enought to send qwertyuiopasdfghjklzxcvbnm string to fool module. It looks like there is problem with flushing data.

below is log with added

Serial.println(str);
stream.flush();

added into serialCallBack.

->qwertyuiopasdfghjklzxcvbnm
qwertyuiopas
unknown command
->help
dfghjk[10]@ Q[10]@
unknown command
->help
lzxcvb[10]@ Q[10]@
unknown command
->help
nm
unknown command
->help
lp
unknown command
->help
lp
unknown command
->help
lp
unknown command
->help
lp
unknown command
->help
lp
unknown command
->help
lp
unknown command
->help
lp
unknown command

Sming 3.3.0, module ESP8266-01

@slaff slaff self-assigned this Sep 18, 2017
slaff pushed a commit to slaff/Sming that referenced this issue Nov 22, 2017

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
without advancing the internal stream pointers.

Fixes: SmingHub#1214.
slaff pushed a commit to slaff/Sming that referenced this issue Nov 22, 2017
without advancing the internal stream pointers.

Fixes: SmingHub#1214.
slaff pushed a commit to slaff/Sming that referenced this issue Nov 22, 2017
without advancing the internal stream pointers.

Fixes: SmingHub#1214.
@slaff
Copy link
Contributor

slaff commented Nov 22, 2017

@kwis2 Can you check if that PR #1290 fixes the reported problem?

@kwis2
Copy link
Author

kwis2 commented Nov 25, 2017

@slaff unfortunatelly I am not able to install current version of sming #1241

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants