Hi Tim
-----Original Message----- From: Tim Harvey tharvey@gateworks.com Sent: Friday, September 24, 2021 4:32 AM To: Gaurav Jain gaurav.jain@nxp.com; Peng Fan peng.fan@nxp.com Cc: u-boot u-boot@lists.denx.de; Stefano Babic sbabic@denx.de; Fabio Estevam festevam@gmail.com; Simon Glass sjg@chromium.org; Priyanka Jain priyanka.jain@nxp.com; Ye Li ye.li@nxp.com; Horia Geanta horia.geanta@nxp.com; Ji Luo ji.luo@nxp.com; Franck Lenormand franck.lenormand@nxp.com; Silvano Di Ninno silvano.dininno@nxp.com; Sahil Malhotra sahil.malhotra@nxp.com; Pankaj Gupta pankaj.gupta@nxp.com; Varun Sethi V.Sethi@nxp.com; dl-uboot-imx uboot-imx@nxp.com; Shengzhou Liu shengzhou.liu@nxp.com; Mingkai Hu mingkai.hu@nxp.com; Rajesh Bhagat rajesh.bhagat@nxp.com; Meenakshi Aggarwal meenakshi.aggarwal@nxp.com; Wasim Khan wasim.khan@nxp.com; Alison Wang alison.wang@nxp.com; Pramod Kumar pramod.kumar_1@nxp.com; Andy Tang andy.tang@nxp.com; Adrian Alonso adrian.alonso@nxp.com; Vladimir Oltean olteanv@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@nxp.com wrote:
This patchset adds the support for following:
- CAAM Driver model for all i.MX, layerscape, PPC platforms.
- Added crypto node in device tree files.
- CAAM support for blob key encryption key(bkek), random number
generation.
- 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