Add an example of using using DMA with PIO #468
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
peterharperuk
force-pushed
the
uart_pio_dma
branch
from
988bccc
to
47053b4
Compare
peterharperuk
force-pushed
the
uart_pio_dma
branch
from
47053b4
to
9232a3a
Compare
lurch
reviewed
Jan 26, 2024
peterharperuk
force-pushed
the
uart_pio_dma
branch
from
9232a3a
to
3fd5556
Compare
lurch
reviewed
Jan 26, 2024
lurch
reviewed
Jan 26, 2024
peterharperuk
force-pushed
the
uart_pio_dma
branch
from
3fd5556
to
77461c7
Compare
peterharperuk
commented
Jan 29, 2024
pio/uart_pio_dma/uart_pio_dma.c
Outdated
|
|
||
| // Allocates a pio and statemachine and loads the pio program into memory | ||
| // Return true on success | ||
| static bool pio_init(const pio_program_t *program, PIO *pio_hw, uint *sm, uint *offset) { |
There was a problem hiding this comment.
See discussion on naming
peterharperuk
commented
Jan 29, 2024
pio/uart_pio_dma/uart_pio_dma.c
Outdated
| } | ||
| } | ||
| // Find a state machine | ||
| *sm = (int8_t)pio_claim_unused_sm(*pio_hw, false); |
There was a problem hiding this comment.
It might be possible to add a program but there are no sm's free?
peterharperuk
commented
Jan 29, 2024
| // write to the pio fifo | ||
| int count_pio_tx = 0; | ||
| while(count_pio_tx < sizeof(buffer_tx) - 1) { | ||
| uart_tx_program_putc(pio_hw_tx, pio_sm_tx, buffer_tx[count_pio_tx++]); |
There was a problem hiding this comment.
This and the following uart_puts will overflow the fifo if dma is not enabled, need to read and write one character at a time?
peterharperuk
force-pushed
the
uart_pio_dma
branch
from
77461c7
to
67e3171
Compare
kilograham
reviewed
Nov 19, 2024
|
|
||
| // Allocates a pio and statemachine and loads the pio program into memory | ||
| // Return true on success | ||
| static bool pio_claim_free_sm_add_program(const pio_program_t *program, PIO *pio_hw, uint *sm, uint *offset) { |
There was a problem hiding this comment.
seems like this can use the new PIO functions
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #467