Skip to content

Commit ea3386f

Browse files
ndoothebentern
authored andcommitted
fix(tlora-pager): Remove SDCARD_USE_SPI1 so SX1262 and SD card can share SPI bus (#9870)
Problem: - Inserting a µSD card causes RadioLib to hit a critical error and reboot - Device enters a boot loop as the SD card remains inserted Reproduction: - Insert a µSD card and power on - RadioLib reports a critical error on boot - Device reboots, repeating indefinitely Root cause: - On T-Lora Pager, SX1262 and the µSD slot share the same physical SPI bus (same SCK/MOSI/MISO pins, differentiated only by CS) - SDCARD_USE_SPI1 is intended for boards where SD is on a separate SPI bus; it initializes a second ESP32 SPI peripheral (SPI3) for SD - SPI2 is already driving those same pins for LoRa, so both controllers simultaneously drive the same GPIO lines, causing bus contention Fix: - Remove SDCARD_USE_SPI1 so both devices share a single SPI peripheral (SPI2), with CS pins providing device selection as intended - Tested on a custom fork of device-ui; LoRa and SD card map tiles both work correctly with an SD card inserted Signed-off-by: Andrew Yong <me@ndoo.sg>
1 parent e1d238b commit ea3386f

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

variants/esp32s3/tlora-pager/platformio.ini

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ build_flags = ${esp32s3_base.build_flags}
2626
-D T_LORA_PAGER
2727
-D BOARD_HAS_PSRAM
2828
-D HAS_SDCARD
29-
-D SDCARD_USE_SPI1
3029
-D ENABLE_ROTARY_PULLUP
3130
-D ENABLE_BUTTON_PULLUP
3231
-D ROTARY_BUXTRONICS

0 commit comments

Comments
 (0)