[U-Boot] [PATCH] Makefile: use u-boot.map for binary_size_check

Tom Rini trini at ti.com
Thu Jul 24 15:50:29 CEST 2014


On Thu, Jul 24, 2014 at 10:31:14AM +0200, Wolfgang Denk wrote:
> Dear Chris Packham,
> 
> In message <1406179627-9496-1-git-send-email-judge.packham at gmail.com> you wrote:
> > u-boot.map is generated automatically by the compiler and more
> > importantly can handle addresses >4GB.
> ...
> > > I think it should be possible to change binary_size_check to use
> > > u-boot.map instead of System.map but would that be OK for all
> > > architectures?
> ...
> > Something like this works for me but maybe there is a way to get nm to handle 
> > addresses >4GB.
> ...
> > +	map_size=$(shell cat u-boot.map | \
> >  		awk '/_image_copy_start/ {start = $$1} /_image_binary_end/ {end = $$1} END {if (start != "" && end != "") print "ibase=16; " toupper(end) " - " toupper(start)}' \
> > +		| sed 's/0X//g' \
> 
> 
> Do we _really_ need all of this?
> 
> This looks very much like creaping featurism to me...

So it's a safetey check.  The various linker script clean-ups we've been
doing have introduced a problem from time to time and binary_check_size
gives a sanity check.  This just fixes a corner case in the test.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140724/89be7428/attachment.pgp>


More information about the U-Boot mailing list