On Tue, Mar 08, 2022 at 01:21:41PM +0000, Gaurav Jain wrote: > > > > -----Original Message----- > > From: Tom Rini <tr...@konsulko.com> > > Sent: Tuesday, March 8, 2022 6:19 PM > > To: Fabio Estevam <feste...@gmail.com> > > Cc: Gaurav Jain <gaurav.j...@nxp.com>; Michael Walle <mich...@walle.cc>; > > sba...@denx.de; 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> > > Subject: Re: [EXT] Re: [PATCH v8 01/15] crypto/fsl: Add support for CAAM Job > > ring driver model > > > > 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. > > 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? > > > SPL exceeds the maximum size for only imx6dl_mamoj. > spl/u-boot-spl.bin exceeds file size limit: > limit: 0xefa0 bytes > actual: 0x1004d bytes > excess: 0x10ad bytes > > on further checking I see that caam is not initialized in SPL but only built. > So disabling the build for caam driver in SPL should not be a problem. > > (Added Jagan, Raffaele, Simone as board maintainer.)
That's a lot of growth, what's going on? And is the CAAM being built but not initialized a generic issue with imx6 platforms? That seems like something to further understand, and hopefully the board maintainers can chime in here soon. -- Tom
signature.asc
Description: PGP signature