diff options
| author | Dario Binacchi <[email protected]> | 2026-02-25 17:16:57 +0100 |
|---|---|---|
| committer | Patrice Chotard <[email protected]> | 2026-04-30 08:01:11 +0200 |
| commit | dd5002856ad7b1caaa5d8f6831ca90a214601baf (patch) | |
| tree | 46b52eeaeda779367d39ef9f115e1170489db772 /drivers | |
| parent | 17c022271d9c6c09ee435aedd155c0b67d128c54 (diff) | |
spi: stm32: add stm32_spi_is_enabled() helper
Add a helper to check the SPE (SPI Enable) bit and replace the
open-coded bitwise check in stm32_spi_stopxfer() with this helper to
improve readability and consistency.
This is also a preparatory step for future driver updates that require
checking the SPI enable state across different code paths.
Signed-off-by: Dario Binacchi <[email protected]>
Reviewed-by: Patrice Chotard <[email protected]>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/spi/stm32_spi.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/drivers/spi/stm32_spi.c b/drivers/spi/stm32_spi.c index adba97915cd..70eb8735e03 100644 --- a/drivers/spi/stm32_spi.c +++ b/drivers/spi/stm32_spi.c @@ -192,6 +192,11 @@ static void stm32_spi_read_rxfifo(struct udevice *bus) log_debug("%d bytes left\n", priv->rx_len); } +static bool stm32_spi_is_enabled(void __iomem *base) +{ + return !!(readl(base + STM32_SPI_CR1) & SPI_CR1_SPE); +} + static int stm32_spi_enable(void __iomem *base) { log_debug("\n"); @@ -245,9 +250,7 @@ static void stm32_spi_stopxfer(struct udevice *dev) dev_dbg(dev, "\n"); - cr1 = readl(base + STM32_SPI_CR1); - - if (!(cr1 & SPI_CR1_SPE)) + if (!stm32_spi_is_enabled(base)) return; /* Wait on EOT or suspend the flow */ |