On Fri, Aug 16, 2024 at 02:43:57PM -0700, Raymond Mao wrote:
Smaller implementation for SHA256 and SHA512 helps to reduce the ROM footprint though it has a certain impact on performance. As a trade-off, enable it as a default config when MbedTLS is enabled can reduce the target size significantly with acceptable performace loss.
Signed-off-by: Raymond Mao raymond.mao@linaro.org
Changes in v6
- Initial patch
lib/mbedtls/Kconfig | 24 ++++++++++++++++++++++++ lib/mbedtls/mbedtls_def_config.h | 6 ++++++ 2 files changed, 30 insertions(+)
diff --git a/lib/mbedtls/Kconfig b/lib/mbedtls/Kconfig index 12f8c965f5a..0e22edf1b6c 100644 --- a/lib/mbedtls/Kconfig +++ b/lib/mbedtls/Kconfig @@ -151,18 +151,42 @@ config SHA1_MBEDTLS config SHA256_MBEDTLS bool "Enable SHA256 support with MbedTLS crypto library" depends on MBEDTLS_LIB_CRYPTO && SHA256
- select SHA256_SMALLER help This option enables support of hashing using SHA256 algorithm with MbedTLS crypto library.
+if SHA256_MBEDTLS
+config SHA256_SMALLER
- bool "Enable SHA256 smaller implementation with MbedTLS crypto library"
- depends on SHA256_MBEDTLS
- help
This option enables support of hashing using SHA256 algorithmsmaller implementation with MbedTLS crypto library.+endif
For each of these, they shouldn't be select'd, they just need to depends on the right option (SHA256_MBEDTLS, etc) and be default y.