Re: [PATCH 4/4] ARM: dts: imx8m: imx8mm-mx8menlo: Enable SPL SDP support

From: Michael Nazzareno Trimarchi <michael@amarulasolutions.com>
To: Adam Ford <aford173@gmail.com>
Cc: Fabio Estevam <festevam@denx.de>, Marek Vasut <marex@denx.de>,
	 U-Boot Mailing List <u-boot@lists.denx.de>,
	Marcel Ziswiler <marcel.ziswiler@toradex.com>,
	 Max Krummenacher <max.krummenacher@toradex.com>,
	Peng Fan <peng.fan@nxp.com>,  Stefano Babic <sbabic@denx.de>
Subject: Re: [PATCH 4/4] ARM: dts: imx8m: imx8mm-mx8menlo: Enable SPL SDP support
Date: Sat, 24 Sep 2022 21:19:28 +0200	[thread overview]
Message-ID: <CAOf5uwkoMje8VDN1umjQ=Q-0f15CNe=cvgxj5H89WbjuXhGdFw@mail.gmail.com> (raw)
In-Reply-To: <CAHCN7xK-O0Tn+VfTPVU6TiqTUdJ7-dbT1isaXcNc_qMQv9ip5A@mail.gmail.com>

Hi

Il sab 24 set 2022, 21:10 Adam Ford <aford173@gmail.com> ha scritto:

> On Sat, Sep 24, 2022 at 2:00 PM Michael Nazzareno Trimarchi
> <michael@amarulasolutions.com> wrote:
> >
> >
> >
> > Il sab 24 set 2022, 20:54 Adam Ford <aford173@gmail.com> ha scritto:
> >>
> >> On Sat, Sep 24, 2022 at 1:52 PM Michael Nazzareno Trimarchi
> >> <michael@amarulasolutions.com> wrote:
> >> >
> >> > On Sat, Sep 24, 2022 at 8:38 PM Adam Ford <aford173@gmail.com> wrote:
> >> > >
> >> > > On Mon, Sep 19, 2022 at 5:02 PM Fabio Estevam <festevam@denx.de>
> wrote:
> >> > > >
> >> > > > Hi Marek,
> >> > > >
> >> > > > On 19/09/2022 16:41, Marek Vasut wrote:
> >> > > > > Enable DM USB, DM PHY and USB gadget support in imx8mm-mx8menlo
> SPL
> >> > > > > to let the board continue SDP loading of second stage after the
> first
> >> > > > > stage was loaded by BootROM SDP implementation. It is not
> possible to
> >> > > > > jump back into BootROM v1 and let the BootROM implementation
> continue
> >> > > > > the SDP loading, all this has to be performed by the U-Boot CI
> HDRC
> >> > > > > controller driver and SDP protocol implementation, both of
> which fit
> >> > > > > into the SPL just barely.
> >> > > > >
> >> > > > > With this patch, it is possible to start both U-Boot SPL and
> U-Boot
> >> > > > > using e.g. uuu on this board as follows:
> >> > > > >
> >> > > > > $ uuu -brun spl flash.bin
> >> > > > >
> >> > > > > Signed-off-by: Marek Vasut <marex@denx.de>
> >> > > >
> >> > > > This is great, I was able to test SPL SDP on an imx8mm-evk after
> >> > > > applying these
> >> > > > same changes to imx8mm-evk and reducing the SPL size.
> >> > >
> >> > > Fabio,
> >> > >
> >> > > Can you tell me what you did to reduce the SPL size?
> >> > >
> >> >
> >> > CONFIG_LTO from the other patch
> >>
> >> Bummer.  I was hoping there was more.  I already have LTO enabled.  I
> >> was hoping there was more to it.
> >
> >
> > What version of U-Boot are you using?
> >
> > Mainline or some older one. Are you compile under yocto?
>
> I am using the Mainline with the aarch64 gcc from Ubuntu 22.04.  I am
> over by ~1100 bytes with LTO enabled, but I'm going through my SPL
> drivers to see what I can either remove or reduce functionality.
>

I was thinking about an issue was solved by patching U-Boot on increase
size of 3% in yocto toolchain.

Michael

>
> adam
> >
> > Michael
> >
> >
> >>
> >> adam
> >> >
> >> > Michael
> >> >
> >> > > adam
> >> > > >
> >> > > > For the series:
> >> > > >
> >> > > > Reviewed-by: Fabio Estevam <festevam@denx.de>
> >> >
> >> >
> >> >
> >> > --
> >> > Michael Nazzareno Trimarchi
> >> > Co-Founder & Chief Executive Officer
> >> > M. +39 347 913 2170
> >> > michael@amarulasolutions.com
> >> > __________________________________
> >> >
> >> > Amarula Solutions BV
> >> > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> >> > T. +31 (0)85 111 9172
> >> > info@amarulasolutions.com
> >> > www.amarulasolutions.com
>