Skip to content

Commit 8d41bc4

Browse files
committed
don't compile RMTHI on unsupprted (old) platforms
1 parent 86473a8 commit 8d41bc4

4 files changed

Lines changed: 7 additions & 1 deletion

File tree

lib/NeoESP32RmtHI/include/NeoEsp32RmtHIMethod.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ License along with NeoPixel. If not, see
2929
#pragma once
3030

3131
#if defined(ARDUINO_ARCH_ESP32)
32+
#if !defined(WLED_USE_SHARED_RMT) && !defined(__riscv) && (ESP_IDF_VERSION_MAJOR >= 4) // WLEDMM don't compile this file on unsupported platforms
3233

3334
// Use the NeoEspRmtSpeed types from the driver-based implementation
3435
#include <NeoPixelBus.h>
@@ -485,3 +486,4 @@ typedef NeoEsp32RmtHI7Ws2805InvertedMethod NeoEsp32RmtHI7Ws2814InvertedMethod;
485486
#endif // !defined(CONFIG_IDF_TARGET_ESP32C3)
486487

487488
#endif
489+
#endif

lib/NeoESP32RmtHI/src/NeoEsp32RmtHI.S

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
*/
77

88
#if defined(__XTENSA__) && defined(ESP32) && !defined(CONFIG_BTDM_CTRL_HLI)
9+
#if !defined(WLED_USE_SHARED_RMT) && !defined(__riscv) && defined(ARDUINO_ARCH_ESP32) && (ESP_IDF_VERSION_MAJOR >= 4) // WLEDMM don't compile this file on unsupported platforms
910

1011
#include <freertos/xtensa_context.h>
1112
#include "sdkconfig.h"
@@ -260,4 +261,5 @@ ld_include_hli_vectors_rmt:
260261

261262

262263
#endif // CONFIG_BTDM_CTRL_HLI
264+
#endif // WLEDMM
263265
#endif // XTensa

lib/NeoESP32RmtHI/src/NeoEsp32RmtHIMethod.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ License along with NeoPixel. If not, see
3030
#include <Arduino.h>
3131

3232
#if defined(ARDUINO_ARCH_ESP32)
33+
#if !defined(WLED_USE_SHARED_RMT) && !defined(__riscv) && (ESP_IDF_VERSION_MAJOR >= 4) // WLEDMM don't compile this file on unsupported platforms
3334

3435
#include <algorithm>
3536
#include "esp_idf_version.h"
@@ -504,4 +505,5 @@ esp_err_t NeoEsp32RmtHiMethodDriver::WaitForTxDone(rmt_channel_t channel, TickTy
504505
return rv;
505506
}
506507

508+
#endif
507509
#endif

wled00/bus_wrapper.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ bool canUseSerial(void); // WLEDMM (wled_serial.cpp) returns true if Serial ca
184184
#endif
185185

186186

187-
// RMT driver selection
187+
// RMT driver selection - only for Xtensa and ESP-IDF 4.x
188188
#if !defined(WLED_USE_SHARED_RMT) && !defined(__riscv) && defined(ARDUINO_ARCH_ESP32) && (ESP_IDF_VERSION_MAJOR >= 4)
189189

190190
#include <NeoEsp32RmtHIMethod.h>

0 commit comments

Comments
 (0)