Re: [PATCH v2 0/8] crypto: caam - add support for LS1043A SoC

[Horia Ioan Geanta Neag]

On 5/5/2016 6:34 PM, Horia Geantă wrote:
> v2:
> As suggested by Arnd, patch 1 fixes io{read,write}{16,32}be accessors
> to prevent the case when {read,write}{w,l} are overriden by arch-specific
> ones having barriers, while the BE accessors previously mentioned are not
> (thus behaving differently, having no barriers).
> 
> Hi,
> 
> [Patches 2-4 add io{read,write}64[be] accessors (generic, arm64, ppc64),
> such that CAAM's accessors in regs.h are simplified a bit.
> Patch 8 adds crypto node for LS1043A platform.
> Let me know if it's ok to go with these through the cryptodev-2.6 tree.]
> 
Herbert,

Could you apply the patch set, but without patch 8/8 I guess - since DT
maintainers haven't ack-ed it?

I assume it's too late for 4.7, however applying the patches would solve
dependencies b/w on-going caam development.

Thanks,
Horia


> This is a follow-up on the following RFC patch set:
> crypto: caam - Revamp I/O accessors
> https://www.mail-archive.com/linux-crypto@xxxxxxxxxxxxxxx/msg15878.html
> 
> There are platforms such as LS1043A (or LS1012A) where core endianness
> does not match CAAM/SEC endianness (LE vs. BE).
> Add support in caam driver for these cases.
> 
> Current patch set detects device endianness at runtime (as opposed to
> compile-time endianness), in order to support multiplatform kernels.
> Detection of device endianness is not device-tree based.
> Instead, SSTA ("SEC STAtus") register has a property such that
> reading it in any endianness and masking it properly, it's possible
> to deduce device endianness.
> 
> The performance drop due to the runtime detection is < 1.0%.
> (An alternative implementation using function pointers has been tried,
> but lead to a bigger performance drop.)
> 
> Thanks,
> Horia
> 
> Cristian Stoica (1):
>   crypto: caam - fix offset field in hw sg entries
> 
> Horia Geantă (7):
>   asm-generic/io.h: allow barriers in io{read,write}{16,32}be
>   asm-generic/io.h: add io{read,write}64 accessors
>   arm64: add io{read,write}64be accessors
>   powerpc: add io{read,write}64 accessors
>   crypto: caam - handle core endianness != caam endianness
>   crypto: caam - add ARCH_LAYERSCAPE to supported architectures
>   arm64: dts: ls1043a: add crypto node
> 
>  arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts |   4 +
>  arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi    |  43 ++++++
>  arch/arm64/include/asm/io.h                       |   4 +-
>  arch/powerpc/kernel/iomap.c                       |  24 ++++
>  drivers/crypto/caam/Kconfig                       |   6 +-
>  drivers/crypto/caam/caamhash.c                    |   5 +-
>  drivers/crypto/caam/ctrl.c                        | 125 +++++++++++-------
>  drivers/crypto/caam/desc.h                        |   9 +-
>  drivers/crypto/caam/desc_constr.h                 |  44 ++++---
>  drivers/crypto/caam/jr.c                          |  22 ++--
>  drivers/crypto/caam/pdb.h                         | 137 +++++++++++++++-----
>  drivers/crypto/caam/regs.h                        | 151 +++++++++++++++-------
>  drivers/crypto/caam/sg_sw_sec4.h                  |  17 +--
>  include/asm-generic/io.h                          |  71 +++++++++-
>  include/asm-generic/iomap.h                       |   8 ++
>  15 files changed, 494 insertions(+), 176 deletions(-)
>