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

ZHIZHIKIN Andrey posted 2 patches 4 years, 5 months ago
Only 0 patches received!
RE: [PATCH v3 0/2] CAAM Driver: re-factor and set proper JR status
Posted by ZHIZHIKIN Andrey 4 years, 5 months ago
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@leica-geosystems.com>
> Sent: Wednesday, December 8, 2021 12:02 AM
> To: linux-kernel@vger.kernel.org
> Cc: robh+dt@kernel.org; shawnguo@kernel.org; michael@walle.cc;
> s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com; linux-
> imx@nxp.com; horia.geanta@nxp.com; pankaj.gupta@nxp.com;
> herbert@gondor.apana.org.au; davem@davemloft.net; l.stach@pengutronix.de;
> qiangqing.zhang@nxp.com; peng.fan@nxp.com; alice.guo@nxp.com; aford173@gmail.com;
> frieder.schrempf@kontron.de; krzk@kernel.org; shengjiu.wang@nxp.com;
> gregkh@linuxfoundation.org; ping.bai@nxp.com; daniel.baluta@nxp.com;
> jun.li@nxp.com; marex@denx.de; thunder.leizhen@huawei.com; martink@posteo.de;
> leonard.crestez@nxp.com; hongxing.zhu@nxp.com; agx@sigxcpu.org;
> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> crypto@vger.kernel.org; op-tee@lists.trustedfirmware.org; Andrey Zhizhikin
> <andrey.zhizhikin@leica-geosystems.com>
> 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
Re: [PATCH v3 0/2] CAAM Driver: re-factor and set proper JR status
Posted by Herbert Xu 4 years, 5 months ago
On Thu, Jan 06, 2022 at 10:56:12AM +0000, ZHIZHIKIN Andrey wrote:
> 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?

Please get the caam driver maintainer to review and ack the
patch series.

Thanks,
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt