From mboxrd@z Thu Jan 1 00:00:00 1970 From: Larry Johnson Date: Mon, 29 Oct 2007 16:36:25 -0500 Subject: [U-Boot-Users] [PATCH] PPC4xx: Add Ethernet 1000BASE-X support for PPC4xx In-Reply-To: <20071029202545.32446247F5@gemini.denx.de> References: <20071029202545.32446247F5@gemini.denx.de> Message-ID: <47265259.60705@arlinx.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Wolfgang Denk wrote: > In message <47262967.8070307@arlinx.com> you wrote: >> Would it be better to run Lindent on the original files, post those as >> changes, and then post a second patch with the new material? > > If it's your code, or other original code, then yes, this is the way > to go. > > But if it's code that was "borrowed" from somewhere else (like from > the Linux kernel, busybox, etc.) and then only slightly modified to > make it work under U-Boot, then please *DO* *NOT* reformat it, as > this basicly makes it impossible to update the file from new versions > of the original code. Because, in general, it's hard to know the source of the code, it sounds like the safest thing is for my to use Lindent only to identify problems the patch, and fix them manually. >> BTW, I've noticed that the "Lindent -pcs" format is used inconsistently >> within files. Since I'm also making changes in Linux, it's hard to >> remember whether to type "foo(bar);" or "foo (bar);", maybe others have >> this difficulty, too. I've tried to use the form that is prevalent in >> each individual file, though some are pretty much split down the middle. >> >> How do others feel? > > It should be "foo (bar)". > > And all this should be sufficiently documented at > http://www.denx.de/wiki/UBoot/CodingStyle > > If you feel that details are missing or not clear enought in this > text please help to extend / fix it. The text is clear, I just couldn't decide if this was "a custom more honour'd in the breach than the observance" :-). > Best regards, > > Wolfgang Denk Thanks for your help, I'll re-redo the patch. Best regards, Larry