ESP32S2 does not compile for ESP32 Arduino core v3

[devarishi7]

It seems that the default method for the ESP32S2 does not compile for core version 3.x.x and i suspect that is because the RMT driver implementation has been completely re-written for this core version. There is a warning that shows up when compiling for other ESP32 boards that the legacy driver is deprecated, but i guess deprecated doesn't quite cover what has happened to the core. I saw an issue stating something similar (or the same) for the S3. I will have a look at it when i have some time, i just happen to have the S2 i used for the 3 step encoding test and thought i'd check my DMX library and Neopixel as an output for the channels is actually the simplest way to verify all channels come in without error.

So since i guess it is about using the RMT method, which makes me wonder if there is an actual I2S method available and how to set it as the default (because if i am correct the default is RMT for the S2, S3 & C3)

// I2s Bus 1 method is the default method for Esp32
// Esp32 S2 & C3 & S3 will use RMT as the default always

Anyway i will have a lok at the RMT method, i went into the differences between core v2 & v3 for the RMT Rx, it shouldn't be that hard for me to work out what to change and how to keep it backwards compatible. (although that does really tend to mess up the overview of the code)

[devarishi7]

Ok, i did some digging. The RMT driver is deprecated but that is not the issue. For the ESP32S2 there is somewhere in the I2S implementation between v2 & v3, also it seems to me that excluding the S3, C3 from I2S is consistent, but the C6 & H2 is not consistently excluded (they are in I2S.c but not in I2S.h and in the I2S methods.h ) On top of that there is something in the bitbang method that shows up as an error. (i just blocked it for all boards other than the normal ESP32) and that is the only way i can compile for the S2 & the S3 and C3 and C6 as well as the H2 (i think) Somehow for the S2 and for the bitbang method i think it's something very small that needs to be corrected.

In file included from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32RmtMethod.h:50,
                 from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/NeoMethods.h:62,
                 from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/NeoPixelBus280.h:43,
                 from C:\Data\Arduino\Sketches\ESP32\Dmx_to_Neopixel_test\Dmx_to_Neopixel_test.ino:2:
C:\Data\Arduino\Sketches\hardware\esp32v336\esp32\tools\esp32-arduino-libs\esp32s2/include/driver/deprecated/driver/rmt.h:18:2: warning: #warning "The legacy RMT driver is deprecated, please use driver/rmt_tx.h and/or driver/rmt_rx.h" [-Wcpp]
   18 | #warning "The legacy RMT driver is deprecated, please use driver/rmt_tx.h and/or driver/rmt_rx.h"
      |  ^~~~~~~
In file included from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src\internal\methods\Esp32_i2s.c:53:
C:\Data\Arduino\Sketches\hardware\esp32v336\esp32\tools\esp32-arduino-libs\esp32s2/include/driver/deprecated/driver/i2s.h:27:2: warning: #warning "This set of I2S APIs has been deprecated, please include 'driver/i2s_std.h', 'driver/i2s_pdm.h' or 'driver/i2s_tdm.h' instead. if you want to keep using the old APIs and ignore this warning, you can enable 'Suppress legacy driver deprecated warning' option under 'I2S Configuration' menu in Kconfig" [-Wcpp]
   27 | #warning "This set of I2S APIs has been deprecated, \
      |  ^~~~~~~
In file included from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src\internal\methods\Esp32_i2s.c:61:
C:\Data\Arduino\Sketches\hardware\esp32v336\esp32\tools\esp32-arduino-libs\esp32s2/include/driver/deprecated/driver/dac.h:16:2: warning: #warning "The legacy DAC driver is deprecated, please use `driver/dac_oneshot.h`, `driver/dac_cosine.h` or `driver/dac_continuous.h` instead" [-Wcpp]
   16 | #warning "The legacy DAC driver is deprecated, please use `driver/dac_oneshot.h`, `driver/dac_cosine.h` or `driver/dac_continuous.h` instead"
      |  ^~~~~~~
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src\internal\methods\Esp32_i2s.c: In function 'i2sInit':
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src\internal\methods\Esp32_i2s.c:519:14: error: 'volatile union <anonymous>' has no member named 'tx_pcm_bypass'
  519 |         conf1.tx_pcm_bypass = 1;
      |              ^
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src\internal\methods\Esp32_i2s.c: At top level:
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src\internal\methods\Esp32_i2s.c:683:1: warning: ignoring attribute 'section (".iram1.1")' because it conflicts with previous 'section (".iram1.0")' [-Wattributes]
  683 | {
      | ^
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src\internal\methods\Esp32_i2s.c:137:16: note: previous declaration here
  137 | void IRAM_ATTR i2sDmaISR(void* arg);
      |                ^~~~~~~~~
exit status 1
Error compiling for board ESP32S2 Dev Module.

(this is from version 2.8.0 but somehow i tried 2.8.4 and that didn't compile either.)

and the bitbang, though almost never used, fails to compile for the S3 (and maybe others with)

In file included from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32RmtMethod.h:50,
                 from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/NeoMethods.h:62,
                 from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/NeoPixelBus280.h:43,
                 from C:\Data\Arduino\Sketches\ESP32\Dmx_to_Neopixel_test\Dmx_to_Neopixel_test.ino:2:
C:\Data\Arduino\Sketches\hardware\esp32v336\esp32\tools\esp32-arduino-libs\esp32s3/include/driver/deprecated/driver/rmt.h:18:2: warning: #warning "The legacy RMT driver is deprecated, please use driver/rmt_tx.h and/or driver/rmt_rx.h" [-Wcpp]
   18 | #warning "The legacy RMT driver is deprecated, please use driver/rmt_tx.h and/or driver/rmt_rx.h"
      |  ^~~~~~~
In file included from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:33,
                 from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/NeoMethods.h:65:
C:\Data\Arduino\Sketches\hardware\esp32v336\esp32\tools\esp32-arduino-libs\esp32s3/include/driver/deprecated/driver/periph_ctrl.h:7:2: warning: #warning driver/periph_ctrl.h header is no longer used, and will be removed in future versions. [-Wcpp]
    7 | #warning driver/periph_ctrl.h header is no longer used, and will be removed in future versions.
      |  ^~~~~~~
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h: In member function 'void NeoEspLcdMonoBuffContext<T_MUXMAP>::Construct(uint16_t)':
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:436:29: warning: 'esp_err_t gdma_new_channel(const gdma_channel_alloc_config_t*, gdma_channel_t**)' is deprecated: please use gdma_new_ahb_channel or gdma_new_axi_channel respectively [-Wdeprecated-declarations]
  436 |             gdma_new_channel(&dma_chan_config, &_dmaChannel);
      |             ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:34:
C:\Data\Arduino\Sketches\hardware\esp32v336\esp32\tools\esp32-arduino-libs\esp32s3/include/esp_hw_support/dma/include/esp_private/gdma.h:481:11: note: declared here
  481 | esp_err_t gdma_new_channel(const gdma_channel_alloc_config_t *config, gdma_channel_handle_t *ret_chan)
      |           ^~~~~~~~~~~~~~~~
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h: In member function 'void NeoEsp32LcdMuxBus<T_BUSCONTEXT>::Initialize(uint8_t, uint16_t, bool)':
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:545:9: error: there are no arguments to 'gpio_hal_iomux_func_sel' that depend on a template parameter, so a declaration of 'gpio_hal_iomux_func_sel' must be available [-fpermissive]
  545 |         gpio_hal_iomux_func_sel(GPIO_PIN_MUX_REG[pin], PIN_FUNC_GPIO);
      |         ^~~~~~~~~~~~~~~~~~~~~~~
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:545:9: note: (if you use '-fpermissive', G++ will accept your code, but allowing the use of an undeclared name is deprecated)
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h: At global scope:
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:705:32: error: 'NeoBitsSpeeddWs2812x' was not declared in this scope; did you mean 'NeoBitsSpeedWs2812x'?
  705 | typedef NeoEsp32LcdXMethodBase<NeoBitsSpeeddWs2812x, NeoEsp32LcdMux16Bus, NeoBitsNotInverted> NeoEsp32LcdX16Ws2812xMethod;
      |                                ^~~~~~~~~~~~~~~~~~~~
      |                                NeoBitsSpeedWs2812x
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:705:93: error: template argument 1 is invalid
  705 | typedef NeoEsp32LcdXMethodBase<NeoBitsSpeeddWs2812x, NeoEsp32LcdMux16Bus, NeoBitsNotInverted> NeoEsp32LcdX16Ws2812xMethod;
      |                                                                                             ^
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:725:32: error: 'NeoBitsSpeeddWs2812x' was not declared in this scope; did you mean 'NeoBitsSpeedWs2812x'?
  725 | typedef NeoEsp32LcdXMethodBase<NeoBitsSpeeddWs2812x, NeoEsp32LcdMux8Bus, NeoBitsInverted>    NeoEsp32LcdX8Ws2812xInvertedMethod;
      |                                ^~~~~~~~~~~~~~~~~~~~
      |                                NeoBitsSpeedWs2812x
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:725:89: error: template argument 1 is invalid
  725 | typedef NeoEsp32LcdXMethodBase<NeoBitsSpeeddWs2812x, NeoEsp32LcdMux8Bus, NeoBitsInverted>    NeoEsp32LcdX8Ws2812xInvertedMethod;
      |                                                                                         ^
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:743:32: error: 'NeoBitsSpeeddWs2812x' was not declared in this scope; did you mean 'NeoBitsSpeedWs2812x'?
  743 | typedef NeoEsp32LcdXMethodBase<NeoBitsSpeeddWs2812x, NeoEsp32LcdMux16Bus, NeoBitsInverted> NeoEsp32LcdX16Ws2812xInvertedMethod;
      |                                ^~~~~~~~~~~~~~~~~~~~
      |                                NeoBitsSpeedWs2812x
C:\Data\Arduino\Sketches\libraries\NeoPixelBus280\src/internal/methods/NeoEsp32LcdXMethod.h:743:90: error: template argument 1 is invalid
  743 | typedef NeoEsp32LcdXMethodBase<NeoBitsSpeeddWs2812x, NeoEsp32LcdMux16Bus, NeoBitsInverted> NeoEsp32LcdX16Ws2812xInvertedMethod;
      |                                                                                          ^
exit status 1
Error compiling for board ESP32S3 Dev Module.

but does compile for the S2. And everything works just fine for core version 2.

The peripheral manager has been completely re-written for v3 and as far as i can tell has a whole different philosophy.
Anyway, sometimes improvement causes a lot of work down the line.

[devarishi7]

As for the different philosophy of the peripheral manager i would like to add, that instead of linking the detach of the bus on the peripheral (eg RMT channelX etc) the identification is now done through the pin number. It is no longer possible to attach 2 peripherals to the same pin (unless pulling a special trick) and it is no longer required to keep track of what is attached to which pin, but rather the peripheral manager takes care of that. Attaching a peripheral to a pin will automatically detach and de-init whatever was attached to the pin. In effect that means that there is no longer a need to keep track of which RMT channel is used and attached. So the whole thing should be simpler... But of course, before it was done, and that it isn't anymore.

One more thing about the RMT channels in particular. The ESP32 specs state that there are 8 RMT channels, but when i looked closer, it turned out that 4 of those are Rx and 4 of those are Tx. This showed up as well when someone tried to use 5 Tx channels on a Devkit (using FastLED 5 strips) Is this also your understanding.

[aaronw2]

The fix is to apply the following in NeoEsp32LcdXMethod.h:

#if ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 0, 0)
esp_rom_gpio_pad_select_gpio(pin);
#else
gpio_hal_iomux_func_sel(GPIO_PIN_MUX_REG[pin], PIN_FUNC_GPIO);
#endif

[devarishi7]

That seems to do the trick ! Great !!