Skip to content

Commit

Permalink
Don't use ROM time functions, they're 32-bit only (#2835)
Browse files Browse the repository at this point in the history
Fixes #2834. Checked HostTests with IDF 4.4 and 5.2 on esp32-wroom32.

Note that using ROM routines are an optimisation for pre SDK 5 so not worth the effort using them.
  • Loading branch information
mikee47 authored Jun 25, 2024
1 parent 257a079 commit 7806972
Showing 1 changed file with 2 additions and 10 deletions.
12 changes: 2 additions & 10 deletions Sming/Arch/Esp32/Components/esp32/component.mk
Original file line number Diff line number Diff line change
Expand Up @@ -350,14 +350,12 @@ LDFLAGS_esp32 := \
$(call LinkerScript,rom.newlib-funcs) \
$(call LinkerScript,rom.newlib-data) \
$(call LinkerScript,rom.syscalls) \
$(call LinkerScript,rom.newlib-time) \
$(call LinkerScript,rom.eco3)

LDFLAGS_esp32s2 := \
$(call LinkerScript,rom.newlib-funcs) \
$(call LinkerScript,rom.newlib-data) \
$(call LinkerScript,rom.spiflash) \
$(call LinkerScript,rom.newlib-time) \
$(call LinkerScript,rom.spiflash)

LDFLAGS_esp32c3 := \
$(call LinkerScript,rom.newlib) \
Expand All @@ -366,20 +364,14 @@ LDFLAGS_esp32c3 := \

LDFLAGS_esp32s3 := \
$(call LinkerScript,rom.newlib) \
$(call LinkerScript,rom.version) \
$(call LinkerScript,rom.newlib-time)
$(call LinkerScript,rom.version)

LDFLAGS_esp32c2 := \
$(call LinkerScript,rom.newlib) \
$(call LinkerScript,rom.version) \
$(call LinkerScript,rom.heap) \
$(call LinkerScript,rom.mbedtls)

ifneq (v5.2,$(IDF_VERSION))
LDFLAGS_esp32c2 += \
$(call LinkerScript,rom.newlib-time)
endif

SDK_WRAP_SYMBOLS :=
SDK_UNDEF_SYMBOLS :=

Expand Down

0 comments on commit 7806972

Please sign in to comment.