[PATCH] Staging: rts5208: Fix code indentation warning as detected by checkpatch.pl
Dan Carpenter
dan.carpenter at oracle.com
Mon Aug 3 12:09:45 UTC 2015
On Mon, Aug 03, 2015 at 04:41:40AM -0700, Joe Perches wrote:
> On Mon, 2015-08-03 at 14:33 +0300, Dan Carpenter wrote:
> > On Mon, Aug 03, 2015 at 11:18:31AM +0000, Shah, Yash (Y.) wrote:
> > > From: Yash Shah <yshah1 at visteon.com>
> > >
> > > Fixed code indentation warning as detected by checkpatch.pl.
> > >
> > > Signed-off-by: Yash Shah <yshah1 at visteon.com>
> >
> > Defines aren't allowed in .c files anyway.
>
> The kernel source tree disagrees with you
>
> $ git grep "^\s*#\s*define\b" -- "*.c" | wc -l
> 175830
Duh. Sorry. I meant ifdefs. I'm sure you can find lots of examples
of those too, but this code would be cleaner without the ifdefs.
regards,
dan carpenter
More information about the devel
mailing list