Hello Michael > -----Original Message----- > From: Michael Walle <mich...@walle.cc> > Sent: Monday, November 8, 2021 2:29 PM > To: ZHIZHIKIN Andrey <andrey.zhizhi...@leica-geosystems.com> > Cc: Gaurav Jain <gaurav.j...@nxp.com>; Adam Ford <aford...@gmail.com>; > U-Boot Mailing List <u-boot@lists.denx.de>; Stefano Babic <sba...@denx.de>; > 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 v4 03/16] i.MX8M: crypto: updated device tree > for > supporting DM in SPL > > Caution: EXT Email > > Am 2021-11-08 09:48, schrieb ZHIZHIKIN Andrey: > .. > > >> > Disabling the JR0 node in the kernel makes this error go away, and > >> > decreases the JR count to 2 which can be observed in the NXP vendor > >> > kernel. > >> > > >> > I suggest you to extract the node disabling from this patch and send it > >> > to > Kernel. > >> > Once accepted in upstream - this change would be picked up by the > >> > U-Boot at next DTB re-sync. > > No. Please don't disable the node in the kernel device tree. IMHO TF-A (or > whoever is using the job ring) is optional. So either u-boot should disable > it, > either statically by -u-boot.dtsi (but please _per_ > board) or at runtime by fixing up the device tree. Another way would be for > linux > to skip probing if its not available. Though I don't know whether this is "the > ususal way". > > But as Andrey mentioned, any job ring could be taken away. So the -u-boot.dtsi > method doesn't seem to be that good. >
Moved the JR0 disabled code to *-u-boot.dtsi files in version 5 of this patch series. Regards Gaurav Jain > -michael