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....@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 algorithm
> +       smaller 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.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to