From 939a10813bc360b1aaa4c7e91bdd7ec784476192 Mon Sep 17 00:00:00 2001 From: Leandro Lanzieri Date: Tue, 10 Mar 2020 15:45:23 +0100 Subject: [PATCH] cpu/esp_common: Check for esp_spi_ram with conditional expansion --- cpu/esp_common/Makefile.include | 26 +++++++++----------------- 1 file changed, 9 insertions(+), 17 deletions(-) diff --git a/cpu/esp_common/Makefile.include b/cpu/esp_common/Makefile.include index 3d9e91020f..25f07ff154 100644 --- a/cpu/esp_common/Makefile.include +++ b/cpu/esp_common/Makefile.include @@ -62,23 +62,15 @@ ifneq (,$(filter lwip,$(USEMODULE))) endif # if SPI RAM is enabled, the qout flash mode has to be used -ifneq (,$(filter esp_spi_ram,$(USEMODULE))) - FLASH_MODE = qout - CFLAGS += -DFLASH_MODE_QOUT=1 -else - ifeq ($(FLASH_MODE), qio) - CFLAGS += -DFLASH_MODE_QIO=1 - endif - ifeq ($(FLASH_MODE), qout) - CFLAGS += -DFLASH_MODE_QOUT=1 - endif - ifeq ($(FLASH_MODE), dio) - CFLAGS += -DFLASH_MODE_DIO=1 - endif - ifeq ($(FLASH_MODE), dout) - CFLAGS += -DFLASH_MODE_DOUT=1 - endif -endif +# NOTE: These checks can be turned into normal conditionals when #9913 is fixed +_FLASH_MODE_PREV := $(FLASH_MODE) +FLASH_MODE = $(if $(filter esp_spi_ram,$(USEMODULE)),qout,$(_FLASH_MODE_PREV)) + +# set CFLAG for the correspondant FLASH_MODE +CFLAGS += $(if $(findstring qout,$(FLASH_MODE)),-DFLASH_MODE_QOUT=1) +CFLAGS += $(if $(findstring qio,$(FLASH_MODE)),-DFLASH_MODE_QIO=1) +CFLAGS += $(if $(findstring dio,$(FLASH_MODE)),-DFLASH_MODE_DIO=1) +CFLAGS += $(if $(findstring dout,$(FLASH_MODE)),-DFLASH_MODE_DOUT=1) BASELIBS += -lhal -lg -lc