btashton commented on a change in pull request #566:
URL:
https://github.com/apache/incubator-nuttx-apps/pull/566#discussion_r563450371
##########
File path: crypto/mbedtls/Kconfig
##########
@@ -0,0 +1,25 @@
+#
+# For a description of the syntax of this configuration file,
+# see the file kconfig-language.txt in the NuttX tools repository.
+#
+
+menuconfig CRYPTO_MBEDTLS
+ bool "Mbed TLS Cryptography Library"
+ default n
+ ---help---
+ Enable support for Mbed TLS.
+
+if CRYPTO_MBEDTLS
+
+config MBEDTLS_VERSION
+ string "MBEDTLS Version"
+ default "2.25.0"
+
+config MBEDTLS_DEFAULT_TASK_STACKSIZE
+ int "Mbed TLS app default stack size"
+ default 8192
Review comment:
The issue is the default stack size is only 2KB but I normally see 3-4KB
on real hardware. I get that the sim is probably much higher so if we kept
this I could conditional this like the DEFAULT_TASK_STACKSIZE. Thoughts?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
