Hi Stefano this patch is fixing dcache issue in fsl_rsa.c
Error reported is in drivers/crypto/fsl/fsl_hash.c addr = (addr << 32) | sec_in32(&ctx->sg_tbl[i].addr_lo); reported error is fixed in [v3] crypto/fsl: fsl_hash: Fix crash in flush dcache http://patchwork.ozlabs.org/project/uboot/patch/20220616101009.809953-1-gaurav.j...@nxp.com/ Regards Gaurav Jain > -----Original Message----- > From: Stefano Babic <sba...@denx.de> > Sent: Wednesday, June 29, 2022 12:51 PM > To: Gaurav Jain <gaurav.j...@nxp.com>; Stefano Babic <sba...@denx.de>; > Fabio Estevam <feste...@gmail.com> > Cc: U-Boot-Denx <u-boot@lists.denx.de>; Peng Fan <peng....@nxp.com>; dl- > uboot-imx <uboot-...@nxp.com>; Horia Geanta <horia.gea...@nxp.com>; > Varun Sethi <v.se...@nxp.com>; Ye Li <ye...@nxp.com> > Subject: Re: [EXT] Re: [PATCH v3] crypto/fsl: fsl_rsa: Fix dcache issue in the > driver > > Caution: EXT Email > > Hi Gaurav, > > On 29.06.22 09:06, Gaurav Jain wrote: > > Hi Stefano > > > > This patch is put into state "changes requested". > > But it is a different patch in fsl_rsa. > > There is no change requested with this patch. > > > > Something went wrong - I have applied this patch to push to 2022.07, but in > CI I > get the same errors I found before, that is PowerPC build was > broken: > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsource.d > enx.de%2Fu-boot%2Fcustodians%2Fu-boot-imx%2F- > %2Fjobs%2F457090&data=05%7C01%7Cgaurav.jain%40nxp.com%7C98808 > 23b97ef4669f51108da599feffc%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0 > %7C0%7C637920840703403368%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4 > wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C > %7C%7C&sdata=FGDL9eBnjyUDc423myN4jVoK7QD9VZiitvZao6mz04s%3D > &reserved=0 > > I hadn't time to check this, and I suppose that issue is not yet fixed and I > drop the > patch from MR (Layerscape is broken, too), that is V3 has not fixed CI as in > V2. > Or is there anothe rpatch I had to pick up ? > > Best regards, > Stefano > > > > > Regards > > Gaurav Jain > > > >> -----Original Message----- > >> From: Stefano Babic <sba...@denx.de> > >> Sent: Wednesday, June 15, 2022 6:20 PM > >> To: Gaurav Jain <gaurav.j...@nxp.com>; Fabio Estevam > >> <feste...@gmail.com> > >> Cc: U-Boot-Denx <u-boot@lists.denx.de>; Peng Fan <peng....@nxp.com>; > >> dl- uboot-imx <uboot-...@nxp.com>; Horia Geanta > >> <horia.gea...@nxp.com>; Varun Sethi <v.se...@nxp.com>; Ye Li > >> <ye...@nxp.com> > >> Subject: Re: [EXT] Re: [PATCH v3] crypto/fsl: fsl_rsa: Fix dcache > >> issue in the driver > >> > >> Caution: EXT Email > >> > >> On 15.06.22 14:45, Gaurav Jain wrote: > >>> Hi Fabio > >>> > >>>> -----Original Message----- > >>>> From: Fabio Estevam <feste...@gmail.com> > >>>> Sent: Wednesday, June 15, 2022 6:11 PM > >>>> To: Gaurav Jain <gaurav.j...@nxp.com> > >>>> Cc: U-Boot-Denx <u-boot@lists.denx.de>; Stefano Babic > >>>> <sba...@denx.de>; Peng Fan <peng....@nxp.com>; dl-uboot-imx > >>>> <uboot-...@nxp.com>; Horia Geanta <horia.gea...@nxp.com>; Varun > >>>> Sethi <v.se...@nxp.com>; Ye Li <ye...@nxp.com> > >>>> Subject: [EXT] Re: [PATCH v3] crypto/fsl: fsl_rsa: Fix dcache issue > >>>> in the driver > >>>> > >>>> Caution: EXT Email > >>>> > >>>> Hi Gaurav, > >>>> > >>>> On Wed, Jun 15, 2022 at 9:35 AM Gaurav Jain <gaurav.j...@nxp.com> > wrote: > >>>>> > >>>>> From: Ye Li <ye...@nxp.com> > >>>>> > >>>>> issue: > >>>>> CAAM fails with key error when perform Modular Exponentiation > >>>>> using PKHA Block in CAAM > >>>>> > >>>>> Fix: > >>>>> add flush and invalidate dcache for keys, signature and output > >>>>> decrypted data processed by CAAM. > >>>>> > >>>>> Fixes: 34276478f7 (DM: crypto/fsl - Add Freescale rsa DM driver) > >>>>> Signed-off-by: Ye Li <ye...@nxp.com> > >>>>> Reviewed-by: Gaurav Jain <gaurav.j...@nxp.com> > >>>>> Acked-by: Peng Fan <peng....@nxp.com> > >>>>> --- > >>>> > >>>> Please always explain what changed in the current version of the patch. > >>>> > >>>> I assume you fixed the build error found in CI, correct? > >>> > >>> This is a different patch in fsl_rsa.c I will send a different patch > >>> for the error reported by you. > >> > >> Ouch...I have misunderstood. Well, then I willwait for your patch.. > >> > >> Regards, > >> Stefano > >> > >>> > >>> Regards > >>> Gaurav Jain > > > -- > ================================================================= > ==== > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, 82194 Groebenzell, Germany > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de > ================================================================= > ====
