drivers: spi: spi_renesas_ra Don't stop SPI transactions early

jeremydick · kartben · commit f51f86e13ab0 · 2025-09-03T17:04:30.000+02:00
Don't stop polled SPI transactions early if they aren't read and write

Signed-off-by: Jeremy Dick &lt;jdick@pivotint.com&gt;
diff --git a/drivers/spi/spi_renesas_ra.c b/drivers/spi/spi_renesas_ra.c
@@ -188,15 +188,7 @@ static int ra_spi_configure(const struct device *dev, const struct spi_config *c
 
 static bool ra_spi_transfer_ongoing(struct ra_spi_data *data)
 {
-#if defined(CONFIG_SPI_INTERRUPT)
 	return (spi_context_tx_on(&data->ctx) || spi_context_rx_on(&data->ctx));
-#else
-	if (spi_context_total_tx_len(&data->ctx) < spi_context_total_rx_len(&data->ctx)) {
-		return (spi_context_tx_on(&data->ctx) || spi_context_rx_on(&data->ctx));
-	} else {
-		return (spi_context_tx_on(&data->ctx) && spi_context_rx_on(&data->ctx));
-	}
-#endif
 }
 
 #ifndef CONFIG_SPI_INTERRUPT

Original file line number	Diff line number	Diff line change
`@@ -188,15 +188,7 @@ static int ra_spi_configure(const struct device dev, const struct spi_config c`
`188`	`188`
`189`	`189`	`static bool ra_spi_transfer_ongoing(struct ra_spi_data *data)`
`190`	`190`	`{`
`191`		`-#if defined(CONFIG_SPI_INTERRUPT)`
`192`	`191`	`return (spi_context_tx_on(&data->ctx) \|\| spi_context_rx_on(&data->ctx));`
`193`		`-#else`
`194`		`- if (spi_context_total_tx_len(&data->ctx) < spi_context_total_rx_len(&data->ctx)) {`
`195`		`- return (spi_context_tx_on(&data->ctx) \|\| spi_context_rx_on(&data->ctx));`
`196`		`- } else {`
`197`		`- return (spi_context_tx_on(&data->ctx) && spi_context_rx_on(&data->ctx));`
`198`		`- }`
`199`		`-#endif`
`200`	`192`	`}`
`201`	`193`
`202`	`194`	`#ifndef CONFIG_SPI_INTERRUPT`