Re: [PATCH] staging: sm750fb: braces, indents, spaces fix

From: Sudip Mukherjee
Date: Fri Mar 13 2015 - 01:54:49 EST


On Thu, Mar 12, 2015 at 10:14:36PM -0700, Ragavendra BN wrote:
> This patch removes the braces for the single line if statement. It fixes
> the indent positions correctly. It fixes the spaces appropriately making the code give no
> warnings by the checpath.pl script check.

your From: is still having [user] in your email address.
and you are doing too many things in a single patch. one patch should
do only one type of change.

>
>
> Signed-off-by: Ragavendra BN <ragavendra.bn@xxxxxxxxx>
>
this extra blank line is not necessary here.

regards
sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/