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

[Michael Nazzareno Trimarchi]

Hi

On Sat, Mar 19, 2022 at 6:25 PM Tom Rini <trini at konsulko.com> wrote:
>
> 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.

Offcourse ;) but this not the case. I don't drink enough to think that
change a config, decrease the build size ;)

Michael

>
> --
> Tom


-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael at amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info at amarulasolutions.com
www.amarulasolutions.com