Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

cpu/esp32: use ESP-IDF function esp_efuse_mac for CPU ID #18273

Merged
merged 1 commit into from
Jun 28, 2022
Merged

cpu/esp32: use ESP-IDF function esp_efuse_mac for CPU ID #18273

merged 1 commit into from
Jun 28, 2022

Conversation

gschorcht
Copy link
Contributor

Contribution description

This PR is a split-off from PR #17841 and provides the changes to use the MAC address as returned by the ESP-IDF function esp_efuse_mac_get_default as CPU ID.

Testing procedure

  1. Green CI
  2. Compile and test with 
    BOARD=esp32-wroom-32 make -C tests/periph_cpuid flash term

Issues/PRs references

Split-off from PR #17841

@gschorcht gschorcht changed the title cpu/esp32: use ESP-IDF function esp_efuse_mac as CPU ID cpu/esp32: use ESP-IDF function esp_efuse_mac for CPU ID Jun 28, 2022
@github-actions github-actions bot added Area: cpu Area: CPU/MCU ports Platform: ESP Platform: This PR/issue effects ESP-based platforms labels Jun 28, 2022
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jun 28, 2022
@benpicco benpicco merged commit 3b3c6d3 into RIOT-OS:master Jun 28, 2022
@gschorcht gschorcht deleted the cpu/esp32/periph_hal_esp32_cpuid branch June 30, 2022 14:14
@gschorcht gschorcht mentioned this pull request Jul 18, 2022
Closed
@chrysn chrysn added this to the Release 2022.07 milestone Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benpicco benpicco benpicco approved these changes

Assignees
No one assigned
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
Projects
None yet
Milestone
Release 2022.07
Development

Successfully merging this pull request may close these issues.
3 participants
@gschorcht @benpicco @chrysn