lkml.org 
[lkml]   [2012]   [Jun]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 06/17] i2c: omap: improve 1p153 errata handling
On Thu, Jun 14, 2012 at 01:20:39PM +0300, Felipe Balbi wrote:
> -static int errata_omap3_1p153(struct omap_i2c_dev *dev, u16 *stat, int *err)
> +static int errata_omap3_1p153(struct omap_i2c_dev *dev)
> {
> - unsigned long timeout = 10000;
> + unsigned long timeout = 10000;
> + u16 stat;

Eww, no, not more of this "lets add tabs to align auto variable
declarations". This is detrimental when you add another variable whose
type is longer than the current indentation - you have to reformat the
entire block.

It's really not worth it. Stick to just using one space, like the rest
of the kernel code. Like the code which Linus writes. And thereby help
to avoid future "pointless churn" whinges.


\
 
 \ /
  Last update: 2012-06-14 14:01    [W:1.286 / U:0.680 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site