From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Sun, 26 May 2019 07:09:37 -0400 Subject: [U-Boot] [PATCH v3] spl: add overall SPL size check In-Reply-To: <272edc6e-1a4a-e61a-4990-d38f02218343@gmail.com> References: <20190524200704.29373-1-simon.k.r.goldschmidt@gmail.com> <272edc6e-1a4a-e61a-4990-d38f02218343@gmail.com> Message-ID: <20190526110937.GV20781@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, May 24, 2019 at 10:10:48PM +0200, Simon Goldschmidt wrote: > Am 24.05.2019 um 22:07 schrieb Simon Goldschmidt: > >This adds a size check for SPL that can dynamically check generated > >SPL binaries (including devicetree) for a size limit that ensures > >this image plus global data, heap and stack fit in initial SRAM. > > > >Since some of these sizes are not available to make, a new host tool > >'spl_size_limit' is added that dumps the resulting maximum size for > >an SPL binary to stdout. This tool is used in toplevel Makefile to > >implement the size check on SPL binaries. > > > >Signed-off-by: Simon Goldschmidt > >--- > > > >Changes in v3: > >- don't build this new tools for 'make tools-only' > > So this is how far I got. > > Tom, your idea with making this multi-config aware (U-Boot, SPL and TPL) > does not seem to work as 'tools' are only built once, not once per > U-Boot/SPL/TPL. So if we wanted to use this for TPL, too, that would either > mean create yet another tool or pass an option to this new tool to differ > between SPL and TPL. OK. Hopefully at least we can use a Makefile rule to transform the source code rather than have a copy/paste version of the code. Thanks again! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: