From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 23 May 2019 16:16:34 -0400 Subject: [U-Boot] [PATCH v2] spl: add overall SPL size check In-Reply-To: <4d51152d-1236-ee77-5aa0-ca60bd3cfe7b@gmail.com> References: <20190422202721.15892-1-simon.k.r.goldschmidt@gmail.com> <20190511015553.GJ6312@bill-the-cat> <4d51152d-1236-ee77-5aa0-ca60bd3cfe7b@gmail.com> Message-ID: <20190523201634.GI22232@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 Thu, May 23, 2019 at 10:08:54PM +0200, Simon Goldschmidt wrote: > Am 13.05.2019 um 14:45 schrieb Simon Goldschmidt: > >On Sat, May 11, 2019 at 3:55 AM Tom Rini wrote: > >> > >>On Mon, Apr 22, 2019 at 10:27:21PM +0200, Simon Goldschmidt wrote: > >> > >>>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 > >> > >>OK, this has a race / dependency problem: > >>https://travis-ci.org/trini/u-boot/jobs/530803338 > > > >Hmm, let me check that. > > OK, so as I see it, this is not a race but a dependency problem: the > "tools-only" target just does not generate the "generic-asm-offsets.h" file. > > However, just adding a dependency to include/generated/generic-asm-offsets.h > leads to an error (no rule to build that file) since that file is generated > by Kbuild magic that I don't really get... > > Any idea how to make this depend on /Kbuild being run? For context, https://patchwork.ozlabs.org/patch/1088885/ Yamada-san, do you have any ideas? Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: