Hi Scott, On Mon, Aug 9, 2010 at 2:25 PM, Scott Wood <scottw...@freescale.com> wrote: > On Wed, 2 Jun 2010 11:58:39 -0400 > Ben Gardiner <bengardi...@nanometrics.ca> wrote: > >> This patch adds a new 'mtdparts add' variant: add.e (with a synomym add.i). > > Why multiple ways to say the same thing, in a new command with no > legacy to be compatible with?
Just mimicking the add syntax. > Even on the commands where .e and .i used to do something, that's now > the default, and the suffix is unnecessary. So I don't see any need to > mimic the syntax. Ok. No objection here. > What do "e" and "i" even stand for? It looks like they were "for > compatibility with older units" even when first committed. I don't really know. Should I stick with 'add.e' or would you prefer 'add.spread' 'add.skip' or something else? Best Regards, Ben Gardiner --- Nanometrics Inc. http://www.nanometrics.ca _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
