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: enable puf_sram feature #17671

Merged
merged 1 commit into from
Feb 18, 2022
Merged

cpu/esp32: enable puf_sram feature #17671

merged 1 commit into from
Feb 18, 2022

Conversation

PeterKietzmann
Copy link
Member

Contribution description

This pull request enables the puf_sram feature for ESP32 platforms.

Testing procedure

Execute tests/puf_sram on a ESP32 board as explained in the test README. I have used the esp32-wroom-32 board.

Run the example test script alike python tests/example_test.py -n 100. This produces 100 SRAM based seeds with a power-off time of 1 second. It should produce outputs similar to:

Iteration 0/100
805719428
Iteration 1/100
2758876560
Iteration 2/100
...
Iteration 99/100
3625304372
Number of seeds: 100       
Seed length    : 32 Bit   
Abs. Entropy   : 28.88 Bit   
Rel. Entropy   : 90.24 perc.

Note: The reset button (EN pin) disables a voltage regulator which causes a hard reset. Hence, pressing reset on that board will not trigger the soft reset detection.

Issues/PRs references

#17665

@PeterKietzmann PeterKietzmann added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: security Area: Security-related libraries and subsystems Platform: ESP Platform: This PR/issue effects ESP-based platforms Area: cpu Area: CPU/MCU ports labels Feb 17, 2022
@github-actions github-actions bot added the Area: Kconfig Area: Kconfig integration label Feb 17, 2022
@fjmolinas
Copy link
Contributor

Since there is an esp32 on ci, I think we should have the test run there once first

@fjmolinas fjmolinas added CI: run tests If set, CI server will run tests on hardware for the labeled PR 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 Feb 18, 2022
@fjmolinas fjmolinas merged commit 60f3410 into RIOT-OS:master Feb 18, 2022
@PeterKietzmann PeterKietzmann deleted the pr_puf_sram_esp32 branch April 6, 2022 21:25
@OlegHahm OlegHahm added this to the Release 2022.04 milestone Apr 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

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

@cgundogan cgundogan Awaiting requested review from cgundogan

@gschorcht gschorcht Awaiting requested review from gschorcht gschorcht is a code owner

Assignees
No one assigned
Labels
Area: cpu Area: CPU/MCU ports Area: Kconfig Area: Kconfig integration Area: security Area: Security-related libraries and subsystems CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: run tests If set, CI server will run tests on hardware for the labeled PR Platform: ESP Platform: This PR/issue effects ESP-based platforms
Projects
None yet
Milestone
Release 2022.04
Development

Successfully merging this pull request may close these issues.
4 participants
@PeterKietzmann @fjmolinas @benpicco @OlegHahm