From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from astoria.ccjclearline.com ([64.235.106.9]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1TfyD1-0007Ue-2D for barebox@lists.infradead.org; Tue, 04 Dec 2012 19:31:11 +0000 Date: Tue, 4 Dec 2012 14:30:59 -0500 (EST) From: "Robert P. J. Day" In-Reply-To: Message-ID: References: MIME-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: barebox-bounces@lists.infradead.org Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: lots of inaccurate references to "FileName: include/asm-arm ..." To: "menon.nishanth@gmail.com" Cc: "U-Boot Version 2 (barebox)" On Tue, 4 Dec 2012, menon.nishanth@gmail.com wrote: > On Tue, Dec 4, 2012 at 10:13 AM, Robert P. J. Day wrote: > > > > while roaming around in the arm/omap files, i noticed that numerous > > files have obsolete "FileName:" values, given that stuff that was once > > in "include/asm-arm" has clearly moved, so you find header files like > > arch/arm/mach-omap/include/mach/gpmc_nand.h that contain: > > > > * FileName: include/asm-arm/arch-omap/gpmc_nand.h > > * > > * OMAP's General Purpose Memory Controller (GPMC) has a NAND controller > > * embedded. this file provides the platform data structure required to > > * hook on to it. > > * > > */ > > /* > > * (C) Copyright 2008 > > * Texas Instruments, > > * Nishanth Menon > > * > > * Originally from Linux kernel: > > * http://linux.omap.com/pub/kernel/3430zoom/linux-ldp-v1.3.tar.gz > > * include/asm-arm/arch-omap/nand.h > > > > so the later comment as to the *origin* of the file is fine, but the > > earlier "FileName" value is obviously incorrect. does that matter? > > is that value used for any sort of documentation processing? > > At least when I created the original file in 2008, I think it was > originally from > http://linux.omap.com/pub/kernel/3430zoom/ > and the file there was include/asm-arm/arch-omap/nand.h > > > p.s. you can see the number of these with a simple: > > > > $ grep -r "include/asm-arm" * > > > > since i'm working with a number of the ARM files, i'd be happy to > > "fix" that stuff for some reasonable definition of "fix". :-) > > > if there is a cleanup, awesome.. that would be really be good. the obvious cleanup is to just delete all those "FileName" lines. it *is* useful info to know where a file came from in the linux kernel, i would definitely leave that in. but (as was explained to me recently) it's pointless to have a file contain its own alleged name, especially if files can be moved. so unless those "FileName" lines have some value, i would just remove all of them. rday -- ======================================================================== Robert P. J. Day Ottawa, Ontario, CANADA http://crashcourse.ca Twitter: http://twitter.com/rpjday LinkedIn: http://ca.linkedin.com/in/rpjday ======================================================================== _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox