tmedicci commented on code in PR #16016: URL: https://github.com/apache/nuttx/pull/16016#discussion_r2033257641
########## arch/xtensa/src/esp32/esp32_sha.c: ########## Review Comment: > but I think that it would be a good idea to have a separate file for SHA specifically. It would make everything cleaner, if, later, support for AES is added, for example. Yes, it makes sense! Thanks for your understanding! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org