Hello Andrey > -----Original Message----- > From: ZHIZHIKIN Andrey <andrey.zhizhi...@leica-geosystems.com> > Sent: Monday, March 7, 2022 1:08 AM > To: Stefano Babic <sba...@denx.de>; Gaurav Jain <gaurav.j...@nxp.com>; u- > b...@lists.denx.de; Marek Vasut <ma...@denx.de> > Cc: Fabio Estevam <feste...@gmail.com>; Peng Fan <peng....@nxp.com>; > Simon Glass <s...@chromium.org>; Priyanka Jain <priyanka.j...@nxp.com>; Ye > Li <ye...@nxp.com>; Horia Geanta <horia.gea...@nxp.com>; Ji Luo > <ji....@nxp.com>; Franck Lenormand <franck.lenorm...@nxp.com>; Silvano Di > Ninno <silvano.dini...@nxp.com>; Sahil Malhotra <sahil.malho...@nxp.com>; > Pankaj Gupta <pankaj.gu...@nxp.com>; Varun Sethi <v.se...@nxp.com>; dl- > uboot-imx <uboot-...@nxp.com>; Shengzhou Liu <shengzhou....@nxp.com>; > Mingkai Hu <mingkai...@nxp.com>; Rajesh Bhagat <rajesh.bha...@nxp.com>; > Meenakshi Aggarwal <meenakshi.aggar...@nxp.com>; Wasim Khan > <wasim.k...@nxp.com>; Alison Wang <alison.w...@nxp.com>; Pramod > Kumar <pramod.kuma...@nxp.com>; Andy Tang <andy.t...@nxp.com>; > Adrian Alonso <adrian.alo...@nxp.com>; Vladimir Oltean <olte...@gmail.com> > Subject: RE: [EXT] Re: [PATCH v8 01/15] crypto/fsl: Add support for CAAM Job > ring driver model > > Caution: EXT Email > > Hello Stefano, > > > -----Original Message----- > > From: U-Boot <u-boot-boun...@lists.denx.de> On Behalf Of Stefano Babic > > Sent: Thursday, March 3, 2022 4:30 PM > > To: Gaurav Jain <gaurav.j...@nxp.com>; Stefano Babic <sba...@denx.de>; > > u- b...@lists.denx.de; Marek Vasut <ma...@denx.de> > > Cc: Fabio Estevam <feste...@gmail.com>; Peng Fan <peng....@nxp.com>; > > Simon Glass <s...@chromium.org>; Priyanka Jain <priyanka.j...@nxp.com>; > > Ye Li <ye...@nxp.com>; Horia Geanta <horia.gea...@nxp.com>; Ji Luo > > <ji....@nxp.com>; Franck Lenormand <franck.lenorm...@nxp.com>; Silvano > > Di Ninno <silvano.dini...@nxp.com>; Sahil Malhotra > > <sahil.malho...@nxp.com>; Pankaj Gupta <pankaj.gu...@nxp.com>; Varun > > Sethi <v.se...@nxp.com>; dl-uboot-imx <uboot-...@nxp.com>; Shengzhou > > Liu <shengzhou....@nxp.com>; Mingkai Hu <mingkai...@nxp.com>; Rajesh > > Bhagat <rajesh.bha...@nxp.com>; Meenakshi Aggarwal > > <meenakshi.aggar...@nxp.com>; Wasim Khan <wasim.k...@nxp.com>; > Alison > > Wang <alison.w...@nxp.com>; Pramod Kumar > <pramod.kuma...@nxp.com>; > > Andy Tang <andy.t...@nxp.com>; Adrian Alonso <adrian.alo...@nxp.com>; > > Vladimir Oltean <olte...@gmail.com> > > Subject: Re: [EXT] Re: [PATCH v8 01/15] crypto/fsl: Add support for > > CAAM Job ring driver model > > > > Hi, > > > > On 03.03.22 14:41, Gaurav Jain wrote: > > > Hi Stefano > > > > > > As we have not received any response from imx6dl_mamoj board maintainer. > > > I propose the below solution > > > > > > --- a/arch/arm/mach-imx/Kconfig > > > +++ b/arch/arm/mach-imx/Kconfig > > > @@ -49,8 +49,8 @@ config USE_IMXIMG_PLUGIN config IMX_HAB > > > - select FSL_CAAM if HAS_CAAM > > > - imply CMD_DEKBLOB if HAS_CAAM > > > + imply FSL_CAAM if HAS_CAAM > > > + imply CMD_DEKBLOB if FSL_CAAM > > > Help > > > > > > > IMO this is ok, I was also wrong, Marek is not the maintainer of this > > board. This was the only board with broken build - let's say, I will > > still wait a couple of days, and if there is no comments, I will apply > > your series (but then V10). I can apply this fix myself, no need to > > post the series again (I have not seen any other comment or request to > change). > > I had a couple of concerns which I raised in V10 without any follow-up. > > Main thing is the JR reservation, which (if applied) would lead to > misconfiguration when upstream TF-A is used, see [1]. > > > > > Regards, > > Stefano > > > > Link: [1]: > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.kern > el.org%2Fu- > boot%2FAM6PR06MB4691178347827CB77F44F537A6309%40AM6PR06MB4691 > .eurprd06.prod.outlook.com%2F&data=04%7C01%7Cgaurav.jain%40nxp.c > om%7C87775d9291044d6aad3808d9ffa8db17%7C686ea1d3bc2b4c6fa92cd99c5 > c301635%7C0%7C0%7C637821922977072803%7CUnknown%7CTWFpbGZsb3d8 > eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D% > 7C3000&sdata=3Gng6vVpNKyV8xNLSEuwG%2F7DEVDjSnYh5iYi47wvyvc%3 > D&reserved=0
Ok, I will follow the approach suggested by you. First a patch will be submitted to ATF and then the one for uboot will follow. Will send a V11 after removing the JR0 reservation code. Any other concern? Gaurav > > -- andrey
