Skip to content

Commit db847f5

Browse files
committed
make sure that the interrupt driver is compiled
see discussion in PR #308 experimental sanity checks added, will possibly remove them later.
1 parent 01331b2 commit db847f5

File tree

3 files changed

+8
-4
lines changed

3 files changed

+8
-4
lines changed

lib/NeoESP32RmtHI/include/NeoEsp32RmtHIMethod.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +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
32+
#if !defined(WLED_USE_SHARED_RMT) // WLEDMM don't compile this file on unsupported platforms
3333

3434
// Use the NeoEspRmtSpeed types from the driver-based implementation
3535
#include <NeoPixelBus.h>

lib/NeoESP32RmtHI/src/NeoEsp32RmtHI.S

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,12 @@
55
*
66
*/
77

8+
#if !defined(ESP32) && !defined(ESP8266) // WLEDMM buildenv sanity check (experimental)
9+
#error neither ESP32 nor ESP8266, don't know what to do
10+
#endif
11+
812
#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
13+
#if !defined(WLED_USE_SHARED_RMT) // WLEDMM don't compile this file on unsupported platforms
1014

1115
#include <freertos/xtensa_context.h>
1216
#include "sdkconfig.h"
@@ -261,5 +265,5 @@ ld_include_hli_vectors_rmt:
261265

262266

263267
#endif // CONFIG_BTDM_CTRL_HLI
264-
#endif // WLEDMM
268+
#endif // WLED_USE_SHARED_RMT
265269
#endif // XTensa

lib/NeoESP32RmtHI/src/NeoEsp32RmtHIMethod.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +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
33+
#if !defined(WLED_USE_SHARED_RMT) // WLEDMM don't compile this file on unsupported platforms
3434

3535
#include <algorithm>
3636
#include "esp_idf_version.h"

0 commit comments

Comments
 (0)