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.
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
esp32-c3: Add support to RNG peripheral #3261
Conversation
/**************************************************************************** | ||
* Name: esp32c3_rng_open | ||
****************************************************************************/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/**************************************************************************** | |
* Name: esp32c3_rng_open | |
****************************************************************************/ |
static pthread_once_t once_control = PTHREAD_ONCE_INIT; | ||
int ret; | ||
|
||
ret = pthread_once(&once_control, esp32c3_rng_initialize); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I think you are right. Probably this check to avoid running xxxx_rng_initialize() twice was added to arch/arm/src/nrf52/nrf52_rng.c because
because someone thought it could be called from devrandom_register() and devurandom_register() during the boot, but the #ifndef is protecting it. I'll remove it!
Co-authored-by: Gustavo Henrique Nihei <gustavo.nihei@espressif.com> Co-authored-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>
@xiaoxiang781216 / @Ouss4 please merge it |
Summary
Add support to RNG peripheral
Impact
Only ESP32-C3
Testing
ESP32C3-Devkitc