[riot-notifications] [RIOT-OS/RIOT] sys/random/fortune: change interval ressed to ms (#16594)

Peter Kietzmann notifications at github.com
Thu Aug 19 16:27:23 CEST 2021


Though I doubt a 100ms reseed interval is reasonable for us (not a decision of this PR), moving to a ms resolution is fine. The new variable name `reseed` in the state struct is suboptimal since `reseeds` already exists. Maybe `needs_reseed` or similar + documentation. Otherwise, as the logic doesn't break, I would leave the review up to timer people.

Side note: Moving the order in `auto_init` is sometimes dangerous. Hence, I would suggest to run some "standard examples" on this PR even if they don't relate to fortuna in particular.



-- 
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/16594#issuecomment-901962293
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210819/611f485d/attachment.htm>


More information about the notifications mailing list