fix(hal): force spiTransaction to update clock line immediately (espressif#9221)

and3rson · and3rson · commit 5d9cfbaceef6 · 2024-03-04T17:07:15.000+02:00
diff --git a/cores/esp32/esp32-hal-spi.c b/cores/esp32/esp32-hal-spi.c
@@ -1115,6 +1115,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, fixes https://github.com/espressif/arduino-esp32/issues/9221
+    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
`@@ -1115,6 +1115,11 @@ void spiTransaction(spi_t * spi, uint32_t clockDiv, uint8_t dataMode, uint8_t bi`
`1115`	`1115`	`spi->dev->ctrl.wr_bit_order = 1;`
`1116`	`1116`	`spi->dev->ctrl.rd_bit_order = 1;`
`1117`	`1117`	`}`
	`1118`	`+#if CONFIG_IDF_TARGET_ESP32C3 \|\| CONFIG_IDF_TARGET_ESP32S3`
	`1119`	`+ // Sync new config with hardware, fixes https://github.com/espressif/arduino-esp32/issues/9221`
	`1120`	`+ spi->dev->cmd.update = 1;`
	`1121`	`+ while (spi->dev->cmd.update);`
	`1122`	`+#endif`
`1118`	`1123`	`}`
`1119`	`1124`
`1120`	`1125`	`void spiSimpleTransaction(spi_t * spi)`