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

Marek Vasut marex at denx.de
Mon Sep 24 19:09:35 CEST 2012


Dear Tom Rini,

> 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.

+1 (actually +inf.)

> Making use of the
> 'exec' step in git rebase is very helpful for making sure this is the
> case too, btw.

exec step ?

Best regards,
Marek Vasut


More information about the U-Boot mailing list