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.

Sign up for GitHub

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

Jump to bottom

uart requires rx and/or tx to be set.. (doc issue) #1356

Open
petermm opened this issue Oct 24, 2024 · 0 comments · May be fixed by #1357
Open

uart requires rx and/or tx to be set.. (doc issue) #1356

petermm opened this issue Oct 24, 2024 · 0 comments · May be fixed by #1357

Comments

@petermm
Copy link
Contributor

petermm commented Oct 24, 2024 •
edited
Loading

Uart docs: https://www.atomvm.net/doc/main/programmers-guide.html#uart suggest setting "UART2" will default rx/tx pins to 16/17.

However this doesn't seem correct and setting a rx pin, eg in case of a gps module is necessary.

eg. this doesn't work:
port = :uart.open("UART2", [{:speed, 9600}])
but this works (for :uart.read()):
port = :uart.open("UART2", [{:speed, 9600}, {:rx, 16}])

upstream docs:
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/peripherals/uart.html#uart-api-setting-communication-pins
@petermm petermm changed the title uart requires rx or tx to be set.. (doc issue) uart requires rx and/or tx to be set.. (doc issue) Oct 24, 2024
petermm added a commit to petermm/AtomVM that referenced this issue Oct 25, 2024
@petermm
Update UART programmers guide.
6a2dde9 
Previously failed to mention RX/TX options, and suggested they would default according to the schema, which they dont.

Fixes atomvm#1356

Signed-off-by: Peter M <petermm@gmail.com>
@petermm petermm linked a pull request Oct 25, 2024 that will close this issue
Open
petermm added a commit to petermm/AtomVM that referenced this issue Oct 25, 2024
@petermm
Update UART programmers guide.
bb1ecad 
Previously failed to mention RX/TX options, and suggested they would default according to the schema, which they dont.

Fixes atomvm#1356

Signed-off-by: Peter M <petermm@gmail.com>
petermm added a commit to petermm/AtomVM that referenced this issue Oct 28, 2024
@petermm
Update UART programmers guide.
9d0d748 
Previously failed to mention RX/TX options, and suggested they would default according to the schema, which they dont.

Fixes atomvm#1356

Signed-off-by: Peter M <petermm@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update UART programmers guide. petermm/AtomVM
1 participant
@petermm