Re: [PATCH 2/2] Staging: RT2860: Fixed multiple errors in rtusb_io.h

[Greg KH <greg@kroah.com>]

On Sat, Oct 23, 2010 at 08:12:17PM +0100, Neil Munro wrote:
> Fixed the errors in rtusb_io.h

"errors"?  What type of errors?  Do you mean checkpatch.pl
warnings/errors?

> 
> Signed-off-by: Neil Munro <neilmunro@gmail.com>
> ---
>  drivers/staging/rt2860/rtusb_io.h |   44 +++++++++++++++++++-----------------
>  1 files changed, 23 insertions(+), 21 deletions(-)
> 
> diff --git a/drivers/staging/rt2860/rtusb_io.h b/drivers/staging/rt2860/rtusb_io.h
> index 64a2fe4..bbb34d2 100644
> --- a/drivers/staging/rt2860/rtusb_io.h
> +++ b/drivers/staging/rt2860/rtusb_io.h
> @@ -144,42 +144,44 @@ struct rt_cmdq {
> 
>  /* add Client security information into ASIC WCID table and IVEIV table */
>  #define RTMP_STA_SECURITY_INFO_ADD(pAd, apidx, KeyID, pEntry)						\
> -	{	RTMP_STA_ENTRY_MAC_RESET(pAd, pEntry->Aid);								\
> -		if (pEntry->Aid >= 1) {														\
> -			struct rt_set_asic_wcid_attri	SetAsicWcidAttri;								\
> -			SetAsicWcidAttri.WCID = pEntry->Aid;									\
> -			if ((pEntry->AuthMode <= Ndis802_11AuthModeAutoSwitch) &&				\
> -				(pEntry->WepStatus == Ndis802_11Encryption1Enabled))				\
> -			{																		\
> -				SetAsicWcidAttri.Cipher = pAd->SharedKey[apidx][KeyID].CipherAlg;	\
> -			}																		\
> -			else if (pEntry->AuthMode == Ndis802_11AuthModeWPANone)					\
> -			{																		\
> -				SetAsicWcidAttri.Cipher = pAd->SharedKey[apidx][KeyID].CipherAlg;	\
> -			}																		\
> -			else SetAsicWcidAttri.Cipher = 0;										\
> -            DBGPRINT(RT_DEBUG_TRACE, ("aid cipher = %ld\n",SetAsicWcidAttri.Cipher));       \
> +{
> +	RTMP_STA_ENTRY_MAC_RESET(pAd, pEntry->Aid);								\
> +	if (pEntry->Aid >= 1) {														\
> +		struct rt_set_asic_wcid_attri	SetAsicWcidAttri;								\
> +		SetAsicWcidAttri.WCID = pEntry->Aid;									\
> +		if ((pEntry->AuthMode <= Ndis802_11AuthModeAutoSwitch) &&				\

Are you sure this is still all ok?  The tabs look pretty wrong and
the ends of the line should be smaller, right?

If you are going to fix up the spacing, please also fix the length of
the lines at the same time, especially when it is so easy to do so.

Care to resend this?

thanks,

greg k-h