Skip to content

Update docs with new HardwareSerial buffer size #1786

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

Closed
PhilColbert opened this issue Mar 17, 2016 · 6 comments · Fixed by #3862
Closed

Update docs with new HardwareSerial buffer size #1786

PhilColbert opened this issue Mar 17, 2016 · 6 comments · Fixed by #3862

Comments

@PhilColbert
Copy link

PhilColbert commented Mar 17, 2016

From 2.0.0. to 2.1.0 the TX and RX buffer sizes are gone :(

My code is dealing with GPS input therefore requires a large buffer size to cope.... please put it back ! :)

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@igrr
Copy link
Member

igrr commented Mar 17, 2016

Removing the software-managed buffers was intentional (#1496), because otherwise we had issues when Serial was used from tasks other than main arduino task. For your application you can probably poll data at a higher rate. There is also a suggestion to support Arduino serialEvent API (#752).

@PhilColbert
Copy link
Author

Thank you for the reply.

Unfortunately I cant poll at a higher rate as the data requires parsing as it comes in and this requires uploading back to a wificlient and to a radio transmitter, this takes some time, in that time the serial buffer fills ...

Sorry I am still new to coding with the esp so might be coding it incorrectly.

I will just have to stick with 2.0.0 for now.

Thank you anyway.

@PhilColbert
Copy link
Author

I am receiving around 500 - 1000 bytes / sec... any idea how to deal with this issue in 2.1.0 without the serial buffer issue?

Thanks

@tablatronix
Copy link
Contributor

Does this mean the docs need to change ?

Apart from hardware FIFO (128 bytes for TX and RX) HardwareSerial has additional 256-byte TX and RX buffers

@igrr igrr changed the title No Hardware RX TX Buffer sizes Update docs with new HardwareSerial buffer size Mar 23, 2016
@igrr igrr added this to the 2.2.0 milestone Mar 23, 2016
@miky2k
Copy link

miky2k commented Mar 24, 2016

I suppose that hardaware fifo is still used.

@igrr igrr modified the milestones: 2.2.0, 2.3.0 Apr 18, 2016
@igrr igrr modified the milestones: 2.3.0, 2.4.0 Jun 3, 2016
@devyte
Copy link
Collaborator

devyte commented Oct 18, 2017

The docs for HardwareSerial need to be updated to describe the setRxBufferSize() method.

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

Successfully merging a pull request may close this issue.

5 participants