update per review

.github/workflows/build.yml

@@ -189,12 +189,14 @@ jobs:
       matrix:
         board:
         # Alphabetical order
+        # ESP32-S2
         - 'adafruit_feather_esp32s2'
         - 'adafruit_magtag_29gray'
         - 'adafruit_metro_esp32s2'
-        # - 'espressif_addax_1' # temporarily remove the board from test while espressif MR is not merged
         - 'espressif_kaluga_1'
         - 'espressif_saola_1'
+        # ESP32-S3
+        # - 'espressif_addax_1' # temporarily remove the board from test while espressif MR is not merged
 
     steps:
     - name: Setup Python

examples/device/board_test/CMakeLists.txt

@@ -7,7 +7,7 @@ set(TOP "../../..")
 get_filename_component(TOP "${TOP}" REALPATH)
 
 # Check for -DFAMILY=
-if(FAMILY MATCHES "^(esp32s[2-3])*")
+if(FAMILY MATCHES "^esp32s[2-3]")
   cmake_minimum_required(VERSION 3.5)
 
   include(${TOP}/hw/bsp/${FAMILY}/family.cmake)

examples/device/cdc_msc_freertos/CMakeLists.txt

@@ -9,7 +9,7 @@ set(TOP "../../..")
 get_filename_component(TOP "${TOP}" REALPATH)
 
 # Check for -DFAMILY=
-if(FAMILY MATCHES "^(esp32s[2-3])*")
+if(FAMILY MATCHES "^esp32s[2-3]")
   include(${TOP}/hw/bsp/${FAMILY}/family.cmake)
   project(${PROJECT})
 

examples/device/hid_composite_freertos/CMakeLists.txt

@@ -9,7 +9,7 @@ set(TOP "../../..")
 get_filename_component(TOP "${TOP}" REALPATH)
 
 # Check for -DFAMILY=
-if(FAMILY MATCHES "^(esp32s[2-3])*")
+if(FAMILY MATCHES "^esp32s[2-3]")
   include(${TOP}/hw/bsp/${FAMILY}/family.cmake)
   project(${PROJECT})
 else()

examples/make.mk

@@ -27,9 +27,6 @@ endif
 # Board within family
 ifeq ($(BOARD_PATH),)
   BOARD_PATH := $(subst $(TOP)/,,$(wildcard $(TOP)/hw/bsp/*/boards/$(BOARD)))
-ifneq ($(wildcard $(TOP)/hw/bsp/*/boards/$(BOARD)/board.mk),)
-    include $(TOP)/hw/bsp/*/boards/$(BOARD)/board.mk
-endif
   FAMILY := $(word 3, $(subst /, ,$(BOARD_PATH)))
   FAMILY_PATH = hw/bsp/$(FAMILY)
 endif