-
Notifications
You must be signed in to change notification settings - Fork 13.3k
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
Bug-fix release EspSoftwareSerial 6.5.3 #6922
Conversation
b8e7f39
to
03c747f
Compare
@earlephilhower Done and ready for merge now. Thanks! |
@earlephilhower This 6.5.1 release fixes #6920, which on it's own is worth merging.
Not much to go by on USB logging. I wonder if IRAM is exhausted :-( ?
On ESP32, running same EspSoftwareSerial release, have yet to see this once the first time. |
OK, @dok-net, will wait for you to give all clear. IRAM is statically allocated. It either has enough space, and you get successful link, or it doesn't and you don't get an .ELF out, so it's not that. Are you getting anything on the HW serial port for the resets? WDT is one place the -32 and the 8266 are very different. Don't sweat the Win build failures. Looks like the early access Win VMs started pooping the bed. I'll look at it tonight, and if I can't figure out then I'll pull the Win build for now. Just worry about your own SWSerial stuff. |
First findings running Now adding WiFi.mode(WIFI_STA);
WiFi.begin("...", "...");
// Wait for connection
while (WiFi.status() != WL_CONNECTED) {
delay(500);
logger.print(".");
}
logger.println("");
logger.print("IP address: ");
logger.println(WiFi.localIP()); to But now, running with WiFi setup again, BOOM after a few seconds or minutes:
@earlephilhower This is on USB serial, same as I said before - isn't that the HW serial port that you are referring to, I have no other? This means there is no diagnostics at all. Anyway, in my mind this means there is no acute reason to believe this is caused by EspSoftwareSerial, specifically there is no regression, and 6.5.2 should be merged now to close the linked issue. Further testing: |
…c. by moving logger setup up to the front.
@earlephilhower could you please review and merge? I am a bit impatient with this, sorry 🙂 |
@dirkmueller In case you're a user of the fairly new parity/9bit feature, while searching for a cause of the resets I found and fixed a loss-of-sync bug - the parity reported drifted permanently from the actual data if there was an overflow. This occurred during delays >= 400ms at 19200bps, for instance :-) |
I didn't expect any of the changes to fix this, because I do not think it's anything in EspSoftwareSerial:
(19200bps)
(9600bps)
(9600bps)
Let's watch this, but not in this PR. Please really do merge. |
…of sync permanently.
Pending approval by 3rd party bug reporter