u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
From: Tim Harvey <tharvey@gateworks.com>
To: Adam Ford <aford173@gmail.com>
Cc: Marek Vasut <marex@denx.de>,
	 Michael Nazzareno Trimarchi <michael@amarulasolutions.com>,
	Fabio Estevam <festevam@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: Mon, 26 Sep 2022 10:14:15 -0700	[thread overview]
Message-ID: <CAJ+vNU1Xz9VwVr+fgB-jZbeBG6FTDXc-q99nZ0g+-AL4kuv0pA@mail.gmail.com> (raw)
In-Reply-To: <CAHCN7xKNSpg2tzm4-ATDKqVoGYRTwT_ya3pzJ-OQPT7zXSUHag@mail.gmail.com>

On Mon, Sep 26, 2022 at 10:06 AM Adam Ford <aford173@gmail.com> wrote:
>
> On Mon, Sep 26, 2022 at 11:59 AM Tim Harvey <tharvey@gateworks.com> wrote:
> >
> > On Sat, Sep 24, 2022, 2:54 PM Adam Ford <aford173@gmail.com> wrote:
> > >
> > > On Sat, Sep 24, 2022 at 4:47 PM Marek Vasut <marex@denx.de> wrote:
> > > >
> > > > On 9/24/22 21:10, Adam Ford wrote:
> > > >
> > > > Hi,
> > > >
> > > > [...]
> > > >
> > > > > 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.
> > > >
> > > > Try
> > > >
> > > > $ aarch64...readelf -s spl/u-boot-spl | sort -nk 3
> > > >
> > > > That should give you a list of large symbols you might want to inspect.
> > >
> > > Thank you!
> > >
> >
> > Adam,
> >
> > I'm curious what you find as my SPL for imx8mm_venice_defconfig also
> > overflows by about 3K when enabling what's needed here for SDP as
> > well. I have 4 different dram configs which are chewing up about 1K
> > each.
>
> Tim,
>
> I have it building successfully now, and it loads over USB. I had to
> disable BD718x7 PMIC children binding which required a small change to
> the PMIC driver [1].  I then removed HS400, HS200 and UHS support in
> SPL.  Once I did all that, the code size fit into SPL space.
>
> [1] - https://patchwork.ozlabs.org/project/uboot/patch/20220817112403.68144-2-aford173@gmail.com/
>
> I'll try to push my patch today or tomorrow.  I was doing some other
> clean-up, but I might just push that as a small series in itself.
>

Adam,

Yes, I have already done those same things for imx8mm_venice but am
still 3K over due to my 4x dram configs. Each dram config must be
about 3K as I only have to remove one of them to successfully build.

Thanks,

Tim

  parent reply	other threads:[~2022-09-26 17:14 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-19 19:41 [PATCH 1/4] ARM: imx8m: Deduplicate CAAM init with arch_misc_init() call Marek Vasut
2022-09-19 19:41 ` [PATCH 2/4] ARM: imx8m: phycore_imx8mm: Drop bogus spl_board_init() Marek Vasut
2022-09-21  5:31   ` Peng Fan
2022-10-21 11:42   ` sbabic
2022-09-19 19:41 ` [PATCH 3/4] ARM: imx8m: verdin-imx8mm: Drop bogus content from spl_board_init() Marek Vasut
2022-09-21  5:31   ` Peng Fan
2022-10-21 11:43   ` sbabic
2022-09-19 19:41 ` [PATCH 4/4] ARM: dts: imx8m: imx8mm-mx8menlo: Enable SPL SDP support Marek Vasut
2022-09-19 22:02   ` Fabio Estevam
2022-09-24 18:38     ` Adam Ford
2022-09-24 18:52       ` Michael Nazzareno Trimarchi
2022-09-24 18:54         ` Adam Ford
2022-09-24 19:00           ` Michael Nazzareno Trimarchi
2022-09-24 19:10             ` Adam Ford
2022-09-24 19:19               ` Michael Nazzareno Trimarchi
2022-09-24 21:47               ` Marek Vasut
2022-09-24 21:54                 ` Adam Ford
2022-09-26 16:58                   ` Tim Harvey
2022-09-26 17:06                     ` Adam Ford
2022-09-26 17:12                       ` Fabio Estevam
2022-09-26 17:28                         ` Adam Ford
2022-09-26 17:50                           ` Sean Anderson
2022-09-26 20:04                         ` Tim Harvey
2022-09-26 17:14                       ` Tim Harvey [this message]
2022-09-21  5:30   ` Peng Fan
2022-09-22  2:42     ` Marek Vasut
2022-10-21 11:43   ` sbabic
2022-09-21  5:31 ` [PATCH 1/4] ARM: imx8m: Deduplicate CAAM init with arch_misc_init() call Peng Fan
2022-10-21 11:52   ` Stefano Babic
2022-10-21 11:42 ` sbabic

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAJ+vNU1Xz9VwVr+fgB-jZbeBG6FTDXc-q99nZ0g+-AL4kuv0pA@mail.gmail.com \
    --to=tharvey@gateworks.com \
    --cc=aford173@gmail.com \
    --cc=festevam@denx.de \
    --cc=marcel.ziswiler@toradex.com \
    --cc=marex@denx.de \
    --cc=max.krummenacher@toradex.com \
    --cc=michael@amarulasolutions.com \
    --cc=peng.fan@nxp.com \
    --cc=sbabic@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).