From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 24 Sep 2012 10:06:11 -0700 Subject: [U-Boot] [PATCH 3/3] common: Add .u_boot_list into all linker files In-Reply-To: <201209241853.19285.marex@denx.de> References: <1348449718-16463-1-git-send-email-marex@denx.de> <1348449718-16463-4-git-send-email-marex@denx.de> <201209241853.19285.marex@denx.de> Message-ID: <20120924170611.GD1797@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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: