Re: [PATCH 01/15] crypto: arm - remove CRYPTO dependency of library functions

Eric Biggers
2025-04-18T03:34:42Z

On Fri, Apr 18, 2025 at 11:12:33AM +0800, Herbert Xu wrote:
> Eric Biggers <ebiggers@kernel.org> wrote:
> >
> > config CRYPTO_CURVE25519_NEON
> > tristate
> > - depends on KERNEL_MODE_NEON
> > + depends on CRYPTO && KERNEL_MODE_NEON
>
> Rather than adding CRYPTO to each symbol, how about grouping all
> the CRYPTO symbols together under one if statement?

I don't think that would be better. The 'if' would be up to 400 lines long, and
it would be easy for people to miss the context when editing the file.

- Eric