ETR01SDK-511: Prepare esp32 examples and tests#420
Merged
Conversation
aad200d to
2a30ec0
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR integrates ESP32 platform support into the libtropic monorepo, adding comprehensive examples and functional tests for ESP32-DevKit-V4 and ESP32-S3-DevKitC-1 boards.
Changes:
- Added LT_CAL_LINK_DEPS CMake option to control CAL dependency linking
- Created ESP32 functional test infrastructure with automated build and test scripts
- Added three example applications (hello_world, identify_chip, fw_update) for both ESP32 boards
- Updated documentation to replace "ESP-IDF" references with "ESP32"
Reviewed changes
Copilot reviewed 47 out of 47 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/functional/src/CMakeLists.txt | Added LT_CAL_LINK_DEPS option to conditionally link CAL dependencies |
| tests/functional/esp32/run_test.sh | Script for flashing and monitoring ESP32 tests |
| tests/functional/esp32/download_deps.sh | Downloads and verifies test dependencies (ed25519, micro-ecc) |
| tests/functional/esp32/common.cmake | Common CMake configuration for ESP32 functional tests |
| tests/functional/esp32/*/sdkconfig | ESP-IDF SDK configuration files for each board |
| tests/functional/esp32/*/main.c | Test entry point for ESP32 boards |
| tests/functional/esp32/*/CMakeLists.txt | Board-specific test build configuration |
| examples/esp32/// | Three example applications for both ESP32 boards |
| mkdocs.yml | Updated documentation navigation |
| docs/other/supported_host_platforms/* | Renamed and updated ESP-IDF documentation to ESP32 |
| docs/get_started/tutorials/esp32/index.md | New ESP32 tutorial |
| .github/workflows/esp32_build_examples.yml | CI workflow for building ESP32 examples |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
… they are declared by functional test CMake
…efaults with minimal config
… only if int pin is used
e55c03c to
17756b4
Compare
andreondra
reviewed
Jan 21, 2026
andreondra
approved these changes
Jan 21, 2026
Contributor
andreondra
left a comment
There was a problem hiding this comment.
Just have a look at the tests/functional/src/CMakeLists.txt. Other than that it's OK.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Integrate ESP32 into the monorepo.
Type of Change
Select the type(s) that best describe your change:
Checklist
Before submitting, please confirm that you have completed the following: