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

[Tom Rini]

On Sat, Mar 19, 2022 at 06:08:18PM +0100, Michael Nazzareno Trimarchi wrote:
> Hi Tom
> 
> On Sat, Mar 19, 2022 at 6:03 PM Tom Rini <trini at konsulko.com> wrote:
> >
> > 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.
> 
> In order to save space I did not change what we have. Every build starts from
> a mrproper and new configuration. I can not mix LTO/non-LTO in this case. I'm
> using upstream defconfig and upstream defconfig + LTO enabled and the result is
> that spl increase in size for LTO building. I'm working to keep out
> part that are not really needed but
> I was hoping that LTO give me some help here

Yes, LTO saves about 5KiB on the SPL binary.  You just need to enable it
in the config before you start building, not after you've built
everything once.

-- 
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/e563927a/attachment.sig>