[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