Hello Michael A gentle reminder!! As you said earlier, can you test the mamoj board with proposed solution to fix SPL size issue.
Regards Gaurav Jain > -----Original Message----- > From: Gaurav Jain > Sent: Monday, March 14, 2022 11:18 AM > To: Michael Nazzareno Trimarchi <mich...@amarulasolutions.com>; Stefano > Babic <sba...@denx.de> > Cc: Tom Rini <tr...@konsulko.com>; Fabio Estevam <feste...@gmail.com>; > Michael Walle <mich...@walle.cc>; Varun Sethi <v.se...@nxp.com>; Adrian > Alonso <adrian.alo...@nxp.com>; Alison Wang <alison.w...@nxp.com>; > Andy Tang <andy.t...@nxp.com>; Franck Lenormand > <franck.lenorm...@nxp.com>; Horia Geanta <horia.gea...@nxp.com>; Ji > Luo <ji....@nxp.com>; ma...@denx.de; Meenakshi Aggarwal > <meenakshi.aggar...@nxp.com>; Mingkai Hu <mingkai...@nxp.com>; > olte...@gmail.com; Pankaj Gupta <pankaj.gu...@nxp.com>; Peng Fan > <peng....@nxp.com>; Pramod Kumar <pramod.kuma...@nxp.com>; > Priyanka Jain <priyanka.j...@nxp.com>; Rajesh Bhagat > <rajesh.bha...@nxp.com>; Sahil Malhotra <sahil.malho...@nxp.com>; > Shengzhou Liu <shengzhou....@nxp.com>; Silvano Di Ninno > <silvano.dini...@nxp.com>; s...@chromium.org; u-boot@lists.denx.de; dl- > uboot-imx <uboot-...@nxp.com>; Wasim Khan <wasim.k...@nxp.com>; > Ye Li <ye...@nxp.com>; Raffaele RECALCATI <raffaele.recalc...@bticino.it>; > Jagan Teki <ja...@amarulasolutions.com> > Subject: RE: [EXT] Re: [PATCH v8 01/15] crypto/fsl: Add support for CAAM Job > ring driver model > > Hi Michael > > > -----Original Message----- > > From: Michael Nazzareno Trimarchi <mich...@amarulasolutions.com> > > Sent: Tuesday, March 8, 2022 7:05 PM > > To: Stefano Babic <sba...@denx.de> > > Cc: Tom Rini <tr...@konsulko.com>; Fabio Estevam > <feste...@gmail.com>; > > Gaurav Jain <gaurav.j...@nxp.com>; Michael Walle <mich...@walle.cc>; > > Varun Sethi <v.se...@nxp.com>; Adrian Alonso <adrian.alo...@nxp.com>; > > Alison Wang <alison.w...@nxp.com>; Andy Tang <andy.t...@nxp.com>; > > Franck Lenormand <franck.lenorm...@nxp.com>; Horia Geanta > > <horia.gea...@nxp.com>; Ji Luo <ji....@nxp.com>; ma...@denx.de; > > Meenakshi Aggarwal <meenakshi.aggar...@nxp.com>; Mingkai Hu > > <mingkai...@nxp.com>; olte...@gmail.com; Pankaj Gupta > > <pankaj.gu...@nxp.com>; Peng Fan <peng....@nxp.com>; Pramod Kumar > > <pramod.kuma...@nxp.com>; Priyanka Jain <priyanka.j...@nxp.com>; > > Rajesh Bhagat <rajesh.bha...@nxp.com>; Sahil Malhotra > > <sahil.malho...@nxp.com>; Shengzhou Liu <shengzhou....@nxp.com>; > > Silvano Di Ninno <silvano.dini...@nxp.com>; s...@chromium.org; u- > > b...@lists.denx.de; dl-uboot-imx <uboot-...@nxp.com>; Wasim Khan > > <wasim.k...@nxp.com>; Ye Li <ye...@nxp.com>; Raffaele RECALCATI > > <raffaele.recalc...@bticino.it>; Jagan Teki > > <ja...@amarulasolutions.com> > > Subject: Re: [EXT] Re: [PATCH v8 01/15] crypto/fsl: Add support for > > CAAM Job ring driver model > > > > Caution: EXT Email > > > > Hi > > > > On Tue, Mar 8, 2022 at 2:28 PM Stefano Babic <sba...@denx.de> wrote: > > > > > > Hi Tom, > > > > > > On 08.03.22 13:48, Tom Rini wrote: > > > > On Tue, Mar 08, 2022 at 08:12:27AM -0300, Fabio Estevam wrote: > > > >> On Tue, Mar 8, 2022 at 8:10 AM Gaurav Jain <gaurav.j...@nxp.com> > > wrote: > > > >> > > > >>> I further checked on your concern and propose the below change > > > >>> to > > stop building caam driver in SPL for imx6dl_mamoj. > > > >>> > > > >>> --- a/configs/imx6dl_mamoj_defconfig > > > >>> +++ b/configs/imx6dl_mamoj_defconfig > > > >>> @@ -61,3 +61,4 @@ CONFIG_USB_GADGET_MANUFACTURER="FSL" > > > >>> CONFIG_USB_GADGET_VENDOR_NUM=0x0525 > > > >>> CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 > > > >>> CONFIG_CI_UDC=y > > > >>> +CONFIG_SPL_CRYPTO=n > > > >> > > > >> No, this is not how defconfig works. > > > >> > > > >> You should set: > > > >> > > > >> # CONFIG_SPL_CRYPTO is not set > > > > > > > > But more, WHY is this the right answer? You're disabling > > > > functionality and the board maintainers aren't even on the CC list > > > > for this thread it looks like. > > > > > > This was my blocking point, too, for this series. There is no answer > > > about the board. > > > > > > I can just imagine (as for other MX6DL) that the required > > > functionality is secure boot, then CONFIG_IMX_HAB is enough. But > > > well, just the board maintainers can say. Added Raffaele and Jagan > > > as reported maintainers for this board. > > > > > > > Why are you not fixing the board so it still links? What makes > > > > this board a problem that other (I assume) imx6dl boards are not? > > > > > > Well, this board has, compared to other MX6DL, a lot of SPL_ option > > > that are bloating the SPL size. And then does not match anymore with > > > the internal RAm, while other MX6DL boards are not so affected by > > > the increased size because they have no SPL at all or SPL with less > > > options and then smaller. We do not need to fix the link, but reduce > > > the size, and nobody else than the maintainers can tell us if > > > removing some feature is ok. > > > > > > > I will test this week, and try to find a board here to test > > > > * Jagan * Do you have a momoj? > > Have you tested mamoj board? > > Regards > Gaurav > > > > Michael > > > > > Regards, > > > Stefano > > > > > > -- > > > > > > ================================================================ > > ===== > > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, > > > Germany > > > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: > > > sba...@denx.de > > > > > > ================================================================ > > ===== > > > > > > > > -- > > Michael Nazzareno Trimarchi > > Co-Founder & Chief Executive Officer > > M. +39 347 913 2170 > > mich...@amarulasolutions.com > > __________________________________ > > > > Amarula Solutions BV > > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL T. +31 (0)85 111 9172 > > i...@amarulasolutions.com > > > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.a > > marulasolutions.com%2F&data=04%7C01%7Cgaurav.jain%40nxp.com% > > > 7C843e57f75346449f13ae08da010867b6%7C686ea1d3bc2b4c6fa92cd99c5c3 > > > 01635%7C0%7C0%7C637823432854617357%7CUnknown%7CTWFpbGZsb3d8 > > > eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3 > > > D%7C3000&sdata=f025HmlIM8FuTEYspDjwOevTT1k7JRZeIiS%2FpTH2hc > > Y%3D&reserved=0
