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

STM32: Wrong uart_event_tx len calculation #35036

Closed
erwango opened this issue May 10, 2021 · 0 comments · Fixed by #34762
Closed

STM32: Wrong uart_event_tx len calculation #35036

erwango opened this issue May 10, 2021 · 0 comments · Fixed by #34762
Assignees
@erwango
Labels
area: DMA Direct Memory Access area: UART Universal Asynchronous Receiver-Transmitter bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: low Low impact/importance bug
Milestone
v2.6.0

Comments

@erwango
Copy link
Member

erwango commented May 10, 2021

Describe the bug
We need to reset the buffer_length when the transmission is finished, but in order to give the correct value to
the uart_event_tx struct, we use the dma_tx.buffer_length in the calculation of the dma_tx.counter, used for the len of the event (number of bytes sent).
@erwango erwango added bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug area: UART Universal Asynchronous Receiver-Transmitter area: DMA Direct Memory Access platform: STM32 ST Micro STM32 labels May 10, 2021
@erwango erwango self-assigned this May 10, 2021
@erwango erwango mentioned this issue May 10, 2021
Merged
@galak galak added this to the v2.6.0 milestone May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erwango erwango

Labels
area: DMA Direct Memory Access area: UART Universal Asynchronous Receiver-Transmitter bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: low Low impact/importance bug
Projects
None yet
Milestone
v2.6.0
Development

Successfully merging a pull request may close this issue.

drivers: serial: stm32: Fixes uart_event_tx len calculation JonathanVanWin/zephyr
2 participants
@galak @erwango