Skip to content

cpu/esp32: enable puf_sram feature#17671

Merged
fjmolinas merged 1 commit intoRIOT-OS:masterfrom
PeterKietzmann:pr_puf_sram_esp32
Feb 18, 2022
Merged

cpu/esp32: enable puf_sram feature#17671
fjmolinas merged 1 commit intoRIOT-OS:masterfrom
PeterKietzmann:pr_puf_sram_esp32

Conversation

@PeterKietzmann
Copy link
Copy Markdown
Member

@PeterKietzmann PeterKietzmann commented Feb 17, 2022

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
Copy Markdown
Contributor

fjmolinas commented Feb 18, 2022

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 enabled auto-merge February 18, 2022 09:24
@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