From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Mon, 24 Sep 2012 19:09:35 +0200
Subject: [U-Boot] [PATCH 3/3] common: Add .u_boot_list into all linker
	files
In-Reply-To: <20120924170611.GD1797@bill-the-cat>
References: <1348449718-16463-1-git-send-email-marex@denx.de>
	<201209241853.19285.marex@denx.de>
	<20120924170611.GD1797@bill-the-cat>
Message-ID: <201209241909.35433.marex@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 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