stm32/spi: Fail spi_init if pins can't be configured.

dpgeorge · dpgeorge · commit 7016900fbe6d · 2025-07-08T16:25:14.000+10:00
Follows the UART and I2C drivers.

Signed-off-by: Damien George &lt;damien@micropython.org&gt;
diff --git a/ports/stm32/spi.c b/ports/stm32/spi.c
@@ -481,7 +481,10 @@ int spi_init(const spi_t *self, bool enable_nss_pin) {
         if (pins[i] == NULL) {
             continue;
         }
-        mp_hal_pin_config_alt(pins[i], mode, pull, AF_FN_SPI, (self - &spi_obj[0]) + 1);
+        if (!mp_hal_pin_config_alt(pins[i], mode, pull, AF_FN_SPI, (self - &spi_obj[0]) + 1)) {
+            // Pin does not have SPI alternate function.
+            return -MP_EINVAL;
+        }
     }
 
     // init the SPI device

Original file line number	Diff line number	Diff line change
`@@ -481,7 +481,10 @@ int spi_init(const spi_t *self, bool enable_nss_pin) {`
`481`	`481`	`if (pins[i] == NULL) {`
`482`	`482`	`continue;`
`483`	`483`	`}`
`484`		`- mp_hal_pin_config_alt(pins[i], mode, pull, AF_FN_SPI, (self - &spi_obj[0]) + 1);`
	`484`	`+ if (!mp_hal_pin_config_alt(pins[i], mode, pull, AF_FN_SPI, (self - &spi_obj[0]) + 1)) {`
	`485`	`+ // Pin does not have SPI alternate function.`
	`486`	`+ return -MP_EINVAL;`
	`487`	`+ }`
`485`	`488`	`}`
`486`	`489`
`487`	`490`	`// init the SPI device`