Skip to content

Commit

Permalink
cpu/esp32: switch from O2 to Os
Browse files Browse the repository at this point in the history
In RIOT-OS#12955 optimization was switche to O2 because with the '-Os'
option, the ESP32 hangs sporadically in 'tests/bench*' if
interrupts where disabled too early by benchmark tests.

Since hasn't been reproduced since and in RIOT-OS#13196 O2 was causing
un-explained hardfaults, since the formentioned issue could not
be reproduced we switch back to Os.
  • Loading branch information
fjmolinas committed May 5, 2020
1 parent 013af64 commit 333fb75
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions cpu/esp32/Makefile.include
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,7 @@ endif

ESP_SDK_DIR = $(ESP32_SDK_DIR)

# With the '-Os' option, the ESP32 hangs sporadically in 'tests/bench*' if
# interrupts are disabled too early by benchmark tests.
CFLAGS_OPT ?= -O2
CFLAGS_OPT ?= -Os

# ESP32 specific flashing options
FLASH_CHIP = esp32
Expand Down

0 comments on commit 333fb75

Please sign in to comment.