On Fri, 12 Dec 2014 07:19:21 +0100 Heiko Schocher <h...@denx.de> wrote:
> Hello Scott, > > Am 11.12.2014 22:43, schrieb Scott Wood: > > On Thu, 2014-12-11 at 22:37 +0100, Marek Vasut wrote: > >> On Thursday, December 11, 2014 at 09:37:10 PM, Scott Wood wrote: > >>> On Thu, 2014-12-11 at 19:49 +0900, Masahiro Yamada wrote: > >>>> Since commit ff94bc40af34 (mtd, ubi, ubifs: resync with Linux-3.14), > >>>> the "nand scrub" command has not been working. > >>>> > >>>> The scrub is a U-Boot extension and we have to modify nand_base.c > >>>> that originates in Linux. > >>>> > >>>> Mark the code with #ifdef __UBOOT__ so we will never accidentally > >>>> drop it when we re-sync the NAND framework with Linux in the future. > >>> > >>> No more "#ifdef __UBOOT__" please. > >> > >> Do you happen to have a helpful suggestion how to clearly mark those bits > >> of > >> code then please ? > > > > This was already discussed. :-) > > > > See the archives for why I think this is bad. > > > >>> Instead, never again do a "start > >>> from scratch" resync the way that the above commit was done. > >> > >> This was already discussed, no need to revive this topic here now. > > > > Sorry, but these patches fixing breakages that resulted from that merge > > demonstrate that there is a need to revive it, if there's anyone that > > still thinks it's a good idea -- Heiko seemed to be in agreement that > > there's no need to do that for future syncs: > > http://lists.denx.de/pipermail/u-boot/2014-November/194256.html > > Yes, I hope a resync works now fine ... but I prefer to mark the > differences between linux and u-boot somehow, because, you immediately > see the differences between linux and u-boot, when you read the u-boot > code ... > I agree with Heiko. Best Regards Masahiro Yamada _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot