[esp32 S2] Allow .bss segment placed in external memory (IDFGH-4320)

[chegewara]

When i am trying to enable Allow .bss segment placed in external memory in menuconfig then code does not build with error:

/home/chegewara/esp/esp-idf/components/esp_system/port/cpu_start.c:395:13: error: '_ext_ram_bss_start' undeclared (first use in this function); did you mean '_rtc_bss_start'?
     memset(&_ext_ram_bss_start, 0, (&_ext_ram_bss_end - &_ext_ram_bss_start) * sizeof(_ext_ram_bss_start));
             ^~~~~~~~~~~~~~~~~~
             _rtc_bss_start
/home/chegewara/esp/esp-idf/components/esp_system/port/cpu_start.c:395:13: note: each undeclared identifier is reported only once for each function it appears in
/home/chegewara/esp/esp-idf/components/esp_system/port/cpu_start.c:395:38: error: '_ext_ram_bss_end' undeclared (first use in this function); did you mean '_rtc_bss_end'?
     memset(&_ext_ram_bss_start, 0, (&_ext_ram_bss_end - &_ext_ram_bss_start) * sizeof(_ext_ram_bss_start));
                                      ^~~~~~~~~~~~~~~~
                                      _rtc_bss_end

IDF: master commit 357a277
board: S2 wrover module
SPI RAM access method - all 3 options

[projectgus]

Hi @chegewara,

Sorry no one got back to you about this earlier. This config option shouldn't actually be visible for ESP32-S2 as this feature has only been implemented for ESP32. Because ESP32-S2 can map PSRAM in different ways, it's not straightforward to map the variable addresses at build time.

Will update this ticket again once we have an ETA for supporting this feature on ESP32-S2.

[sgnes]

@projectgus Any update for this issue, do you have any plan to support esp32-s2?

[igrr]

@sgnes this feature will be supported on S2, the code is in review now. Should be available in the next couple weeks. This issue will be updated when the feature is merged.

[pratikpanchal4]

Any update on timeline for this feature support on esp32-s2. As timeline was mentioned around couple of weeks, Is it still the same?

[Alvin1Zhang]

Thanks for reporting and sorry for slow turnaround, the fix is available b287e01, feel free to reopen.