Re: [PATCHv2] staging: wlan-ng: if statement style cleanup

2016-09-17 Thread Greg Kroah-Hartman
On Fri, Sep 16, 2016 at 10:42:21AM -0400, Jon Eyolfson wrote:
> Changed if statement bodies from 2 levels deeper than if statement to 1.
> 
> Signed-off-by: Jon Eyolfson 
> ---
>  drivers/staging/wlan-ng/cfg80211.c | 8 
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/wlan-ng/cfg80211.c 
> b/drivers/staging/wlan-ng/cfg80211.c
> index a36e40d..457f176 100644
> --- a/drivers/staging/wlan-ng/cfg80211.c
> +++ b/drivers/staging/wlan-ng/cfg80211.c
> @@ -476,11 +476,11 @@ static int prism2_connect(struct wiphy *wiphy, struct 
> net_device *dev,
>  
> /* Set the authorization */
> if ((sme->auth_type == NL80211_AUTHTYPE_OPEN_SYSTEM) ||
> -   ((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && !is_wep))
> -   msg_join.authtype.data = 
> P80211ENUM_authalg_opensystem;
> +   ((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && !is_wep))
> +   msg_join.authtype.data = P80211ENUM_authalg_opensystem;
> else if ((sme->auth_type == NL80211_AUTHTYPE_SHARED_KEY) ||
> -   ((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && is_wep))
> -   msg_join.authtype.data = P80211ENUM_authalg_sharedkey;
> +((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && is_wep))
> +   msg_join.authtype.data = P80211ENUM_authalg_sharedkey;
> else
> netdev_warn(dev,
> "Unhandled authorisation type for connect (%d)\n",

Your email client ate the tabs and spit out spaces, making this
impossible to be applied.  Please fix up and resend.

thanks,

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


[PATCHv2] staging: wlan-ng: if statement style cleanup

2016-09-16 Thread Jon Eyolfson
Changed if statement bodies from 2 levels deeper than if statement to 1.

Signed-off-by: Jon Eyolfson 
---
 drivers/staging/wlan-ng/cfg80211.c | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/wlan-ng/cfg80211.c 
b/drivers/staging/wlan-ng/cfg80211.c
index a36e40d..457f176 100644
--- a/drivers/staging/wlan-ng/cfg80211.c
+++ b/drivers/staging/wlan-ng/cfg80211.c
@@ -476,11 +476,11 @@ static int prism2_connect(struct wiphy *wiphy, struct 
net_device *dev,
 
/* Set the authorization */
if ((sme->auth_type == NL80211_AUTHTYPE_OPEN_SYSTEM) ||
-   ((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && !is_wep))
-   msg_join.authtype.data = P80211ENUM_authalg_opensystem;
+   ((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && !is_wep))
+   msg_join.authtype.data = P80211ENUM_authalg_opensystem;
else if ((sme->auth_type == NL80211_AUTHTYPE_SHARED_KEY) ||
-   ((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && is_wep))
-   msg_join.authtype.data = P80211ENUM_authalg_sharedkey;
+((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && is_wep))
+   msg_join.authtype.data = P80211ENUM_authalg_sharedkey;
else
netdev_warn(dev,
"Unhandled authorisation type for connect (%d)\n",
-- 
2.9.3
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel