Re: [PATCH 1/4] Staging: wlan-ng: Solved Coding Style issues

[Greg KH <gregkh@linuxfoundation.org>]

On Sun, Sep 16, 2018 at 04:39:11PM -0300, Pablo Pellecchia wrote:
> Solved parenthesis alignment issues throw by checkpatch.pl
> 
> Signed-off-by: Pablo Pellecchia <pablo9891@gmail.com>
> ---
>  drivers/staging/wlan-ng/cfg80211.c | 40 ++++++++++++++++++--------------------
>  1 file changed, 19 insertions(+), 21 deletions(-)
> 
> diff --git a/drivers/staging/wlan-ng/cfg80211.c b/drivers/staging/wlan-ng/cfg80211.c
> index d4cf09b11e33..38bf555a8d3e 100644
> --- a/drivers/staging/wlan-ng/cfg80211.c
> +++ b/drivers/staging/wlan-ng/cfg80211.c
> @@ -234,8 +234,8 @@ static int prism2_set_default_key(struct wiphy *wiphy, struct net_device *dev,
>  	int result = 0;
>  
>  	result = prism2_domibset_uint32(wlandev,
> -		DIDmib_dot11smt_dot11PrivacyTable_dot11WEPDefaultKeyID,
> -		key_index);
> +					DIDmib_dot11smt_dot11PrivacyTable_dot11WEPDefaultKeyID,
> +					key_index);

And now you violate the line length rule :(

Which is why the code looks like it does, what you should really do is
fix up that horridly long variable name.  That will fix this properly.
Can you do that instead?

thanks,

greg k-h