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

Greg Kroah-Hartman gregkh at linuxfoundation.org
Sat Sep 17 18:41:42 UTC 2016


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 <jon at eyl.io>
> ---
>  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


More information about the devel mailing list