Re: [PATCH] crypto: p10-aes-gcm - remove duplicate include header
From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2023-03-16 03:58:13
Also in:
linux-crypto, lkml
From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2023-03-16 03:58:13
Also in:
linux-crypto, lkml
Herbert Xu [off-list ref] writes:
On Tue, Mar 14, 2023 at 09:44:52PM +1100, Michael Ellerman wrote:quoted
Hmm. Seems none of them were ever Cc'ed to linuxppc-dev. So this is the first I've seen of them.Sorry, I didn't know that you weren't aware of this change. I will be more careful with these ppc patches in future.
No worries, not your fault. My comments were mostly intended for Danny. Although one question I do have for you is what rules, if any, do we have for deciding whether crypto code goes in drivers/crypto vs arch/*/crypto? On powerpc we have some in arch/powerpc/crypto and some in drivers/crypto/vmx, and I don't really know why it's split that way. It seems like drivers/crypto is where non-CPU crypto accelerator drivers go, but then it also has lots of in-CPU crypto code as well AFAICS. I wonder if we should move drivers/crypto/vmx into arch/powerpc/crypto, so that all the powerpc CRYPTOGAMS code is in one place. That would help to clean up some of the duplication of perl scripts we now have. cheers