From: Eugen.Hristev at microchip.com <Eugen.Hristev@microchip.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/4] at91: SPL size check for at91 boards
Date: Mon, 19 Nov 2018 08:22:55 +0000 [thread overview]
Message-ID: <1542615470-17992-1-git-send-email-eugen.hristev@microchip.com> (raw)
This is a review of the patch
[U-Boot,RFC] ARM: at91: lds: add test SPL binary size and bbs size
Commit:
ARM: at91: lds: add test SPL binary size and bbs size
will check for the SPL size and bss to see if it can fit the known RAM area
where it's supposed to be loaded. If the size exceeds, it will print an error
in build.
However, some platforms already have the SPL broken: it is too big for the RAM
area. These platforms would fail to build if we enable this SPL size check.
So, I made the commits for these defconfigs to disable SPL before adding the
actual check (for bisectability, do not break any builds in between the series
commits).
In short, first three commits will prepare defconfigs for the SPL check and the
last commit will implement the actual size checks.
Eugen Hristev (2):
configs: corvus_defconfig: Remove SPL
configs: wb45n_defconfig: Remove SPL
Wenyou Yang (2):
configs: picosam9g45_defconfig: Remove SPL
ARM: at91: lds: add test SPL binary size and bbs size
arch/arm/mach-at91/arm926ejs/u-boot-spl.lds | 10 ++++++++++
configs/corvus_defconfig | 8 --------
configs/picosam9g45_defconfig | 10 ----------
configs/wb45n_defconfig | 6 ------
4 files changed, 10 insertions(+), 24 deletions(-)
--
2.7.4
next reply other threads:[~2018-11-19 8:22 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-19 8:22 Eugen.Hristev at microchip.com [this message]
2018-11-19 8:22 ` [U-Boot] [PATCH 1/4] configs: picosam9g45_defconfig: Remove SPL Eugen.Hristev at microchip.com
2018-11-19 8:22 ` [U-Boot] [PATCH 2/4] configs: corvus_defconfig: " Eugen.Hristev at microchip.com
2018-11-19 8:23 ` [U-Boot] [PATCH 3/4] configs: wb45n_defconfig: " Eugen.Hristev at microchip.com
2018-11-19 10:29 ` Ben Whitten
2018-11-19 14:27 ` Ben Whitten
2018-11-19 8:23 ` [U-Boot] [PATCH 4/4] ARM: at91: lds: add test SPL binary size and bbs size Eugen.Hristev at microchip.com
2018-11-19 14:31 ` Ben Whitten
2018-11-19 15:13 ` Eugen.Hristev at microchip.com
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=1542615470-17992-1-git-send-email-eugen.hristev@microchip.com \
--to=eugen.hristev@microchip.com \
--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