[U-Boot] [PATCH v4 0/3] mtd, ubi, ubifs: resync with Linux-3.14

Scott Wood scottwood at freescale.com
Mon Jun 30 22:20:36 CEST 2014


On Thu, 2014-06-26 at 09:51 +0200, Wolfgang Denk wrote:
> Dear Scott,
> 
> In message <1403637570.26908.38.camel at snotra.buserror.net> you wrote:
> >
> > I still disagree with #ifndef __UBOOT__ as it will hurt more than it
> > helps with future merges.
> 
> I agree that #ifdef's should be avoided, but then here they also serve
> a documentation purpose as they clearly mark areas of code that are
> specific to U-Boot, or that are not used in U-Boot.

git diff can do that too, without reducing readability or increasing the
likelihood of mismerges.

> Do you think just of a removal, so there will be no trace left of
> which code was added for U-Boot, and which was removed? Or should
> these be turned into comments, something like
> 
> 	/* BEGIN U-Boot specific code */
> 	...
> 	/* END U-Boot specific code */
> 
> [but that would not handle code removal well].
> 
> So what exactly is your proposal how the #fdef's should be removed?

My proposal is to just remove them, similar to how the NAND code has
been handled.

-Scott




More information about the U-Boot mailing list