[PATCH] staging: ks7010: remove unnecessary brackets in single statement block

[PATCH] staging: ks7010: remove unnecessary brackets in single statement block

[Sergio Paracuellos]

This patch fix the following sparse warning:
    braces {} are not necessary for single statement blocks

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
 drivers/staging/ks7010/ks_hostif.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c
index 05f7be4..c05102d 100644
--- a/drivers/staging/ks7010/ks_hostif.c
+++ b/drivers/staging/ks7010/ks_hostif.c
@@ -1385,9 +1385,8 @@ static __le16 ks_wlan_cap(struct ks_wlan_private *priv)
 {
 	u16 capability = 0x0000;
 
-	if (priv->reg.preamble == SHORT_PREAMBLE) {
+	if (priv->reg.preamble == SHORT_PREAMBLE)
 		capability |= WLAN_CAPABILITY_SHORT_PREAMBLE;
-	}
 
 	capability &= ~(WLAN_CAPABILITY_PBCC);	/* pbcc not support */
 
-- 
2.7.4

Re: [PATCH] staging: ks7010: remove unnecessary brackets in single statement block

[Greg KH]

On Sun, Mar 11, 2018 at 08:22:55AM +0100, Sergio Paracuellos wrote:
> This patch fix the following sparse warning:
>     braces {} are not necessary for single statement blocks

sparse gives you this warning?  Are you sure it's not checkpatch that
does this?

Please fix up and resend.

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel