[riot-notifications] [RIOT-OS/RIOT] sys/entropy_source: add new module for entropy sources (#14324)

Peter Kietzmann 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:
https://github.com/RIOT-OS/RIOT/pull/14324#discussion_r530942507
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201126/eeaf27ae/attachment.htm>


More information about the notifications mailing list