[U-Boot-Users] [PATCH] PPC4xx: Add Ethernet 1000BASE-X support for PPC4xx

[Ben Warren <bwarren@qstreams.com>]

Jon Loeliger wrote:
> On Mon, 2007-10-29 at 13:41, Larry Johnson 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?
>>     
>
> In my opinion, yes, please.
>
>   
>> 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);", 
>>     
>
> In my opinion, "foo (bar)" is bad style, and I prefer
> the Linux form, "foo(bar)".
>
> jdl
>
>   
Yeah, in Larry's patch, Lindent mostly created the 'bad-style' format. I 
thought the idea behind Lindent was that you didn't need to figure out 
all the appropriate switches for 'indent'. I don't know what '-pcs' 
does... Of course, my Lindent is old (c 2.6.19) and maybe it's been 
changed since then.

regards,
Ben