[U-Boot] [PATCH 3/3] common: Add .u_boot_list into all linker files

Tom Rini trini at ti.com
Mon Sep 24 19:06:11 CEST 2012


On Mon, Sep 24, 2012 at 06:53:19PM +0200, Marek Vasut wrote:
> Dear Joe Hershberger,
> 
> [...]
> 
> > While you're touching all these files, why not replace the
> > __u_boot_cmd* implementation with this as well.  There's no need to
> > leave a special case in there as well is there?
> 
> Not in the first stab, it can be indeed done later, but now I'd like to avoid 
> breakage. Besides, replacing u_boot_cmd would break bisectability, I want to 
> preserve it and rather apply more patches slowly than less patches recklessly.

Agreed.  We need to work hard to make sure we do lots of incremental
steps here to make sure breakage can be bisected.  Making use of the
'exec' step in git rebase is very helpful for making sure this is the
case too, btw.

-- 
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/20120924/3f1864fc/attachment.pgp>


More information about the U-Boot mailing list