fix(hal): force spiTransaction to update clock line immediately (backport of #9333) (#10697)

and3rson · web-flow · commit 1f038677eb2e · 2024-12-09T15:25:55.000+02:00
diff --git a/cores/esp32/esp32-hal-spi.c b/cores/esp32/esp32-hal-spi.c
@@ -1135,6 +1135,11 @@ void spiTransaction(spi_t * spi, uint32_t clockDiv, uint8_t dataMode, uint8_t bi
         spi->dev->ctrl.wr_bit_order = 1;
         spi->dev->ctrl.rd_bit_order = 1;
     }
+#if CONFIG_IDF_TARGET_ESP32C3 || CONFIG_IDF_TARGET_ESP32S3
+    // Sync new config with hardware, backported from https://github.com/espressif/arduino-esp32/pull/9333
+    spi->dev->cmd.update = 1;
+    while (spi->dev->cmd.update);
+#endif
 }
 
 void spiSimpleTransaction(spi_t * spi)

Original file line number	Diff line number	Diff line change
`@@ -1135,6 +1135,11 @@ void spiTransaction(spi_t * spi, uint32_t clockDiv, uint8_t dataMode, uint8_t bi`
`1135`	`1135`	`spi->dev->ctrl.wr_bit_order = 1;`
`1136`	`1136`	`spi->dev->ctrl.rd_bit_order = 1;`
`1137`	`1137`	`}`
	`1138`	`+#if CONFIG_IDF_TARGET_ESP32C3 \|\| CONFIG_IDF_TARGET_ESP32S3`
	`1139`	`+ // Sync new config with hardware, backported from https://github.com/espressif/arduino-esp32/pull/9333`
	`1140`	`+ spi->dev->cmd.update = 1;`
	`1141`	`+ while (spi->dev->cmd.update);`
	`1142`	`+#endif`
`1138`	`1143`	`}`
`1139`	`1144`
`1140`	`1145`	`void spiSimpleTransaction(spi_t * spi)`