[OPW,kernel,v2,2/2] staging: vt6656: removed braces from single line if-statement
diff mbox

Message ID 1382554212-2697-2-git-send-email-nandu.hgowda@gmail.com
State Accepted
Headers show

Commit Message

nandini h Oct. 23, 2013, 6:50 p.m. UTC
removed braces from if-control flow structs whenever it
contained only one statement to comply with the coding style

Signed-off-by: Nandini Hanumanthagowda <nandu.hgowda@gmail.com>
---
 drivers/staging/vt6656/datarate.c |    9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

Comments

Lisa Nguyen Oct. 24, 2013, 6:21 a.m. UTC | #1
On Wed, Oct 23, 2013 at 11:50 AM, Nandini Hanumanthagowda
<nandu.hgowda@gmail.com> wrote:
> removed braces from if-control flow structs whenever it
> contained only one statement to comply with the coding style
>
> Signed-off-by: Nandini Hanumanthagowda <nandu.hgowda@gmail.com>

Reviewed-by: Lisa Nguyen <lisa@xenapiadmin.com>

> ---
>  drivers/staging/vt6656/datarate.c |    9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/vt6656/datarate.c b/drivers/staging/vt6656/datarate.c
> index 2207198..af9eab0 100644
> --- a/drivers/staging/vt6656/datarate.c
> +++ b/drivers/staging/vt6656/datarate.c
> @@ -274,15 +274,13 @@ void RATEvTxRateFallBack(struct vnt_private *pDevice,
>                 return;
>         }
>
> -       if (psNodeDBTable->uTimeCount >= AUTORATE_TIMEOUT) {
> +       if (psNodeDBTable->uTimeCount >= AUTORATE_TIMEOUT)
>                 psNodeDBTable->uTimeCount = 0;
> -       }
>
>         for (ii = 0; ii < MAX_RATE; ii++) {
>                 if (psNodeDBTable->wSuppRate & (0x0001<<ii)) {
> -                       if (bAutoRate[ii] == true) {
> +                       if (bAutoRate[ii] == true)
>                                 wIdxUpRate = (u16) ii;
> -                       }
>                 } else {
>                         bAutoRate[ii] = false;
>                 }
> @@ -292,9 +290,8 @@ void RATEvTxRateFallBack(struct vnt_private *pDevice,
>                 if ((psNodeDBTable->uTxOk[ii] != 0) ||
>                                 (psNodeDBTable->uTxFail[ii] != 0)) {
>                         dwThroughputTbl[ii] *= psNodeDBTable->uTxOk[ii];
> -                       if (ii < RATE_11M) {
> +                       if (ii < RATE_11M)
>                                 psNodeDBTable->uTxFail[ii] *= 4;
> -                       }
>                         dwThroughputTbl[ii] /= (psNodeDBTable->uTxOk[ii] + psNodeDBTable->uTxFail[ii]);
>                 }
>                 DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"Rate %d,Ok: %d, Fail:%d, Throughput:%d\n",
> --
> 1.7.9.5
>
> --
> You received this message because you are subscribed to the Google Groups "opw-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to opw-kernel+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.

Patch
diff mbox

diff --git a/drivers/staging/vt6656/datarate.c b/drivers/staging/vt6656/datarate.c
index 2207198..af9eab0 100644
--- a/drivers/staging/vt6656/datarate.c
+++ b/drivers/staging/vt6656/datarate.c
@@ -274,15 +274,13 @@  void RATEvTxRateFallBack(struct vnt_private *pDevice,
 		return;
 	}
 
-	if (psNodeDBTable->uTimeCount >= AUTORATE_TIMEOUT) {
+	if (psNodeDBTable->uTimeCount >= AUTORATE_TIMEOUT)
 		psNodeDBTable->uTimeCount = 0;
-	}
 
 	for (ii = 0; ii < MAX_RATE; ii++) {
 		if (psNodeDBTable->wSuppRate & (0x0001<<ii)) {
-			if (bAutoRate[ii] == true) {
+			if (bAutoRate[ii] == true)
 				wIdxUpRate = (u16) ii;
-			}
 		} else {
 			bAutoRate[ii] = false;
 		}
@@ -292,9 +290,8 @@  void RATEvTxRateFallBack(struct vnt_private *pDevice,
 		if ((psNodeDBTable->uTxOk[ii] != 0) ||
 				(psNodeDBTable->uTxFail[ii] != 0)) {
 			dwThroughputTbl[ii] *= psNodeDBTable->uTxOk[ii];
-			if (ii < RATE_11M) {
+			if (ii < RATE_11M)
 				psNodeDBTable->uTxFail[ii] *= 4;
-			}
 			dwThroughputTbl[ii] /= (psNodeDBTable->uTxOk[ii] + psNodeDBTable->uTxFail[ii]);
 		}
 		DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"Rate %d,Ok: %d, Fail:%d, Throughput:%d\n",