Messages in this thread | | | Date | Wed, 2 Mar 2022 22:23:23 +0000 | From | Eric Biggers <> | Subject | Re: [PATCH 1/3] crypto: sm4 - create SM4 library based on sm4 generic code |
| |
On Wed, Mar 02, 2022 at 01:26:13AM +0100, Jason A. Donenfeld wrote: > On Wed, Mar 2, 2022 at 1:24 AM Herbert Xu <herbert@gondor.apana.org.au> wrote: > > > > On Tue, Mar 01, 2022 at 11:34:28AM +0100, Jason A. Donenfeld wrote: > > > > lib/crypto/Kconfig | 3 + > > > > lib/crypto/Makefile | 3 + > > > > lib/crypto/sm4.c | 184 +++++++++++++++++++++++++++++++++++++++++++ > > > > > > If this is only used by the crypto API, it does not belong in > > > lib/crypto. > > > > Nope there is no such rule. lib/crypto is fine if you're adding > > code that is shared between crypto and arch/*/crypto. > > The sprawling madness continues then... Noted.
I think it would make more sense for this code to be in crypto/, for the reason that Jason gave.
- Eric
| |