New version 1.12 does not compile for T-Beam SX1262

[Makoland]

Hello,

the new version 1.12 does not compile for the T-Beam SX1262 Companion with following message:
Linking .pio\build\Tbeam_SX1262_companion_radio_ble\firmware.elf ./.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: .pio/build/Tbeam_SX1262_companion_radio_ble/firmware.elf section .dram0.bss' will not fit in region dram0_0_seg' ./.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: DRAM segment data does not fit. ./.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: DRAM segment data does not fit. ./.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: region dram0_0_seg' overflowed by 512 bytes
collect2.exe: error: ld returned 1 exit status
`

Can someone give a hint where to look for fixing this?
Where is the size for the BSS segment set, or is this dynamically defined at compile time?

Sorry, my coding skill are a little rusty, especially with platformio.

Thanks,
DG1GAC Markus

[oltaco]

Unfortunately it's dynamic, it looks like something will have to be pruned 😢

[oltaco]

The commit that pushed it over by 504 bytes is 5c6c159

Probably because of wifi.h being included. I don't have time right now to figure out where/how to make some savings but at least it's a start.

[IoTThinks]

@oltaco For ESP32, there is an existing OTA feature with WiFi.h. So another include WiFi.h should not be an issue.

BTW, this is my merged PR for PowerSaving.
https://github.com/meshcore-dev/MeshCore/pull/1266/changes#diff-bf2a85f9de86622aa1706e48181c52038e533cae13e3f52f5cd2a2a063b61dc8

So I use esp_wifi.h instead. WiFi.h has been reviewed and removed in the final version.

[IoTThinks]

@oltaco The merge has been done correctly in main for v1.12.0.
esp_wifi.h is in use.

[oltaco]

@oltaco The merge has been done correctly in main for v1.12.0. esp_wifi.h is in use.

I might have got confused about which library was added, but either way it pushes the size of DRAM over the limit by 512 bytes and causes linking to fail as you can see in #1526 (comment)
region `dram0_0_seg' overflowed by 512 bytes

[IoTThinks]

@oltaco Hmm, T-Beam sx1262 and Heltec v3 should have the same amount of DRAM.
Heltec v3 companion ble can compile fine.

T-Beam boards have extra custom codes so it does not have enough space eventually.
https://github.com/meshcore-dev/MeshCore/blob/main/src/helpers/esp32/TBeamBoard.h
https://github.com/meshcore-dev/MeshCore/blob/main/src/helpers/esp32/TBeamBoard.cpp

We may need to use a flag to skipSleep in board to save space like your PR.

I will try to push a fix tomorrow.

[IoTThinks]

@Makoland Please try the hotfixes.
https://github.com/IoTThinks/EasySkyMesh/tree/main/firmware/MeshCore-v1.12.0-HotFixes

[oltaco]

@oltaco Hmm, T-Beam sx1262 and Heltec v3 should have the same amount of DRAM. Heltec v3 companion ble can compile fine.

T-Beam boards have extra custom codes so it does not have enough space eventually. https://github.com/meshcore-dev/MeshCore/blob/main/src/helpers/esp32/TBeamBoard.h https://github.com/meshcore-dev/MeshCore/blob/main/src/helpers/esp32/TBeamBoard.cpp

We may need to use a flag to skipSleep in board to save space like your PR.

I will try to push a fix tomorrow.

I already pushed a fix, no need to skip sleep. Check #1532 it will save 500 bytes of DRAM for all boards.

[IoTThinks]

@oltaco Great. Let me test the PR. Should be OK.
In the meantime, the hotfixes bin files help our friend to enjoy MeshCore v1.12.0 for now.

[Makoland]

Hi,
I'll try the hotfix this evening and let you know the Renault.
Many thanks and nice greetings,
Markus DG1GAC

[Makoland]

Hi,
I tried the hot fix and it's working fine on y side.
Many thanks for the great support.