From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tom <Tom.Rix@windriver.com>
Date: Sun, 27 Sep 2009 06:56:37 -0500
Subject: [U-Boot] [PATCH] OMAP3: Clean up whitespace in mux configs
In-Reply-To: <4ABF2376.5080408@googlemail.com>
References: <20090922225314.GA16136@lixom.net>
	<4ABF2376.5080408@googlemail.com>
Message-ID: <4ABF52F5.4010804@windriver.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dirk Behme wrote:
> Olof Johansson wrote:
>> Switch from space-based indentation to tab-based in mux configs, as pointed
>> out by WD at:
> 
> You have only two options here: Violate space/tab rule or 80 char 
> rule. We had this discussion with Wolfgang already in the past ;)
> 
> Last time we had:
> 
> ...
> [So Jan 25 2009] [18:50:28] <dirk2>	wdenk: Regarding your code 
> comments in 
> http://lists.denx.de/pipermail/u-boot/2009-January/046090.html:
> [So Jan 25 2009] [18:51:34] <dirk2>	wdenk: Regarding MUX indent: 
> normally yes, but ...
> [So Jan 25 2009] [18:51:59] <wdenk_>	...you have too long lines, then. 
> I know
> [So Jan 25 2009] [18:52:49] <dirk2>	wdenk: ... we tried several 
> formatting options, and jkridner converted one MUX file with beginning 
> TABs. The result wasn't readable better
> [So Jan 25 2009] [18:53:34] <wdenk_>	I'll not fight about this one.
> ...
> 
> So last time we stayed with space instead of tab and have 80 char fine.
> 
> If we want to switch now to break 80 chars (and this patch does so) 
> but have no spaces any more, I'm fine with it. Just want to give some 
> history.
> 
> Best regards
> 

 From reviewing the patch, it looks like the mux args line up better.
So it will make scanning them easier on the developer.

Tom


> Dirk
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot