-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
[TW#25601] Bug in esp-idf\components\driver\spi_common.c #2307
Comments
Hi @karawin . |
igrr
pushed a commit
that referenced
this issue
Aug 27, 2018
The dma configuration function called in the ISR should be put into the IRAM. Fixes #2307.
igrr
pushed a commit
that referenced
this issue
Aug 28, 2018
The dma configuration function called in the ISR should be put into the IRAM. Fixes #2307.
catalinio
pushed a commit
to catalinio/pycom-esp-idf
that referenced
this issue
Jun 28, 2019
The dma configuration function called in the ISR should be put into the IRAM. Fixes espressif/esp-idf#2307.
catalinio
pushed a commit
to catalinio/pycom-esp-idf
that referenced
this issue
Jun 28, 2019
The dma configuration function called in the ISR should be put into the IRAM. Fixes espressif/esp-idf#2307.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Environment
git rev-parse --short HEAD
to get the commit id.): 020ade6//bd6ea4393c7d2f059fc4decc70f1ec3eb3597268
Problem Description
Cache disabled but cached memory region accessed on time to time on spi master
Diagnostic:
A call to void spicommon_setup_dma_desc_links(lldesc_t *dmadesc, int len, const uint8_t *data, bool isrx)
in static void SPI_MASTER_ISR_ATTR spi_intr(void *arg) fails because spicommon_setup_dma_desc_links is not an IRAM_ATTR function.
This function is called in spi_intr on line 641 and 670
Adding IRAM_ATTR in esp-idf\components\driver\spi_common.c
line 333 ie:
void IRAM_ATTR spicommon_setup_dma_desc_links(lldesc_t *dmadesc, int len, const uint8_t *data, bool isrx)
solved the problem.
The text was updated successfully, but these errors were encountered: