RE: [PATCH v3 0/2] CAAM Driver: re-factor and set proper JR status

[ZHIZHIKIN Andrey]

Hello Herbert,

Gentle ping on this V3. I see that in Patchwork this series state is set to "Deferred".

Is there anything missing here to proceed further?

> -----Original Message-----
> From: Andrey Zhizhikin <andrey.zhizhikin@xxxxxxxxxxxxxxxxxxxx>
> Sent: Wednesday, December 8, 2021 12:02 AM
> To: linux-kernel@xxxxxxxxxxxxxxx
> Cc: robh+dt@xxxxxxxxxx; shawnguo@xxxxxxxxxx; michael@xxxxxxxx;
> s.hauer@xxxxxxxxxxxxxx; kernel@xxxxxxxxxxxxxx; festevam@xxxxxxxxx; linux-
> imx@xxxxxxx; horia.geanta@xxxxxxx; pankaj.gupta@xxxxxxx;
> herbert@xxxxxxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx; l.stach@xxxxxxxxxxxxxx;
> qiangqing.zhang@xxxxxxx; peng.fan@xxxxxxx; alice.guo@xxxxxxx; aford173@xxxxxxxxx;
> frieder.schrempf@xxxxxxxxxx; krzk@xxxxxxxxxx; shengjiu.wang@xxxxxxx;
> gregkh@xxxxxxxxxxxxxxxxxxx; ping.bai@xxxxxxx; daniel.baluta@xxxxxxx;
> jun.li@xxxxxxx; marex@xxxxxxx; thunder.leizhen@xxxxxxxxxx; martink@xxxxxxxxx;
> leonard.crestez@xxxxxxx; hongxing.zhu@xxxxxxx; agx@xxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-
> crypto@xxxxxxxxxxxxxxx; op-tee@xxxxxxxxxxxxxxxxxxxxxxxxx; Andrey Zhizhikin
> <andrey.zhizhikin@xxxxxxxxxxxxxxxxxxxx>
> Subject: [PATCH v3 0/2] CAAM Driver: re-factor and set proper JR status
> 
> This V3 series covers points uncovered during the review of the previous
> series, one major point being that register readout should not be used
> for dynamic JR availability check due to its unreliability.
> 
> Instead, JR should have a proper status set in FDT which indicates the
> availability of the ring in NS-World. This status is aligned with what
> BootROM code configures, and can be modified by all actors in the boot
> chain.
> 
> Therefore, patch in V2 series that was handling the dynamic JR
> availability check is dropped in this series and replaced by the patch
> which sets proper DT status for JR nodes.
> 
> Andrey Zhizhikin (2):
>   crypto: caam - convert to use capabilities
>   arm64: dts: imx8m: define proper status for caam jr
> 
>  arch/arm64/boot/dts/freescale/imx8mm.dtsi |   4 +
>  arch/arm64/boot/dts/freescale/imx8mn.dtsi |   4 +
>  arch/arm64/boot/dts/freescale/imx8mp.dtsi |   4 +
>  arch/arm64/boot/dts/freescale/imx8mq.dtsi |   4 +
>  drivers/crypto/caam/caamalg_qi.c          |   2 +-
>  drivers/crypto/caam/ctrl.c                | 115 ++++++++++++++--------
>  drivers/crypto/caam/intern.h              |  20 ++--
>  drivers/crypto/caam/jr.c                  |  19 +++-
>  drivers/crypto/caam/regs.h                |   2 -
>  9 files changed, 122 insertions(+), 52 deletions(-)
> 
> 
> base-commit: 04fe99a8d936d46a310ca61b8b63dc270962bf01
> --
> 2.25.1
> 

-- andrey