Merge branch 'bugfix/spi_concurrency_3.0' into 'release/v3.0'

spi: fix a possible concurrency issue (backport v3.0)

See merge request idf/esp-idf!5189
This commit is contained in:
Angus Gratton 2019-06-21 13:22:07 +08:00
commit 5340abcc7d
2 changed files with 12 additions and 6 deletions

View file

@ -419,6 +419,9 @@ static void IRAM_ATTR spi_intr(void *arg)
/*------------ new transaction starts here ------------------*/
//ToDo: This is a stupidly simple low-cs-first priority scheme. Make this configurable somehow. - JD
//Disable interrupt before checking to avoid concurrency issue.
esp_intr_disable(host->intr);
for (i=0; i<NO_CS; i++) {
if (host->device[i]) {
r=xQueueReceiveFromISR(host->device[i]->trans_queue, &host->cur_trans_buf, &do_yield);
@ -428,13 +431,14 @@ static void IRAM_ATTR spi_intr(void *arg)
}
}
if (i==NO_CS) {
//No packet waiting. Disable interrupt.
esp_intr_disable(host->intr);
#ifdef CONFIG_PM_ENABLE
//Release APB frequency lock
esp_pm_lock_release(host->pm_lock);
#endif
} else {
//enable the interrupt again if there is packet to send
esp_intr_enable(host->intr);
host->hw->slave.trans_done=0; //clear int bit
//We have a transaction. Send it.
spi_device_t *dev=host->device[i];

View file

@ -375,12 +375,14 @@ static void IRAM_ATTR spi_intr(void *arg)
}
}
//Disable interrupt before checking to avoid concurrency issue.
esp_intr_disable(host->intr);
//Grab next transaction
r = xQueueReceiveFromISR(host->trans_queue, &trans, &do_yield);
if (!r) {
//No packet waiting. Disable interrupt.
esp_intr_disable(host->intr);
} else {
if (r) {
//enable the interrupt again if there is packet to send
esp_intr_enable(host->intr);
//We have a transaction. Send it.
host->hw->slave.trans_done = 0; //clear int bit
host->cur_trans = trans;