[riot-notifications] [RIOT-OS/RIOT] sys/entropy_source: add new module for entropy sources (#14324)
notifications at github.com
Thu Nov 26 11:53:51 CET 2020
@PeterKietzmann commented on this pull request.
> + * Although APIs between multiple entropy sources might be similar, every source has to define its
+ * own API to be open for specific requirements. Entropy sources are crucial, thus, separate APIs should
+ * make the choice for one or the other explicit. Furthermore, multiple sources should be able to run in parallel
+ * and separate APIs simplify access (source management).
@maribu, I have marked the APIs as `@internal` as you suggested, however, looking at it know I'd argue to remove it again.
- Why hiding all information about this module? Even if an application developer would not use related APIs, it is worth to read the full documentation. Also for configuration purposes.
- System developers might need documentation anyway.
- The documentation has a warning and an experimental tag. The text describes how these APIs are (not) to be used.
- The follow-up that ports an entropy module might make it even clearer which API to use (if at all, since application developers would probably not deal with entropy but a PRNG).
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications