Skip to content

cpu/esp: add feature arch_esp for all Espressif SoCs#14375

Merged
benpicco merged 1 commit intoRIOT-OS:masterfrom
gschorcht:cpu/esp/add_feature_arch_esp
Jun 27, 2020
Merged

cpu/esp: add feature arch_esp for all Espressif SoCs#14375
benpicco merged 1 commit intoRIOT-OS:masterfrom
gschorcht:cpu/esp/add_feature_arch_esp

Conversation

@gschorcht
Copy link
Copy Markdown
Contributor

Contribution description

This PR is a very small change which introduces feature arch_esp for Espressif SoCs.

There are features that can be used with both ESP32 and ESP8266, for example the esp_wifi module. In order to be able to test for any ESP SoC, the feature 'arch_esp' is introduced.With this feature it is possible to define dependencies like

ifneq (,$(filter arch_esp,$(FEATURES_USED)))
  USEMODULE += esp_wifi
endif

Testing procedure

Use commands

make BOARD=esp32-wroom-32 -C tests/thread_basic info-debug-variable-FEATURES_USED | grep arch
make BOARD=esp8266-esp-12x -C tests/thread_basic info-debug-variable-FEATURES_USED | grep arch

and check for the arch_esp feature.

Issues/PRs references

Required for PR #12647.

There are common features that can be used by ESP32 as well as ESP8266, for example the `esp_wifi` module. To be able to test for any ESP SoC, feature `arch_esp` is introduced.
@gschorcht gschorcht requested a review from benpicco June 27, 2020 09:39
@gschorcht gschorcht added Area: cpu Area: CPU/MCU ports Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jun 27, 2020
@benpicco
Copy link
Copy Markdown
Contributor

benpicco commented Jun 27, 2020

There should also already be an automatic feature ‘cpu_esp8266’/‘cpu_esp32’, but given those two share a lot of code a common feature makes sense.

@gschorcht gschorcht added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jun 27, 2020
@benpicco benpicco merged commit 9c99b22 into RIOT-OS:master Jun 27, 2020
@gschorcht gschorcht deleted the cpu/esp/add_feature_arch_esp branch June 27, 2020 14:36
@gschorcht
Copy link
Copy Markdown
Contributor Author

@benpicco Thanks for reviewing and merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: cpu Area: CPU/MCU ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants