Re: [PATCH] staging: rtl8192u: Patch to modify if, else conditions

From: Dhere, Chaitanya (C.)
Date: Tue Jun 02 2015 - 01:13:28 EST


On Sun, May 31, 2015 at 10:38:18AM +0900, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
> On Tue, May 19, 2015 at 04:55:57PM +0000, Dhere, Chaitanya (C.) wrote:
> > In this patch:
> > 1. the if condition is modified to remove
> > the unnecessary equality check.
> > 2. The else if condition is replaced by an else condition
> > since the variable checked in the if statement is a bool
> > making the else if unnecessary.
> > 3. Fixed indentation problem on line 1369.
> > These changes were detected with help of coccinelle tool.
>
> 3 different things means you need to send 3 different patches. Please
> only do one logical thing per patch.
>
> thanks,
>
> greg k-h

Hello,
I will make the correction in subject and resend the patch

Thanks,
chaitanya --
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/