esp32[s2|s3]: add volatile bitfields compilation flag

This compilation flag was added in the same way as done for esp32 in order to avoid unexpected behaviors when accessing registers using struct bitfields. Signed-off-by: Almir Okato <almir.okato@espressif.com>
almir-okato · Jul 3, 2024 · 7b016b8 · 7b016b8
1 parent ad2d660
commit 7b016b8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/arch/xtensa/src/esp32s2/Make.defs b/arch/xtensa/src/esp32s2/Make.defs
@@ -168,6 +168,10 @@ chip/$(ESP_HAL_3RDPARTY_REPO):
 
 CFLAGS += -Wno-undef -Wno-unused-variable
 
+# Enable strict volatile bitfield access
+
+CFLAGS += -fstrict-volatile-bitfields
+
 CHIP_SERIES = $(patsubst "%",%,$(CONFIG_ESPRESSIF_CHIP_SERIES))
 
 include chip/hal.mk

diff --git a/arch/xtensa/src/esp32s3/Make.defs b/arch/xtensa/src/esp32s3/Make.defs
@@ -230,6 +230,10 @@ chip/$(ESP_HAL_3RDPARTY_REPO):
 CFLAGS += -Wno-undef -Wno-unused-variable
 CFLAGS += ${DEFINE_PREFIX}_RETARGETABLE_LOCKING
 
+# Enable strict volatile bitfield access
+
+CFLAGS += -fstrict-volatile-bitfields
+
 AFLAGS += $(CFLAGS)
 
 # Files that require the HAL recipe