lkml.org 
[lkml]   [2010]   [Nov]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2] Staging: RT2860: Fixed multiple errors in rtusb_io.h
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


\
 
 \ /
  Last update: 2010-11-09 22:39    [W:0.036 / U:0.336 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site