Skip to content

Commit

Permalink
feat(wifi_remote): Add build test for current IDF examples
Browse files Browse the repository at this point in the history
Building also standard IDF examples that use esp-wifi-remote (building
for wifiless targets ESP32P4, ESP32H2 and one wifi target ESP32s3)
  • Loading branch information
david-cermak committed Sep 26, 2024
1 parent bde9720 commit 50c113e
Show file tree
Hide file tree
Showing 4 changed files with 48 additions and 2 deletions.
35 changes: 34 additions & 1 deletion .github/workflows/wifi_remote__build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
run: |
. ${IDF_PATH}/export.sh
pip install idf-component-manager idf-build-apps --upgrade
python ./scripts/generate_slave_configs.py ./components/esp_wifi_remote/${{matrix.test.path}}
python ./components/esp_wifi_remote/scripts/generate_slave_configs.py ./components/esp_wifi_remote/${{matrix.test.path}}
python ./ci/build_apps.py ./components/esp_wifi_remote/${{matrix.test.path}} -vv --preserve-all
build_wifi_remote_example:
Expand Down Expand Up @@ -77,3 +77,36 @@ jobs:
. ${IDF_PATH}/export.sh
pip install idf-component-manager idf-build-apps --upgrade
python ./ci/build_apps.py ./components/esp_wifi_remote/${{matrix.example.path}} -vv --preserve-all
build_idf_examples_with_wifi_remote:
if: contains(github.event.pull_request.labels.*.name, 'wifi_remote') || github.event_name == 'push'
name: Build IDF examples with WiFi Remote
strategy:
matrix:
idf_ver: ["latest", "release-v5.3"]
idf_target: ["esp32p4", "esp32h2", "esp32s3"]
test: [ { app: idf_mqtt_example, path: "examples/protocols/mqtt/tcp" }]
runs-on: ubuntu-20.04
container: espressif/idf:${{ matrix.idf_ver }}
steps:
- name: Checkout esp-protocols
uses: actions/checkout@v3
- name: ccache
uses: hendrikmuhs/[email protected]
with:
key: ${{ matrix.idf_ver }}
- name: Build ${{ matrix.test.app }} with IDF-${{ matrix.idf_ver }} for ${{ matrix.idf_target }}
shell: bash
run: |
. ${IDF_PATH}/export.sh
pip install idf-component-manager idf-build-apps --upgrade
export OVERRIDE_PATH=`pwd`/components/esp_wifi_remote
echo ${OVERRIDE_PATH}
sed -i '/espressif\/esp_wifi_remote:/a \ \ \ \ override_path: "${OVERRIDE_PATH}"' ${IDF_PATH}/${{matrix.test.path}}/main/idf_component.yml
cat ${IDF_PATH}/${{matrix.test.path}}/main/idf_component.yml
export PEDANTIC_FLAGS="-DIDF_CI_BUILD -Werror -Werror=deprecated-declarations -Werror=unused-variable -Werror=unused-but-set-variable -Werror=unused-function"
export EXTRA_CFLAGS="${PEDANTIC_FLAGS} -Wstrict-prototypes"
export EXTRA_CXXFLAGS="${PEDANTIC_FLAGS}"
cd ${IDF_PATH}/${{matrix.test.path}}
idf-build-apps find --config sdkconfig.ci* -vv --target ${{ matrix.idf_target }}
idf-build-apps build --config sdkconfig.ci* -vv --target ${{ matrix.idf_target }}
Empty file.
12 changes: 12 additions & 0 deletions components/esp_wifi_remote/idf_v5.4/Kconfig.soc_wifi_caps.in
Original file line number Diff line number Diff line change
Expand Up @@ -222,6 +222,10 @@ if SLAVE_IDF_TARGET_ESP32C6
bool
default y

config SLAVE_SOC_WIFI_MAC_VERSION_NUM
int
default 2

endif # ESP32C6

if SLAVE_IDF_TARGET_ESP32C5
Expand Down Expand Up @@ -266,6 +270,10 @@ if SLAVE_IDF_TARGET_ESP32C5
bool
default y

config SLAVE_SOC_WIFI_MAC_VERSION_NUM
int
default 3

endif # ESP32C5

if SLAVE_IDF_TARGET_ESP32C61
Expand Down Expand Up @@ -306,4 +314,8 @@ if SLAVE_IDF_TARGET_ESP32C61
bool
default y

config SLAVE_SOC_WIFI_MAC_VERSION_NUM
int
default 3

endif # ESP32C61
3 changes: 2 additions & 1 deletion components/esp_wifi_remote/scripts/generate_slave_configs.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,8 @@
output_directory = sys.argv[1]

# Input Kconfig file
kconfig_file = f"idf_v{os.getenv('ESP_IDF_VERSION')}/Kconfig.slave_select.in"
component_path = os.path.normpath(os.path.join(os.path.realpath(__file__),'..', '..'))
kconfig_file = os.path.join(component_path, f"idf_v{os.getenv('ESP_IDF_VERSION')}", 'Kconfig.slave_select.in')

# Output file prefix
output_prefix = 'sdkconfig.ci.'
Expand Down

0 comments on commit 50c113e

Please sign in to comment.