Skip to content

Conversation

kbidani
Copy link
Contributor

@kbidani kbidani commented Aug 18, 2025

The source and destination addresses in the HAL_MDMA_Start_IT call were swapped, causing incorrect data transfers. The data buffer (pData) should be the source, and the OSPI data register (DR) should be the destination. This commit fixes the argument order to ensure proper data transmission via DMA.

fixes: #94350

Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks.
Is there a matching issue on HAL so we're sure this is also fixed in HAL directly ?
Also, please update README.

@kbidani kbidani force-pushed the fix_ospi_transmit_dma branch from 4a38537 to f6167f6 Compare August 19, 2025 14:46
@kbidani
Copy link
Contributor Author

kbidani commented Aug 19, 2025

README updated.

@erwango
Copy link
Member

erwango commented Aug 25, 2025

Please rebase and provide a matching zephyr PR (updating west manifest)

…resses

The source and destination addresses in the HAL_MDMA_Start_IT call were swapped,
causing incorrect data transfers. The data buffer (pData) should be the source,
and the OSPI data register (DR) should be the destination. This commit fixes the
argument order to ensure proper data transmission via DMA.

Signed-off-by: Khaoula Bidani <[email protected]>
@erwango erwango merged commit 465fb02 into zephyrproject-rtos:main Aug 29, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants