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.

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

New SPI Master transaction flag to not use DMA on DMA enabled host. (IDFGH-7501) #9069

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Pe3ucTop
Copy link

This pull request add transaction option to not use DMA for case when host have DMA enabled.

New Flag: SPI_TRANS_DONT_DMA
Prevent DMA usage for transaction = process it without DMA.
Reason described in issue #8636
It work for me with ESP32 , did not tested on other chips.
Documentation need be updated by someone.

Flag: SPI_TRANS_DONT_DMA
Prevend DMA usage for transaction = process it without DMA.
@CLAassistant
Copy link

CLAassistant commented May 31, 2022

CLA assistant check
All committers have signed the CLA.

@espressif-bot espressif-bot added the Status: Opened Issue is new label May 31, 2022
@github-actions github-actions bot changed the title New SPI Master transaction flag to not use DMA on DMA enabled host. New SPI Master transaction flag to not use DMA on DMA enabled host. (IDFGH-7501) May 31, 2022
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution.

@suda-morris suda-morris requested a review from ginkgm June 1, 2022 03:46
@ginkgm
Copy link
Collaborator

ginkgm commented Jun 1, 2022

Relates to: #8636

@Pe3ucTop
Copy link
Author

Pe3ucTop commented Jul 19, 2022

@ginkgm any progress on review ?

@Pe3ucTop
Copy link
Author

@ginkgm , @suda-morris , @Alvin1Zhang is there anything I can do (more description / comments / documentation / ??) to help on review ?

@Pe3ucTop
Copy link
Author

@ginkgm
I fixed conflict between branches, would like to hear your review.

@Pe3ucTop
Copy link
Author

Any updates ?

@Pe3ucTop
Copy link
Author

Pe3ucTop commented Sep 8, 2023

@ginkgm , @Alvin1Zhang
Sorry, is there any stoppers in review ?
Can I help somehow ?

@espressif-bot espressif-bot added the Type: Feature Request Feature request for IDF label Sep 13, 2023
@espressif-bot espressif-bot added Status: Selected for Development Issue is selected for development and removed Status: Opened Issue is new labels Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Selected for Development Issue is selected for development Type: Feature Request Feature request for IDF
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants