Update SPI construction usages

Randall-Scharpf · Randall-Scharpf · commit b4598025b2e5 · 2023-11-07T13:51:57.000-08:00
diff --git a/locale/circuitpython.pot b/locale/circuitpython.pot
@@ -1955,6 +1955,13 @@ msgstr ""
 msgid "Size not supported"
 msgstr ""
 
+#: ports/atmel-samd/common-hal/busio/SPI.c ports/cxd56/common-hal/busio/SPI.c
+#: ports/espressif/common-hal/busio/SPI.c
+#: ports/mimxrt10xx/common-hal/busio/SPI.c ports/nrf/common-hal/busio/SPI.c
+#: ports/raspberrypi/common-hal/busio/SPI.c ports/stm/common-hal/busio/SPI.c
+msgid "Slave mode SPI is not implemented"
+msgstr ""
+
 #: shared-bindings/alarm/SleepMemory.c shared-bindings/memorymap/AddressRange.c
 #: shared-bindings/nvm/ByteArray.c
 msgid "Slice and value different lengths."
diff --git a/ports/atmel-samd/boards/hallowing_m4_express/board.c b/ports/atmel-samd/boards/hallowing_m4_express/board.c
@@ -50,7 +50,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_PA01, &pin_PA00, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_PA01, &pin_PA00, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/atmel-samd/boards/monster_m4sk/board.c b/ports/atmel-samd/boards/monster_m4sk/board.c
@@ -50,7 +50,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_PA13, &pin_PA12, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_PA13, &pin_PA12, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/atmel-samd/boards/openbook_m4/board.c b/ports/atmel-samd/boards/openbook_m4/board.c
@@ -59,7 +59,7 @@ uint8_t refresh_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_PB13, &pin_PB15, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_PB13, &pin_PB15, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/atmel-samd/boards/pewpew_lcd/board.c b/ports/atmel-samd/boards/pewpew_lcd/board.c
@@ -48,7 +48,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_PA23, &pin_PA22, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_PA23, &pin_PA22, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/atmel-samd/boards/pewpew_m4/board.c b/ports/atmel-samd/boards/pewpew_m4/board.c
@@ -99,7 +99,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_PA13, &pin_PA15, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_PA13, &pin_PA15, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/atmel-samd/boards/pybadge/board.c b/ports/atmel-samd/boards/pybadge/board.c
@@ -70,7 +70,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_PB13, &pin_PB15, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_PB13, &pin_PB15, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/atmel-samd/boards/pygamer/board.c b/ports/atmel-samd/boards/pygamer/board.c
@@ -72,7 +72,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_PB13, &pin_PB15, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_PB13, &pin_PB15, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/atmel-samd/boards/seeeduino_wio_terminal/board.c b/ports/atmel-samd/boards/seeeduino_wio_terminal/board.c
@@ -67,7 +67,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_PB20, &pin_PB19, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_PB20, &pin_PB19, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/atmel-samd/common-hal/busio/SPI.h b/ports/atmel-samd/common-hal/busio/SPI.h
@@ -40,6 +40,7 @@ typedef struct {
     uint8_t clock_pin;
     uint8_t MOSI_pin;
     uint8_t MISO_pin;
+    bool slave_mode;
 } busio_spi_obj_t;
 
 #endif // MICROPY_INCLUDED_ATMEL_SAMD_COMMON_HAL_BUSIO_SPI_H
diff --git a/ports/espressif/boards/adafruit_funhouse/board.c b/ports/espressif/boards/adafruit_funhouse/board.c
@@ -52,7 +52,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO36, &pin_GPIO35, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO36, &pin_GPIO35, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/espressif/boards/adafruit_magtag_2.9_grayscale/board.c b/ports/espressif/boards/adafruit_magtag_2.9_grayscale/board.c
@@ -116,7 +116,7 @@ const uint8_t refresh_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO36, &pin_GPIO35, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO36, &pin_GPIO35, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/espressif/boards/espressif_esp32s3_box/board.c b/ports/espressif/boards/espressif_esp32s3_box/board.c
@@ -45,7 +45,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO7, &pin_GPIO6, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO7, &pin_GPIO6, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/espressif/boards/espressif_esp32s3_box_lite/board.c b/ports/espressif/boards/espressif_esp32s3_box_lite/board.c
@@ -46,7 +46,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO7, &pin_GPIO6, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO7, &pin_GPIO6, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/espressif/boards/espressif_esp32s3_usb_otg_n8/board.c b/ports/espressif/boards/espressif_esp32s3_usb_otg_n8/board.c
@@ -73,7 +73,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO6, &pin_GPIO7, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO6, &pin_GPIO7, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/espressif/boards/hardkernel_odroid_go/board.c b/ports/espressif/boards/hardkernel_odroid_go/board.c
@@ -65,7 +65,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO18, &pin_GPIO23, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO18, &pin_GPIO23, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/espressif/boards/hiibot_iots2/board.c b/ports/espressif/boards/hiibot_iots2/board.c
@@ -73,7 +73,9 @@ static void display_init(void) {
         &pin_GPIO34,    // CLK
         &pin_GPIO33,    // MOSI
         NULL,           // MISO not connected
-        false);         // Not half-duplex
+        false,          // Not half-duplex
+        false           // operate SPI bus as master
+        );
 
     common_hal_busio_spi_never_reset(spi);
 
diff --git a/ports/espressif/boards/lilygo_tembed_esp32s3/board.c b/ports/espressif/boards/lilygo_tembed_esp32s3/board.c
@@ -45,7 +45,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO12, &pin_GPIO11, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO12, &pin_GPIO11, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/espressif/boards/lilygo_ttgo_t8_s2_st7789/board.c b/ports/espressif/boards/lilygo_ttgo_t8_s2_st7789/board.c
@@ -73,7 +73,9 @@ static void display_init(void) {
         &pin_GPIO36,    // CLK
         &pin_GPIO35,    // MOSI
         NULL,           // MISO not connected
-        false);         // Not half-duplex
+        false,          // Not half-duplex
+        false           // operate SPI bus as master
+        );
 
     common_hal_busio_spi_never_reset(spi);
 
diff --git a/ports/espressif/boards/lilygo_ttgo_tdisplay_esp32_16m/board.c b/ports/espressif/boards/lilygo_ttgo_tdisplay_esp32_16m/board.c
@@ -53,7 +53,9 @@ static void display_init(void) {
         &pin_GPIO18,    // CLK
         &pin_GPIO19,    // MOSI
         NULL,           // MISO not connected
-        false);         // Not half-duplex
+        false,          // Not half-duplex
+        false           // operate SPI bus as master
+        );
 
     common_hal_busio_spi_never_reset(spi);
 
diff --git a/ports/espressif/boards/lilygo_twatch_2020_v3/board.c b/ports/espressif/boards/lilygo_twatch_2020_v3/board.c
@@ -56,7 +56,8 @@ static void display_init(void) {
         &pin_GPIO18,    // CLK
         &pin_GPIO19,    // MOSI
         NULL,           // MISO not connected
-        false           // Not half-duplex
+        false,          // Not half-duplex
+        false           // operate SPI bus as master
         );
 
     common_hal_busio_spi_never_reset(spi);
diff --git a/ports/espressif/boards/m5stack_atoms3/board.c b/ports/espressif/boards/m5stack_atoms3/board.c
@@ -54,7 +54,7 @@ void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     // busio_spi_obj_t *spi = common_hal_board_create_spi(0);
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO17, &pin_GPIO21, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO17, &pin_GPIO21, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
     bus->base.type = &fourwire_fourwire_type;
 
diff --git a/ports/espressif/boards/m5stack_stick_c/board.c b/ports/espressif/boards/m5stack_stick_c/board.c
@@ -167,7 +167,7 @@ static bool pmic_init(busio_i2c_obj_t *i2c) {
 static bool display_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO13, &pin_GPIO15, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO13, &pin_GPIO15, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/espressif/boards/m5stack_stick_c_plus/board.c b/ports/espressif/boards/m5stack_stick_c_plus/board.c
@@ -167,7 +167,7 @@ static bool pmic_init(busio_i2c_obj_t *i2c) {
 static bool display_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO13, &pin_GPIO15, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO13, &pin_GPIO15, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/espressif/boards/morpheans_morphesp-240/board.c b/ports/espressif/boards/morpheans_morphesp-240/board.c
@@ -149,7 +149,9 @@ void board_init(void) {
         &pin_GPIO12,    // CLK
         &pin_GPIO11,    // MOSI
         NULL,           // MISO not connected
-        false);         // Not half-duplex
+        false,          // Not half-duplex
+        false           // operate SPI bus as master
+        );
 
     common_hal_busio_spi_never_reset(spi);
 
diff --git a/ports/espressif/boards/waveshare_esp32_s2_pico_lcd/board.c b/ports/espressif/boards/waveshare_esp32_s2_pico_lcd/board.c
@@ -72,7 +72,9 @@ static void display_init(void) {
         &pin_GPIO10,    // CLK
         &pin_GPIO11,    // MOSI
         NULL,           // MISO not connected
-        false);         // Not half-duplex
+        false,          // Not half-duplex
+        false           // operate SPI bus as master
+        );
 
     common_hal_busio_spi_never_reset(spi);
 
diff --git a/ports/nrf/boards/clue_nrf52840_express/board.c b/ports/nrf/boards/clue_nrf52840_express/board.c
@@ -49,7 +49,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_P0_14, &pin_P0_15, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_P0_14, &pin_P0_15, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/nrf/boards/espruino_banglejs2/board.c b/ports/nrf/boards/espruino_banglejs2/board.c
@@ -52,7 +52,7 @@ void board_init(void) {
     fb->base.type = &sharpdisplay_framebuffer_type;
 
     busio_spi_obj_t *spi = &fb->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_P0_26, &pin_P0_27, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_P0_26, &pin_P0_27, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     common_hal_sharpdisplay_framebuffer_construct(fb, spi, &pin_P0_05, 500000, 176, 176, true);
diff --git a/ports/nrf/boards/hiibot_bluefi/board.c b/ports/nrf/boards/hiibot_bluefi/board.c
@@ -49,7 +49,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_P0_07, &pin_P1_08, NULL, false); // SCK, MOSI, MISO, not half-duplex
+    common_hal_busio_spi_construct(spi, &pin_P0_07, &pin_P1_08, NULL, false, false); // SCK, MOSI, MISO, not half-duplex
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/nrf/boards/makerdiary_nrf52840_m2_devkit/board.c b/ports/nrf/boards/makerdiary_nrf52840_m2_devkit/board.c
@@ -50,7 +50,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_P0_11, &pin_P0_12, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_P0_11, &pin_P0_12, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/nrf/boards/ohs2020_badge/board.c b/ports/nrf/boards/ohs2020_badge/board.c
@@ -49,7 +49,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_P0_11, &pin_P0_12, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_P0_11, &pin_P0_12, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/raspberrypi/boards/adafruit_macropad_rp2040/board.c b/ports/raspberrypi/boards/adafruit_macropad_rp2040/board.c
@@ -61,7 +61,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO26, &pin_GPIO27, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO26, &pin_GPIO27, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/raspberrypi/boards/hack_club_sprig/board.c b/ports/raspberrypi/boards/hack_club_sprig/board.c
@@ -80,7 +80,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO18, &pin_GPIO19, &pin_GPIO16, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO18, &pin_GPIO19, &pin_GPIO16, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/raspberrypi/boards/lilygo_t_display_rp2040/board.c b/ports/raspberrypi/boards/lilygo_t_display_rp2040/board.c
@@ -74,7 +74,9 @@ static void display_init(void) {
         &pin_GPIO2,     // CLK
         &pin_GPIO3,     // MOSI
         NULL,           // MISO not connected
-        false);         // Not half-duplex
+        false,          // Not half-duplex
+        false           // operate SPI bus as master
+        );
 
     common_hal_busio_spi_never_reset(spi);
 
diff --git a/ports/raspberrypi/boards/pajenicko_picopad/board.c b/ports/raspberrypi/boards/pajenicko_picopad/board.c
@@ -66,7 +66,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO18, &pin_GPIO19, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO18, &pin_GPIO19, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/raspberrypi/boards/pimoroni_badger2040/board.c b/ports/raspberrypi/boards/pimoroni_badger2040/board.c
@@ -276,7 +276,7 @@ void board_init(void) {
     // Set up the SPI object used to control the display
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO18, &pin_GPIO19, &pin_GPIO16, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO18, &pin_GPIO19, &pin_GPIO16, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     // Set up the DisplayIO pin object
diff --git a/ports/raspberrypi/boards/pimoroni_picosystem/board.c b/ports/raspberrypi/boards/pimoroni_picosystem/board.c
@@ -66,7 +66,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO6, &pin_GPIO7, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO6, &pin_GPIO7, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/raspberrypi/boards/ugame22/board.c b/ports/raspberrypi/boards/ugame22/board.c
@@ -66,7 +66,7 @@ uint8_t display_init_sequence[] = {
 void board_init(void) {
     fourwire_fourwire_obj_t *bus = &allocate_display_bus()->fourwire_bus;
     busio_spi_obj_t *spi = &bus->inline_bus;
-    common_hal_busio_spi_construct(spi, &pin_GPIO2, &pin_GPIO3, NULL, false);
+    common_hal_busio_spi_construct(spi, &pin_GPIO2, &pin_GPIO3, NULL, false, false);
     common_hal_busio_spi_never_reset(spi);
 
     bus->base.type = &fourwire_fourwire_type;
diff --git a/ports/raspberrypi/boards/waveshare_rp2040_lcd_0_96/board.c b/ports/raspberrypi/boards/waveshare_rp2040_lcd_0_96/board.c
@@ -70,7 +70,8 @@ static void display_init(void) {
         &pin_GPIO10,    // CLK
         &pin_GPIO11,    // MOSI
         NULL,           // MISO not connected
-        false           // Not half-duplex
+        false,          // Not half-duplex
+        false           // operate SPI bus as master
         );
 
     common_hal_busio_spi_never_reset(spi);
diff --git a/shared-bindings/busio/SPI.c b/shared-bindings/busio/SPI.c
@@ -74,7 +74,7 @@
 //|         MOSI: Optional[microcontroller.Pin] = None,
 //|         MISO: Optional[microcontroller.Pin] = None,
 //|         half_duplex: bool = False,
-//|         slave_mode: bool = False
+//|         slave_mode: bool = False,
 //|     ) -> None:
 //|         """Construct an SPI object on the given pins.
 //|
diff --git a/shared-module/board/__init__.c b/shared-module/board/__init__.c
@@ -130,7 +130,7 @@ mp_obj_t common_hal_board_create_spi(const mp_int_t instance) {
     assert_pin_free(spi_pin[instance].mosi);
     assert_pin_free(spi_pin[instance].miso);
 
-    common_hal_busio_spi_construct(self, spi_pin[instance].clock, spi_pin[instance].mosi, spi_pin[instance].miso, false);
+    common_hal_busio_spi_construct(self, spi_pin[instance].clock, spi_pin[instance].mosi, spi_pin[instance].miso, false, false);
 
     spi_obj_created[instance] = true;
     return &spi_obj[instance];
diff --git a/supervisor/shared/external_flash/spi_flash.c b/supervisor/shared/external_flash/spi_flash.c
@@ -149,7 +149,7 @@ void spi_flash_init(void) {
     common_hal_digitalio_digitalinout_never_reset(&cs_pin);
 
     supervisor_flash_spi_bus.base.type = &busio_spi_type;
-    common_hal_busio_spi_construct(&supervisor_flash_spi_bus, SPI_FLASH_SCK_PIN, SPI_FLASH_MOSI_PIN, SPI_FLASH_MISO_PIN, false);
+    common_hal_busio_spi_construct(&supervisor_flash_spi_bus, SPI_FLASH_SCK_PIN, SPI_FLASH_MOSI_PIN, SPI_FLASH_MISO_PIN, false, false);
     common_hal_busio_spi_never_reset(&supervisor_flash_spi_bus);
     common_hal_busio_spi_configure(&supervisor_flash_spi_bus, SPI_FLASH_MAX_BAUDRATE, 0, 0, 8);
 
diff --git a/supervisor/shared/status_leds.c b/supervisor/shared/status_leds.c
