[EXT] Re: [PATCH v8 01/15] crypto/fsl: Add support for CAAM Job ring driver model

[Tom Rini]

On Sat, Mar 19, 2022 at 05:48:59PM +0100, Michael Nazzareno Trimarchi wrote:
> Hi Tom
> 
> On Sat, Mar 19, 2022 at 5:05 PM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Sat, Mar 19, 2022 at 02:51:01PM +0100, Michael Nazzareno Trimarchi wrote:
> > > Hi
> > >
> > > On Sat, Mar 19, 2022 at 2:25 PM Fabio Estevam <festevam at gmail.com> wrote:
> > > >
> > > > Hi Michael,
> > > >
> > > > On Sat, Mar 19, 2022 at 6:47 AM Michael Nazzareno Trimarchi
> > > > <michael at amarulasolutions.com> wrote:
> > > > >
> > > > > HI
> > > > >
> > > > > Please send me a link to apply your series
> > > >
> > > > You can get the series from patchwork (just click in the 'series' button)
> > > > https://patchwork.ozlabs.org/project/uboot/patch/20220112133127.16880-2-gaurav.jain@nxp.com/
> > >
> > > Done, well, I have tried remove DM_GPIO in SPL. Seems that I need to
> > > remove a lot of features. I have tested LTO build but size
> > > of SPL increase. Is that strange?
> >
> > That is very strange.  Can you post your patches somewhere?
> >
> > --
> > Tom
> 
> Start from here 9776c4e9d00ac49d6388ffe9e084ff03b37ae479
> export ARCH=arm
> export CROSS_COMPILE=arm-linux-gnueabihf-
> 
> make imx6dl_mamoj_defconfig
> build it
> 
> Try then to enable LTO and build it again
> 
> spl/u-boot-spl.bin exceeds file size limit:
>   limit:  0xefa0 bytes
>   actual: 0xf071 bytes
>   excess: 0xd1 bytes
> 
> So LTO does not help even on beginning on this board. You don't need
> to apply any patch for this test

I think that's some artifact of mixing LTO/non-LTO and the world not
getting rebuilt?  Just enabling LTO after the defconfig works fine and
is smaller than before.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220319/45d32428/attachment.sig>