-----Original Message----- From: Tom Rini trini@konsulko.com Sent: Tuesday, March 8, 2022 6:19 PM To: Fabio Estevam festevam@gmail.com Cc: Gaurav Jain gaurav.jain@nxp.com; Michael Walle michael@walle.cc; sbabic@denx.de; Varun Sethi V.Sethi@nxp.com; Adrian Alonso adrian.alonso@nxp.com; Alison Wang alison.wang@nxp.com; Andy Tang andy.tang@nxp.com; Franck Lenormand franck.lenormand@nxp.com; Horia Geanta horia.geanta@nxp.com; Ji Luo ji.luo@nxp.com; marex@denx.de; Meenakshi Aggarwal meenakshi.aggarwal@nxp.com; Mingkai Hu mingkai.hu@nxp.com; olteanv@gmail.com; Pankaj Gupta pankaj.gupta@nxp.com; Peng Fan peng.fan@nxp.com; Pramod Kumar pramod.kumar_1@nxp.com; Priyanka Jain priyanka.jain@nxp.com; Rajesh Bhagat rajesh.bhagat@nxp.com; Sahil Malhotra sahil.malhotra@nxp.com; Shengzhou Liu shengzhou.liu@nxp.com; Silvano Di Ninno silvano.dininno@nxp.com; sjg@chromium.org; u-boot@lists.denx.de; dl- uboot-imx uboot-imx@nxp.com; Wasim Khan wasim.khan@nxp.com; Ye Li ye.li@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.jain@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.)
Gaurav
-- Tom