[EXT] Re: [PATCH v2 00/15] Add CAAM driver model support

[Gaurav Jain]

Hi Tim

> -----Original Message-----
> From: Tim Harvey <tharvey at gateworks.com>
> Sent: Friday, September 24, 2021 4:32 AM
> To: Gaurav Jain <gaurav.jain at nxp.com>; Peng Fan <peng.fan at nxp.com>
> Cc: u-boot <u-boot at lists.denx.de>; Stefano Babic <sbabic at denx.de>; Fabio
> Estevam <festevam at gmail.com>; Simon Glass <sjg at chromium.org>; Priyanka
> Jain <priyanka.jain at nxp.com>; Ye Li <ye.li at nxp.com>; Horia Geanta
> <horia.geanta at nxp.com>; Ji Luo <ji.luo at nxp.com>; Franck Lenormand
> <franck.lenormand at nxp.com>; Silvano Di Ninno <silvano.dininno at nxp.com>;
> Sahil Malhotra <sahil.malhotra at nxp.com>; Pankaj Gupta
> <pankaj.gupta at nxp.com>; Varun Sethi <V.Sethi at nxp.com>; dl-uboot-imx
> <uboot-imx at nxp.com>; Shengzhou Liu <shengzhou.liu at nxp.com>; Mingkai Hu
> <mingkai.hu at nxp.com>; Rajesh Bhagat <rajesh.bhagat at nxp.com>; Meenakshi
> Aggarwal <meenakshi.aggarwal at nxp.com>; Wasim Khan
> <wasim.khan at nxp.com>; Alison Wang <alison.wang at nxp.com>; Pramod
> Kumar <pramod.kumar_1 at nxp.com>; Andy Tang <andy.tang at nxp.com>;
> Adrian Alonso <adrian.alonso at nxp.com>; Vladimir Oltean <olteanv at gmail.com>
> Subject: [EXT] Re: [PATCH v2 00/15] Add CAAM driver model support
> 
> Caution: EXT Email
> 
> On Fri, Sep 3, 2021 at 12:04 AM Gaurav Jain <gaurav.jain at nxp.com> wrote:
> >
> > This patchset adds the support for following:
> > 1) CAAM Driver model for all i.MX, layerscape, PPC platforms.
> > 2) Added crypto node in device tree files.
> > 3) CAAM support for blob key encryption key(bkek), random number
> generation.
> > 4) fixed hwrng performance issue in kernel.
> >
> > i.MX platforms:
> > i.MX6, i.MX7, i.MX7ULP, i.MX8MM/MN/MP/MQ, i.MX8QM/QXP
> >
> > Layerscape platforms:
> > LS1021, LS1012, LS1028, LS1043, LS1046, LS1088, LS2088, LX2160, LX2162
> >
> > Powerpc platforms:
> > P3041, P4080, P5040, P2041, T1024, T1042, T2080, T4240
> >
> > Gaurav Jain (14):
> >   crypto/fsl: Add support for CAAM Job ring driver model
> >   crypto/fsl: Add CAAM support for bkek, random number generation
> >   i.MX8M: crypto: updated device tree for supporting DM in SPL
> >   crypto/fsl: i.MX8M: Enable Job ring driver model in SPL and U-Boot.
> >   i.MX6: Enable Job ring driver model in U-Boot.
> >   i.MX7: Enable Job ring driver model in U-Boot.
> >   i.MX7ULP: Enable Job ring driver model in U-Boot.
> >   i.MX8: Add crypto node in device tree
> >   crypto/fsl: i.MX8: Enable Job ring driver model in SPL and U-Boot.
> >   Layerscape: Add crypto node in device tree
> >   Layerscape: Enable Job ring driver model in U-Boot.
> >   PPC: Add crypto node in device tree
> >   PPC: Enable Job ring driver model in U-Boot
> >   update CAAM MAINTAINER
> >
> > Ye Li (1):
> >   crypto/fsl: Fix kick_trng
> >
> 
> Gaurav,
> 
> I am interested in seeing support added to provide a command to blob/deblob
> data using CAAM. Peng submitted previously a patch
> 'crypto: caam: Add fsl caam driver' [1] which was not merged because it was
> adding a new CAAM driver instead of adding functionality to the existing one.
> Peng's patch added driver support and added a 'caam'
> command to allow 'caam genblob' and 'caam decap'.
> 
> If that is what you are trying to do in this series I see you add a function
> derive_blob_kek but it appears to be not used anywhere. Were you going to also
> add a command or add this to the 'dek_blob' command?
> 

This patch series is modifying the existing CAAM driver to caam driver model.
And is common caam implementation for all i.MX, Layerscape and powerpc.

We already have blob encap/decap support added in cmd/blob.c
After this patch series, We are also planning to add more feature for blob and command for derive_blob_kek later.

Regards
Gaurav Jain

> Best regards,
> 
> Tim
> 
> [1]
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchwo
> rk.ozlabs.org%2Fproject%2Fuboot%2Fpatch%2F20210325093036.3270101-32-
> peng.fan%40oss.nxp.com%2F&data=04%7C01%7Cgaurav.jain%40nxp.com
> %7C7952867f2b414e4a6c8508d97ee62e2b%7C686ea1d3bc2b4c6fa92cd99c5c3
> 01635%7C0%7C0%7C637680349347068156%7CUnknown%7CTWFpbGZsb3d8ey
> JWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C
> 1000&sdata=AhEHy7E192SGEPn8zhnDM0GbVsKqwXL2EvKTDoV1T1Q%3D&
> amp;reserved=0