Skip to content

Commit c8cee6c

Browse files
committed
-S2/-C3: removing stupid double protection of PSRAM pins
these pins are already forbidden in isPinOK. In MM, trying to allocate them produces a buch of false warnings.
1 parent 0413679 commit c8cee6c

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

wled00/wled.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -565,12 +565,12 @@ void WLED::setup()
565565
pinManager.allocateMultiplePins(pins, sizeof(pins)/sizeof(managed_pin_type), PinOwner::SPI_RAM);
566566
#elif defined(CONFIG_IDF_TARGET_ESP32S2)
567567
// S2: reserve GPIO 26-32 for PSRAM (may fail due to isPinOk() but that will also prevent other allocation)
568-
managed_pin_type pins[] = { {26, true}, {27, true}, {28, true}, {29, true}, {30, true}, {31, true}, {32, true} };
569-
pinManager.allocateMultiplePins(pins, sizeof(pins)/sizeof(managed_pin_type), PinOwner::SPI_RAM);
568+
//managed_pin_type pins[] = { {26, true}, {27, true}, {28, true}, {29, true}, {30, true}, {31, true}, {32, true} };
569+
//pinManager.allocateMultiplePins(pins, sizeof(pins)/sizeof(managed_pin_type), PinOwner::SPI_RAM);
570570
#elif defined(CONFIG_IDF_TARGET_ESP32C3)
571571
// C3: reserve GPIO 12-17 for PSRAM (may fail due to isPinOk() but that will also prevent other allocation)
572-
managed_pin_type pins[] = { {12, true}, {13, true}, {14, true}, {15, true}, {16, true}, {17, true} };
573-
pinManager.allocateMultiplePins(pins, sizeof(pins)/sizeof(managed_pin_type), PinOwner::SPI_RAM);
572+
//managed_pin_type pins[] = { {12, true}, {13, true}, {14, true}, {15, true}, {16, true}, {17, true} };
573+
//pinManager.allocateMultiplePins(pins, sizeof(pins)/sizeof(managed_pin_type), PinOwner::SPI_RAM);
574574
#else
575575
// GPIO16/GPIO17 reserved for SPI RAM
576576
managed_pin_type pins[] = { {16, true}, {17, true} };

0 commit comments

Comments
 (0)