From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wolfgang Denk <wd@denx.de>
Date: Wed, 16 Feb 2011 19:58:07 +0100
Subject: [U-Boot] [PATCH] Introduce a new linker flag LDFLAGS_FINAL
In-Reply-To: <1297881643.1977.28.camel@haiying-laptop>
References: <1296498767-26408-1-git-send-email-Haiying.Wang@freescale.com>
	<20110131193309.745E7D4D67C@gemini.denx.de>
	<1296504850.2049.434.camel@haiying-laptop>
	<20110131203041.E3A12D4D67C@gemini.denx.de>
	<1297878184.1977.18.camel@haiying-laptop>
	<20110216182938.754031539C5@gemini.denx.de>
	<1297881643.1977.28.camel@haiying-laptop>
Message-ID: <20110216185807.85766151B54@gemini.denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Haiying Wang,

In message <1297881643.1977.28.camel@haiying-laptop> you wrote:
>
> > We have some 524 PowerPC configurations in boards.cfg, and the compare
> > must be done for _all_ off these, i. e. there are 2 x 524 pairs of
> > files to be compared.  Did you really do this?
> > > it is ok to apply this patch, if you are ok with the name _FINAL?
> Sorry, I did not get the 2x254 pairs of System.map and u-boot.map. Running "MAKEALL powerpc" only left the last pair. Could you please show me how I can keep the former 523 pairs?

I usually do this on the fly using shell scripting capabilities, i.e.
something like this (copying some shell functions from MAKEALL):

-----------------------------------------------------------------------
boards_by_field()
{
        awk \
                -v field="$1" \
                -v select="$2" \
                '($1 !~ /^#/ && $field == select) { print $1 }' \
                boards.cfg
}
boards_by_arch() { boards_by_field 2 "$@" ; }

for board in $(boards_by_arch powerpc) ; do
./MAKEALL $board 
cp System.map System.map.${board}.unpatched
cp u-boot.map u-boot.map.${board}.unpatched
done

...apply patch...

for board in $(boards_by_arch powerpc) ; do
./MAKEALL $board
cmp System.map.${board}.unpatched System.map || diff -u System.map.${board}.unpatched System.map >System.map.${board}.diff
cmp u-boot.map.${board}.unpatched u-boot.map || diff -u u-boot.map.${board}.unpatched u-boot.map >u-boot.map.${board}.diff
done

less *.diff
-----------------------------------------------------------------------

Note this is untested; you probably get the idea.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
For every complex problem, there is a solution that is simple,  neat,
and wrong.                                               - Mark Twain