staging: rtl8192e: Coding style changes
PATCH v4 1/1 staging: rtl8192e: Coding style changes
five patches with simple coding style changes to the file drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
The last two patches should probably have been combined as both dealt with the addition or removal of blank lines, but I missed one style issue on the first pass. In that situation I guess I should possibly roll back the previous commit and re-edit?
PATCH v4 1/5 staging: rtl8192e: Add spaces where required by coding standard
There were numerous coding syle errors in this file where spaces were required around operators.
--- .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 126 ++++++++++----------- 1 file changed, 63 insertions(+), 63 deletions(-)
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c +63 -63
@@ -60,8 +60,8 @@ void HTUpdateDefaultSetting(struct ieee80211_device *ieee)//printk("pHTinfo:%p, &pHTinfo:%p, mptr:%p, offsetof:%x\n", pHTInfo, &pHTInfo, __mptr, offsetof(struct ieee80211_device, pHTInfo)); //printk("===>ieee:%p,\n", ieee); // ShortGI support - pHTInfo->bRegShortGI20MHz=1 ; - pHTInfo->bRegShortGI40MHz=1 ; + pHTInfo->bRegShortGI20MHz =1 ; + pHTInfo->bRegShortGI40MHz =1 ;// 40MHz channel support pHTInfo->bRegBW40MHz =1 ;
@@ -138,12 +138,12 @@ void HTDebugHTCapability(u8 *CapIE, u8 *TitleString )IEEE80211_DEBUG (IEEE80211_DL_HT ,"<Log HT Capability>. Called by %s\n" , TitleString ); -IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupported Channel Width = %s\n" , (pCapELE->ChlWidth)?"20MHz" :"20/40MHz" ); -IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupport Short GI for 20M = %s\n" , (pCapELE->ShortGI20Mhz)?"YES" :"NO" ); -IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupport Short GI for 40M = %s\n" , (pCapELE->ShortGI40Mhz)?"YES" :"NO" ); -IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupport TX STBC = %s\n" , (pCapELE->TxSTBC)?"YES" :"NO" ); -IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tMax AMSDU Size = %s\n" , (pCapELE->MaxAMSDUSize)?"3839" :"7935" ); -IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupport CCK in 20/40 mode = %s\n" , (pCapELE->DssCCk)?"YES" :"NO" ); +IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupported Channel Width = %s\n" , (pCapELE->ChlWidth) ?"20MHz" :"20/40MHz" ); +IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupport Short GI for 20M = %s\n" , (pCapELE->ShortGI20Mhz) ?"YES" :"NO" ); +IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupport Short GI for 40M = %s\n" , (pCapELE->ShortGI40Mhz) ?"YES" :"NO" ); +IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupport TX STBC = %s\n" , (pCapELE->TxSTBC) ?"YES" :"NO" ); +IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tMax AMSDU Size = %s\n" , (pCapELE->MaxAMSDUSize) ?"3839" :"7935" ); +IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupport CCK in 20/40 mode = %s\n" , (pCapELE->DssCCk) ?"YES" :"NO" );IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tMax AMPDU Factor = %d\n" , pCapELE->MaxRxAMPDUFactor);IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tMPDU Density = %d\n" , pCapELE->MPDUDensity); IEEE80211_DEBUG(IEEE80211_DL_HT, "\tMCS Rate Set = [%x][%x][%x][%x][%x]\n", pCapELE->MCS[0],\
@@ -193,7 +193,7 @@ void HTDebugHTInfo(u8 *InfoIE, u8 *TitleString)IEEE80211_DEBUG (IEEE80211_DL_HT ,"Lower Channel\n" );break ; } -IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tRecommended channel width = %s\n" , (pHTInfoEle->RecommemdedTxWidth)?"20Mhz" :"40Mhz" ); +IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tRecommended channel width = %s\n" , (pHTInfoEle->RecommemdedTxWidth) ?"20Mhz" :"40Mhz" );IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tOperation mode for protection = " ); switch (pHTInfoEle->OptMode)
@@ -272,8 +272,8 @@ u16 HTHalfMcsToDataRate(struct ieee80211_device *ieee, u8 nMcsRate)u8 is40MHz;u8 isShortGI; - is40MHz = (IsHTHalfNmode40Bandwidth (ieee))?1 :0 ; - isShortGI = (IsHTHalfNmodeSGI (ieee, is40MHz))?1 :0 ; + is40MHz = (IsHTHalfNmode40Bandwidth (ieee)) ?1 :0 ; + isShortGI = (IsHTHalfNmodeSGI (ieee, is40MHz)) ?1 :0 ;return MCS_DATA_RATE [is40MHz][isShortGI][(nMcsRate&0x7f )]; }
@@ -283,10 +283,10 @@ u16 HTMcsToDataRate(struct ieee80211_device *ieee, u8 nMcsRate) {PRT_HIGH_THROUGHPUT pHTInfo = ieee->pHTInfo; -u8 is40MHz = (pHTInfo->bCurBW40MHz)?1 :0 ; -u8 isShortGI = (pHTInfo->bCurBW40MHz)? - ((pHTInfo->bCurShortGI40MHz)?1 :0 ): - ((pHTInfo->bCurShortGI20MHz)?1 :0 ); +u8 is40MHz = (pHTInfo->bCurBW40MHz) ?1 :0 ; +u8 isShortGI = (pHTInfo->bCurBW40MHz) ? + ((pHTInfo->bCurShortGI40MHz) ?1 :0 ) : + ((pHTInfo->bCurShortGI20MHz) ?1 :0 );return MCS_DATA_RATE [is40MHz][isShortGI][(nMcsRate&0x7f )]; }
@@ -318,7 +318,7 @@ u16 TxCountToDataRate(struct ieee80211_device *ieee, u8 nDataRate)// nDataRate = nDataRate - 12; } -else if (nDataRate >=0x20 && nDataRate <=0x2f )//(27, 44) +else if (nDataRate >=0x20 && nDataRate <=0x2f )//(27, 44) { is40MHz =1 ; isShortGI =0 ;
@@ -349,17 +349,17 @@ bool IsHTHalfNmodeAPs(struct ieee80211_device *ieee) {bool retValue = false;struct ieee80211_network *net = &ieee->current_network; -if ((memcmp (net->bssid,BELKINF5D8233V1_RALINK ,3 )==0 ) || - (memcmp (net->bssid,BELKINF5D82334V3_RALINK ,3 )==0 ) || - (memcmp (net->bssid,PCI_RALINK ,3 )==0 ) || - (memcmp (net->bssid,EDIMAX_RALINK ,3 )==0 ) || - (memcmp (net->bssid,AIRLINK_RALINK ,3 )==0 ) || +if ((memcmp (net->bssid,BELKINF5D8233V1_RALINK ,3 ) ==0 ) || + (memcmp (net->bssid,BELKINF5D82334V3_RALINK ,3 ) ==0 ) || + (memcmp (net->bssid,PCI_RALINK ,3 ) ==0 ) || + (memcmp (net->bssid,EDIMAX_RALINK ,3 ) ==0 ) || + (memcmp (net->bssid,AIRLINK_RALINK ,3 ) ==0 ) || (net->ralink_cap_exist)) retValue = true; -else if ((memcmp (net->bssid,UNKNOWN_BORADCOM ,3 )==0 ) || - (memcmp (net->bssid,LINKSYSWRT330_LINKSYSWRT300_BROADCOM ,3 )==0 )|| - (memcmp (net->bssid,LINKSYSWRT350_LINKSYSWRT150_BROADCOM ,3 )==0 )|| - (memcmp (net->bssid, NETGEAR834Bv2_BROADCOM,3 )==0 ) || +else if ((memcmp (net->bssid,UNKNOWN_BORADCOM ,3 ) ==0 ) || + (memcmp (net->bssid,LINKSYSWRT330_LINKSYSWRT300_BROADCOM ,3 ) ==0 ) || + (memcmp (net->bssid,LINKSYSWRT350_LINKSYSWRT150_BROADCOM ,3 ) ==0 ) || + (memcmp (net->bssid, NETGEAR834Bv2_BROADCOM,3 ) ==0 ) || (net->broadcom_cap_exist)) retValue = true;else if (net->bssht.bdRT2RTAggregation)
@@ -385,21 +385,21 @@ static void HTIOTPeerDetermine(struct ieee80211_device *ieee) pHTInfo->IOTPeer =HT_IOT_PEER_REALTEK ; else if (net->broadcom_cap_exist) pHTInfo->IOTPeer = HT_IOT_PEER_BROADCOM; - else if ((memcmp(net->bssid, UNKNOWN_BORADCOM, 3)==0) || - (memcmp(net->bssid, LINKSYSWRT330_LINKSYSWRT300_BROADCOM, 3)==0)|| - (memcmp(net->bssid, LINKSYSWRT350_LINKSYSWRT150_BROADCOM, 3)==0)|| - (memcmp(net->bssid, NETGEAR834Bv2_BROADCOM, 3)==0) ) + else if ((memcmp(net->bssid, UNKNOWN_BORADCOM, 3) == 0) || + (memcmp(net->bssid, LINKSYSWRT330_LINKSYSWRT300_BROADCOM, 3) == 0) || + (memcmp(net->bssid, LINKSYSWRT350_LINKSYSWRT150_BROADCOM, 3) == 0) || + (memcmp(net->bssid, NETGEAR834Bv2_BROADCOM, 3) == 0) ) pHTInfo->IOTPeer = HT_IOT_PEER_BROADCOM; - else if ((memcmp(net->bssid, BELKINF5D8233V1_RALINK, 3)==0) || - (memcmp(net->bssid, BELKINF5D82334V3_RALINK, 3)==0) || - (memcmp(net->bssid, PCI_RALINK, 3)==0) || - (memcmp(net->bssid, EDIMAX_RALINK, 3)==0) || - (memcmp(net->bssid, AIRLINK_RALINK, 3)==0) || + else if ((memcmp(net->bssid, BELKINF5D8233V1_RALINK, 3) == 0) || + (memcmp(net->bssid, BELKINF5D82334V3_RALINK, 3) == 0) || + (memcmp(net->bssid, PCI_RALINK, 3) == 0) || + (memcmp(net->bssid, EDIMAX_RALINK, 3) == 0) || + (memcmp(net->bssid, AIRLINK_RALINK, 3) == 0) || net->ralink_cap_exist) pHTInfo->IOTPeer = HT_IOT_PEER_RALINK; else if (net->atheros_cap_exist) pHTInfo->IOTPeer = HT_IOT_PEER_ATHEROS; - else if (memcmp(net->bssid, CISCO_BROADCOM, 3)==0) + else if (memcmp(net->bssid, CISCO_BROADCOM, 3) == 0) pHTInfo->IOTPeer = HT_IOT_PEER_CISCO; else pHTInfo->IOTPeer = HT_IOT_PEER_UNKNOWN;
@@ -438,7 +438,7 @@ static bool HTIOTActIsDisableMCS15(struct ieee80211_device *ieee)#ifdef TODO // Apply for 819u only -#if (HAL_CODE_BASE ==RTL8192 ) +#if (HAL_CODE_BASE ==RTL8192 )#if (DEV_BUS_TYPE ==USB_INTERFACE )// Alway disable MCS15 by Jerry Chang's request.by Emily, 2008.04.15
@@ -515,9 +515,9 @@ static u8 HTIOTActIsMgntUseCCK6M(struct ieee80211_network *network)static u8 HTIOTActIsCCDFsync (u8 *PeerMacAddr) {u8 retValue =0 ; -if ((memcmp (PeerMacAddr,UNKNOWN_BORADCOM ,3 )==0 ) || - (memcmp (PeerMacAddr,LINKSYSWRT330_LINKSYSWRT300_BROADCOM ,3 )==0 ) || - (memcmp (PeerMacAddr,LINKSYSWRT350_LINKSYSWRT150_BROADCOM ,3 ) ==0 )) +if ((memcmp (PeerMacAddr,UNKNOWN_BORADCOM ,3 ) ==0 ) || + (memcmp (PeerMacAddr,LINKSYSWRT330_LINKSYSWRT300_BROADCOM ,3 ) ==0 ) || + (memcmp (PeerMacAddr,LINKSYSWRT350_LINKSYSWRT150_BROADCOM ,3 ) ==0 )) { retValue =1 ; }
@@ -575,7 +575,7 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, u }else { - pCapELE->ChlWidth = (pHT->bRegBW40MHz?1 :0 ); + pCapELE->ChlWidth = (pHT->bRegBW40MHz ?1 :0 ); }// pCapELE->ChlWidth = (pHT->bRegBW40MHz?1:0);
@@ -588,8 +588,8 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, u pCapELE->TxSTBC =1 ; pCapELE->RxSTBC =0 ; pCapELE->DelayBA =0 ;// Do not support now!! - pCapELE->MaxAMSDUSize = (MAX_RECEIVE_BUFFER_SIZE >=7935 )?1 :0 ; - pCapELE->DssCCk = ((pHT->bRegBW40MHz)?(pHT->bRegSuppCCK?1 :0 ):0 ); + pCapELE->MaxAMSDUSize = (MAX_RECEIVE_BUFFER_SIZE >=7935 ) ?1 :0 ; + pCapELE->DssCCk = ((pHT->bRegBW40MHz) ? (pHT->bRegSuppCCK ?1 :0 ) :0 ); pCapELE->PSMP =0 ;// Do not support now!! pCapELE->LSigTxopProtect =0 ;// Do not support now!!
@@ -600,11 +600,11 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, uif (IsEncrypt) { pCapELE->MPDUDensity =7 ;// 8us - pCapELE->MaxRxAMPDUFactor =2 ;// 2 is for 32 K and 3 is 64K + pCapELE->MaxRxAMPDUFactor =2 ;// 2 is for 32 K and 3 is 64K }else { - pCapELE->MaxRxAMPDUFactor =3 ;// 2 is for 32 K and 3 is 64K + pCapELE->MaxRxAMPDUFactor =3 ;// 2 is for 32 K and 3 is 64K pCapELE->MPDUDensity =0 ;// no density }
@@ -624,7 +624,7 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, uif (ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev)) {int i; -for (i =1 ; i<16 ; i++) +for (i =1 ; i <16 ; i++) pCapELE->MCS[i] =0 ; }
@@ -676,9 +676,9 @@ void HTConstructInfoElement(struct ieee80211_device *ieee, u8 *posHTInfo, u8 *leif ( (ieee->iw_mode ==IW_MODE_ADHOC ) || (ieee->iw_mode ==IW_MODE_MASTER ))//ap mode is not currently supported { pHTInfoEle->ControlChl = ieee->current_network.channel; - pHTInfoEle->ExtChlOffset = ((!pHT->bRegBW40MHz)?HT_EXTCHNL_OFFSET_NO_EXT : - (ieee->current_network.channel<=6 )? -HT_EXTCHNL_OFFSET_UPPER :HT_EXTCHNL_OFFSET_LOWER ); + pHTInfoEle->ExtChlOffset = ((!pHT->bRegBW40MHz) ?HT_EXTCHNL_OFFSET_NO_EXT : + (ieee->current_network.channel <=6 ) ? +HT_EXTCHNL_OFFSET_UPPER :HT_EXTCHNL_OFFSET_LOWER ); pHTInfoEle->RecommemdedTxWidth = pHT->bRegBW40MHz; pHTInfoEle->RIFS =0 ; pHTInfoEle->PSMPAccessOnly =0 ;
@@ -805,7 +805,7 @@ static u8 HT_PickMCSRate(struct ieee80211_device *ieee, u8 *pOperateMCS)//legacy rate routine handled at selectedrate //no MCS rate -for (i=0 ;i<=15 ;i++){ +for (i =0 ; i <=15 ; i++){ pOperateMCS[i] =0 ; }break ;
@@ -817,9 +817,9 @@ static u8 HT_PickMCSRate(struct ieee80211_device *ieee, u8 *pOperateMCS)//HT part // TODO: may be different if we have different number of antenna - pOperateMCS[0 ] &=RATE_ADPT_1SS_MASK ;//support MCS 0~7 - pOperateMCS[1 ] &=RATE_ADPT_2SS_MASK ; - pOperateMCS[3 ] &=RATE_ADPT_MCS32_MASK ; + pOperateMCS[0 ] &=RATE_ADPT_1SS_MASK ;//support MCS 0~7 + pOperateMCS[1 ] &=RATE_ADPT_2SS_MASK ; + pOperateMCS[3 ] &=RATE_ADPT_MCS32_MASK ;break ;//should never reach here
@@ -862,7 +862,7 @@ u8 HTGetHighestMCSRate(struct ieee80211_device *ieee, u8 *pMCSRateSet, u8 *pMCSFIEEE80211_DEBUG (IEEE80211_DL_ERR ,"pMCSRateSet or pMCSFilter can't be null in HTGetHighestMCSRate()\n" );return false; } -for (i=0 ; i<16 ; i++) +for (i =0 ; i <16 ; i++) availableMcsRate[i] = pMCSRateSet[i] & pMCSFilter[i];for (i =0 ; i <16 ; i++)
@@ -907,10 +907,10 @@ static u8 HTFilterMCSRate(struct ieee80211_device *ieee, u8 *pSupportMCS, u8 *pOperateMCS) { -u8 i=0 ; +u8 i =0 ;// filter out operational rate set not supported by AP, the length of it is 16 -for (i=0 ;i<=15 ;i++){ +for (i =0 ; i <=15 ; i++){ pOperateMCS[i] = ieee->Regdot11HTOperationalRateSet[i]&pSupportMCS[i]; }
@@ -929,7 +929,7 @@ static u8 HTFilterMCSRate(struct ieee80211_device *ieee, u8 *pSupportMCS,// For RTL819X, we support only MCS0~15. // And also, we do not know how to use MCS32 now. // -for (i=2 ; i<=15 ; i++) +for (i =2 ; i <=15 ; i++) pOperateMCS[i] =0 ;return true;
@@ -1006,7 +1006,7 @@ void HTOnAssocRsp(struct ieee80211_device *ieee)// pHTInfo->bCurrent_AMSDU_Support = pHTInfo->bAMSDU_Support; - nMaxAMSDUSize = (pPeerHTCap->MaxAMSDUSize==0 )?3839 :7935 ; + nMaxAMSDUSize = (pPeerHTCap->MaxAMSDUSize ==0 ) ?3839 :7935 ;if (pHTInfo->nAMSDU_MaxSize > nMaxAMSDUSize ) pHTInfo->nCurrent_AMSDU_MaxSize = nMaxAMSDUSize;
@@ -1196,7 +1196,7 @@ void HTInitializeBssDesc(PBSS_HT pBssHT)memset (pBssHT->bdHTInfoBuf,0 ,sizeof (pBssHT->bdHTInfoBuf)); pBssHT->bdHTInfoLen =0 ; - pBssHT->bdHTSpecVer=HT_SPEC_VER_IEEE ; + pBssHT->bdHTSpecVer =HT_SPEC_VER_IEEE ; pBssHT->bdRT2RTAggregation = false; pBssHT->bdRT2RTLongSlotTime = false;
@@ -1357,12 +1357,12 @@ void HTSetConnectBwMode(struct ieee80211_device *ieee, HT_CHANNEL_WIDTH Bandwidtreturn ; }//if in half N mode, set to 20M bandwidth please 09.08.2008 WB. -if (Bandwidth==HT_CHANNEL_WIDTH_20_40 && (!ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev))) +if (Bandwidth ==HT_CHANNEL_WIDTH_20_40 && (!ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev))) {// Handle Illegal extension channel offset!! -if (ieee->current_network.channel<2 && Offset==HT_EXTCHNL_OFFSET_LOWER ) +if (ieee->current_network.channel <2 && Offset ==HT_EXTCHNL_OFFSET_LOWER ) Offset =HT_EXTCHNL_OFFSET_NO_EXT ; -if (Offset==HT_EXTCHNL_OFFSET_UPPER || Offset==HT_EXTCHNL_OFFSET_LOWER ) { +if (Offset ==HT_EXTCHNL_OFFSET_UPPER || Offset ==HT_EXTCHNL_OFFSET_LOWER ) { pHTInfo->bCurBW40MHz = true; pHTInfo->CurSTAExtChnlOffset = Offset; }else {
@@ -1393,9 +1393,9 @@ void HTSetConnectBwModeCallback(struct ieee80211_device *ieee)if (pHTInfo->bCurBW40MHz) { -if (pHTInfo->CurSTAExtChnlOffset==HT_EXTCHNL_OFFSET_UPPER ) +if (pHTInfo->CurSTAExtChnlOffset ==HT_EXTCHNL_OFFSET_UPPER ) ieee->set_chan (ieee->dev, ieee->current_network.channel+2 ); -else if (pHTInfo->CurSTAExtChnlOffset==HT_EXTCHNL_OFFSET_LOWER ) +else if (pHTInfo->CurSTAExtChnlOffset ==HT_EXTCHNL_OFFSET_LOWER ) ieee->set_chan (ieee->dev, ieee->current_network.channel-2 );else ieee->set_chan (ieee->dev, ieee->current_network.channel);
Signature · 1 line
2.16.3
PATCH v4 2/5 staging: rtl8192e: correct position of '{', '}', '(' and ')' - coding style
In reply to John WhitmoreCorrect the coding style of parenthesis and braces in various code blocks
--- .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 217 ++++++++------------- 1 file changed, 79 insertions(+), 138 deletions(-)
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c +79 -138
@@ -7,28 +7,28 @@ u8 MCS_FILTER_ALL[16] = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0u8 MCS_FILTER_1SS [16 ] = {0xff ,0x00 ,0x00 ,0x01 ,0x00 ,0x00 ,0x00 ,0x00 ,0x00 ,0x00 ,0x00 ,0x00 ,0x00 ,0x00 ,0x00 ,0x00 }; -u16 MCS_DATA_RATE [2 ][2 ][77 ] = - { { {13 ,26 ,39 ,52 ,78 ,104 ,117 ,130 ,26 ,52 ,78 ,104 ,156 ,208 ,234 ,260 , -39 ,78 ,117 ,234 ,312 ,351 ,390 ,52 ,104 ,156 ,208 ,312 ,416 ,468 ,520 , -0 ,78 ,104 ,130 ,117 ,156 ,195 ,104 ,130 ,130 ,156 ,182 ,182 ,208 ,156 ,195 , -195 ,234 ,273 ,273 ,312 ,130 ,156 ,181 ,156 ,181 ,208 ,234 ,208 ,234 ,260 ,260 , -286 ,195 ,234 ,273 ,234 ,273 ,312 ,351 ,312 ,351 ,390 ,390 ,429 },// Long GI, 20MHz - {14 ,29 ,43 ,58 ,87 ,116 ,130 ,144 ,29 ,58 ,87 ,116 ,173 ,231 ,260 ,289 , -43 ,87 ,130 ,173 ,260 ,347 ,390 ,433 ,58 ,116 ,173 ,231 ,347 ,462 ,520 ,578 , -0 ,87 ,116 ,144 ,130 ,173 ,217 ,116 ,144 ,144 ,173 ,202 ,202 ,231 ,173 ,217 , -217 ,260 ,303 ,303 ,347 ,144 ,173 ,202 ,173 ,202 ,231 ,260 ,231 ,260 ,289 ,289 , -318 ,217 ,260 ,303 ,260 ,303 ,347 ,390 ,347 ,390 ,433 ,433 ,477 } },// Short GI, 20MHz - { {27 ,54 ,81 ,108 ,162 ,216 ,243 ,270 ,54 ,108 ,162 ,216 ,324 ,432 ,486 ,540 , -81 ,162 ,243 ,324 ,486 ,648 ,729 ,810 ,108 ,216 ,324 ,432 ,648 ,864 ,972 ,1080 , -12 ,162 ,216 ,270 ,243 ,324 ,405 ,216 ,270 ,270 ,324 ,378 ,378 ,432 ,324 ,405 , -405 ,486 ,567 ,567 ,648 ,270 ,324 ,378 ,324 ,378 ,432 ,486 ,432 ,486 ,540 ,540 , -594 ,405 ,486 ,567 ,486 ,567 ,648 ,729 ,648 ,729 ,810 ,810 ,891 },// Long GI, 40MHz - {30 ,60 ,90 ,120 ,180 ,240 ,270 ,300 ,60 ,120 ,180 ,240 ,360 ,480 ,540 ,600 , -90 ,180 ,270 ,360 ,540 ,720 ,810 ,900 ,120 ,240 ,360 ,480 ,720 ,960 ,1080 ,1200 , -13 ,180 ,240 ,300 ,270 ,360 ,450 ,240 ,300 ,300 ,360 ,420 ,420 ,480 ,360 ,450 , -450 ,540 ,630 ,630 ,720 ,300 ,360 ,420 ,360 ,420 ,480 ,540 ,480 ,540 ,600 ,600 , -660 ,450 ,540 ,630 ,540 ,630 ,720 ,810 ,720 ,810 ,900 ,900 ,990 } }// Short GI, 40MHz - }; +u16 MCS_DATA_RATE [2 ][2 ][77 ] = { + { {13 ,26 ,39 ,52 ,78 ,104 ,117 ,130 ,26 ,52 ,78 ,104 ,156 ,208 ,234 ,260 , +39 ,78 ,117 ,234 ,312 ,351 ,390 ,52 ,104 ,156 ,208 ,312 ,416 ,468 ,520 , +0 ,78 ,104 ,130 ,117 ,156 ,195 ,104 ,130 ,130 ,156 ,182 ,182 ,208 ,156 ,195 , +195 ,234 ,273 ,273 ,312 ,130 ,156 ,181 ,156 ,181 ,208 ,234 ,208 ,234 ,260 ,260 , +286 ,195 ,234 ,273 ,234 ,273 ,312 ,351 ,312 ,351 ,390 ,390 ,429 },// Long GI, 20MHz + {14 ,29 ,43 ,58 ,87 ,116 ,130 ,144 ,29 ,58 ,87 ,116 ,173 ,231 ,260 ,289 , +43 ,87 ,130 ,173 ,260 ,347 ,390 ,433 ,58 ,116 ,173 ,231 ,347 ,462 ,520 ,578 , +0 ,87 ,116 ,144 ,130 ,173 ,217 ,116 ,144 ,144 ,173 ,202 ,202 ,231 ,173 ,217 , +217 ,260 ,303 ,303 ,347 ,144 ,173 ,202 ,173 ,202 ,231 ,260 ,231 ,260 ,289 ,289 , +318 ,217 ,260 ,303 ,260 ,303 ,347 ,390 ,347 ,390 ,433 ,433 ,477 } },// Short GI, 20MHz + { {27 ,54 ,81 ,108 ,162 ,216 ,243 ,270 ,54 ,108 ,162 ,216 ,324 ,432 ,486 ,540 , +81 ,162 ,243 ,324 ,486 ,648 ,729 ,810 ,108 ,216 ,324 ,432 ,648 ,864 ,972 ,1080 , +12 ,162 ,216 ,270 ,243 ,324 ,405 ,216 ,270 ,270 ,324 ,378 ,378 ,432 ,324 ,405 , +405 ,486 ,567 ,567 ,648 ,270 ,324 ,378 ,324 ,378 ,432 ,486 ,432 ,486 ,540 ,540 , +594 ,405 ,486 ,567 ,486 ,567 ,648 ,729 ,648 ,729 ,810 ,810 ,891 },// Long GI, 40MHz + {30 ,60 ,90 ,120 ,180 ,240 ,270 ,300 ,60 ,120 ,180 ,240 ,360 ,480 ,540 ,600 , +90 ,180 ,270 ,360 ,540 ,720 ,810 ,900 ,120 ,240 ,360 ,480 ,720 ,960 ,1080 ,1200 , +13 ,180 ,240 ,300 ,270 ,360 ,450 ,240 ,300 ,300 ,360 ,420 ,420 ,480 ,360 ,450 , +450 ,540 ,630 ,630 ,720 ,300 ,360 ,420 ,360 ,420 ,480 ,540 ,480 ,540 ,600 ,600 , +660 ,450 ,540 ,630 ,540 ,630 ,720 ,810 ,720 ,810 ,900 ,900 ,990 } }// Short GI, 40MHz +};static u8 UNKNOWN_BORADCOM [3 ] = {0x00 ,0x14 ,0xbf };static u8 LINKSYSWRT330_LINKSYSWRT300_BROADCOM [3 ] = {0x00 ,0x1a ,0x70 };
@@ -122,21 +122,20 @@ void HTUpdateDefaultSetting(struct ieee80211_device *ieee)* return: none * notice: Driver should not print out this message by default. * *****************************************************************************************************************/ -void HTDebugHTCapability (u8 *CapIE,u8 *TitleString ) +void HTDebugHTCapability (u8 *CapIE,u8 *TitleString) {static u8 EWC11NHTCap[] = {0x00 ,0x90 ,0x4c ,0x33 };// For 11n EWC definition, 2007.07.17, by Emily PHT_CAPABILITY_ELE pCapELE; -if (!memcmp (CapIE, EWC11NHTCap,sizeof (EWC11NHTCap))) - { +if (!memcmp (CapIE, EWC11NHTCap,sizeof (EWC11NHTCap))) {//EWC IE IEEE80211_DEBUG (IEEE80211_DL_HT ,"EWC IE in %s()\n" , __func__); pCapELE = (PHT_CAPABILITY_ELE )(&CapIE[4 ]); - }else + }else { pCapELE = (PHT_CAPABILITY_ELE )(&CapIE[0 ]); - -IEEE80211_DEBUG (IEEE80211_DL_HT ,"<Log HT Capability>. Called by %s\n" , TitleString ); + } +IEEE80211_DEBUG (IEEE80211_DL_HT ,"<Log HT Capability>. Called by %s\n" , TitleString);IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupported Channel Width = %s\n" , (pCapELE->ChlWidth) ?"20MHz" :"20/40MHz" );IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSupport Short GI for 20M = %s\n" , (pCapELE->ShortGI20Mhz) ?"YES" :"NO" );
@@ -165,21 +164,19 @@ void HTDebugHTInfo(u8 *InfoIE, u8 *TitleString)static u8 EWC11NHTInfo[] = {0x00 ,0x90 ,0x4c ,0x34 };// For 11n EWC definition, 2007.07.17, by Emily PHT_INFORMATION_ELE pHTInfoEle; -if (!memcmp (InfoIE, EWC11NHTInfo,sizeof (EWC11NHTInfo))) - { +if (!memcmp (InfoIE, EWC11NHTInfo,sizeof (EWC11NHTInfo))) {// Not EWC IE IEEE80211_DEBUG (IEEE80211_DL_HT ,"EWC IE in %s()\n" , __func__); pHTInfoEle = (PHT_INFORMATION_ELE )(&InfoIE[4 ]); - }else + }else { pHTInfoEle = (PHT_INFORMATION_ELE )(&InfoIE[0 ]); - + }IEEE80211_DEBUG (IEEE80211_DL_HT ,"<Log HT Information Element>. Called by %s\n" , TitleString);IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tPrimary channel = %d\n" , pHTInfoEle->ControlChl);IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSecondary channel =" ); -switch (pHTInfoEle->ExtChlOffset) - { +switch (pHTInfoEle->ExtChlOffset) {case 0 :IEEE80211_DEBUG (IEEE80211_DL_HT ,"Not Present\n" );break ;
@@ -196,8 +193,7 @@ void HTDebugHTInfo(u8 *InfoIE, u8 *TitleString)IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tRecommended channel width = %s\n" , (pHTInfoEle->RecommemdedTxWidth) ?"20Mhz" :"40Mhz" );IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tOperation mode for protection = " ); -switch (pHTInfoEle->OptMode) - { +switch (pHTInfoEle->OptMode) {case 0 :IEEE80211_DEBUG (IEEE80211_DL_HT ,"No Protection\n" );break ;
@@ -248,15 +244,12 @@ static bool IsHTHalfNmodeSGI(struct ieee80211_device *ieee, bool is40MHz) retValue = false; else if (!ieee->GetHalfNmodeSupportByAPsHandler(ieee->dev)) // station in half n mode retValue = false; - else if (is40MHz) // ap support 40 bw - { + else if (is40MHz) { // ap support 40 bwif (((PHT_CAPABILITY_ELE )(pHTInfo->PeerHTCapBuf))->ShortGI40Mhz)// ap support 40 bw short GI retValue = true;else retValue = false; - } - else - { + } else {if (((PHT_CAPABILITY_ELE )(pHTInfo->PeerHTCapBuf))->ShortGI20Mhz)// ap support 40 bw short GI retValue = true;else
@@ -305,35 +298,25 @@ u16 TxCountToDataRate(struct ieee80211_device *ieee, u8 nDataRate)u8 is40MHz =0 ;u8 isShortGI =0 ; -if (nDataRate <12 ) - { +if (nDataRate <12 ) {return CCKOFDMRate[nDataRate]; - } -else - { -if (nDataRate >=0x10 && nDataRate <=0x1f )//if(nDataRate > 11 && nDataRate < 28 ) - { + }else { +if (nDataRate >=0x10 && nDataRate <=0x1f ) {//if(nDataRate > 11 && nDataRate < 28 ) is40MHz =0 ; isShortGI =0 ;// nDataRate = nDataRate - 12; - } -else if (nDataRate >=0x20 && nDataRate <=0x2f )//(27, 44) - { + }else if (nDataRate >=0x20 && nDataRate <=0x2f ) {//(27, 44) is40MHz =1 ; isShortGI =0 ;//nDataRate = nDataRate - 28; - } -else if (nDataRate >=0x30 && nDataRate <=0x3f )//(43, 60) - { + }else if (nDataRate >=0x30 && nDataRate <=0x3f ) {//(43, 60) is40MHz =0 ; isShortGI =1 ;//nDataRate = nDataRate - 44; - } -else if (nDataRate >=0x40 && nDataRate <=0x4f )//(59, 76) - { + }else if (nDataRate >=0x40 && nDataRate <=0x4f ) {//(59, 76) is40MHz =1 ; isShortGI =1 ;
@@ -388,7 +371,7 @@ static void HTIOTPeerDetermine(struct ieee80211_device *ieee)else if ((memcmp (net->bssid,UNKNOWN_BORADCOM ,3 ) ==0 ) || (memcmp (net->bssid,LINKSYSWRT330_LINKSYSWRT300_BROADCOM ,3 ) ==0 ) || (memcmp (net->bssid,LINKSYSWRT350_LINKSYSWRT150_BROADCOM ,3 ) ==0 ) || - (memcmp (net->bssid, NETGEAR834Bv2_BROADCOM,3 ) ==0 ) ) + (memcmp (net->bssid, NETGEAR834Bv2_BROADCOM,3 ) ==0 )) pHTInfo->IOTPeer =HT_IOT_PEER_BROADCOM ; else if ((memcmp(net->bssid, BELKINF5D8233V1_RALINK, 3) == 0) || (memcmp(net->bssid, BELKINF5D82334V3_RALINK, 3) == 0) ||
@@ -505,9 +488,7 @@ static u8 HTIOTActIsMgntUseCCK6M(struct ieee80211_network *network)// 2008/01/28 MH We must prevent that we select null bssid to link. if (network->broadcom_cap_exist) - { retValue =1 ; - }return retValue; }
@@ -518,9 +499,7 @@ static u8 HTIOTActIsCCDFsync(u8 *PeerMacAddr)if ((memcmp (PeerMacAddr,UNKNOWN_BORADCOM ,3 ) ==0 ) || (memcmp (PeerMacAddr,LINKSYSWRT330_LINKSYSWRT300_BROADCOM ,3 ) ==0 ) || (memcmp (PeerMacAddr,LINKSYSWRT350_LINKSYSWRT150_BROADCOM ,3 ) ==0 )) - { retValue =1 ; - }return retValue; }
@@ -550,31 +529,25 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, uPHT_CAPABILITY_ELE pCapELE =NULL ;//u8 bIsDeclareMCS13; -if ((posHTCap ==NULL ) || (pHT ==NULL )) - { +if ((posHTCap ==NULL ) || (pHT ==NULL )) {IEEE80211_DEBUG (IEEE80211_DL_ERR ,"posHTCap or pHTInfo can't be null in HTConstructCapabilityElement()\n" );return ; }memset (posHTCap,0 , *len); -if (pHT->ePeerHTSpecVer ==HT_SPEC_VER_EWC ) - { +if (pHT->ePeerHTSpecVer ==HT_SPEC_VER_EWC ) {u8 EWC11NHTCap[] = {0x00 ,0x90 ,0x4c ,0x33 };// For 11n EWC definition, 2007.07.17, by Emily memcpy (posHTCap, EWC11NHTCap,sizeof (EWC11NHTCap)); pCapELE = (PHT_CAPABILITY_ELE )&(posHTCap[4 ]); - }else - { + }else { pCapELE = (PHT_CAPABILITY_ELE )posHTCap; }//HT capability info pCapELE->AdvCoding =0 ;// This feature is not supported now!! -if (ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev)) - { +if (ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev)) { pCapELE->ChlWidth =0 ; - } -else - { + }else { pCapELE->ChlWidth = (pHT->bRegBW40MHz ?1 :0 ); }
@@ -601,9 +574,7 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, uif (IsEncrypt) { pCapELE->MPDUDensity =7 ;// 8us pCapELE->MaxRxAMPDUFactor =2 ;// 2 is for 32 K and 3 is 64K - } -else - { + }else { pCapELE->MaxRxAMPDUFactor =3 ;// 2 is for 32 K and 3 is 64K pCapELE->MPDUDensity =0 ;// no density }
@@ -621,8 +592,7 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, u// 2008.06.12 // For RTL819X, if pairwisekey = wep/tkip, ap is ralink, we support only MCS0~7. -if (ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev)) - { +if (ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev)) {int i;for (i =1 ; i <16 ; i++) pCapELE->MCS[i] =0 ;
@@ -666,15 +636,13 @@ void HTConstructInfoElement(struct ieee80211_device *ieee, u8 *posHTInfo, u8 *le { PRT_HIGH_THROUGHPUT pHT = ieee->pHTInfo; PHT_INFORMATION_ELE pHTInfoEle = (PHT_INFORMATION_ELE)posHTInfo; - if ((posHTInfo == NULL) || (pHTInfoEle == NULL)) - { + if ((posHTInfo == NULL) || (pHTInfoEle == NULL)) { IEEE80211_DEBUG(IEEE80211_DL_ERR, "posHTInfo or pHTInfoEle can't be null in HTConstructInfoElement()\n"); return; } memset(posHTInfo, 0, *len); - if ( (ieee->iw_mode == IW_MODE_ADHOC) || (ieee->iw_mode == IW_MODE_MASTER)) //ap mode is not currently supported - { + if ((ieee->iw_mode == IW_MODE_ADHOC) || (ieee->iw_mode == IW_MODE_MASTER)) { //ap mode is not currently supported pHTInfoEle->ControlChl = ieee->current_network.channel; pHTInfoEle->ExtChlOffset = ((!pHT->bRegBW40MHz) ? HT_EXTCHNL_OFFSET_NO_EXT : (ieee->current_network.channel <= 6) ?
@@ -696,9 +664,7 @@ void HTConstructInfoElement(struct ieee80211_device *ieee, u8 *posHTInfo, u8 *le *len =22 +2 ;//same above - } -else - { + }else {//STA should not generate High Throughput Information Element *len =0 ; }
@@ -791,23 +757,20 @@ void HTConstructRT2RTAggElement(struct ieee80211_device *ieee, u8 *posRT2RTAgg,static u8 HT_PickMCSRate (struct ieee80211_device *ieee,u8 *pOperateMCS) {u8 i; -if (pOperateMCS ==NULL ) - { +if (pOperateMCS ==NULL ) {IEEE80211_DEBUG (IEEE80211_DL_ERR ,"pOperateMCS can't be null in HT_PickMCSRate()\n" );return false; } -switch (ieee->mode) - { +switch (ieee->mode) {case IEEE_A :case IEEE_B :case IEEE_G ://legacy rate routine handled at selectedrate //no MCS rate -for (i =0 ; i <=15 ; i++){ +for (i =0 ; i <=15 ; i++) pOperateMCS[i] =0 ; - }break ;case IEEE_N_24G ://assume CCK rate ok
@@ -857,31 +820,25 @@ u8 HTGetHighestMCSRate(struct ieee80211_device *ieee, u8 *pMCSRateSet, u8 *pMCSFu8 bitMap;u8 mcsRate =0 ;u8 availableMcsRate[16 ]; -if (pMCSRateSet ==NULL || pMCSFilter ==NULL ) - { +if (pMCSRateSet ==NULL || pMCSFilter ==NULL ) {IEEE80211_DEBUG (IEEE80211_DL_ERR ,"pMCSRateSet or pMCSFilter can't be null in HTGetHighestMCSRate()\n" );return false; }for (i =0 ; i <16 ; i++) availableMcsRate[i] = pMCSRateSet[i] & pMCSFilter[i]; -for (i =0 ; i <16 ; i++) - { +for (i =0 ; i <16 ; i++) {if (availableMcsRate[i] !=0 )break ; }if (i ==16 )return false; -for (i =0 ; i <16 ; i++) - { -if (availableMcsRate[i] !=0 ) - { +for (i =0 ; i <16 ; i++) { +if (availableMcsRate[i] !=0 ) { bitMap = availableMcsRate[i]; -for (j =0 ; j <8 ; j++) - { -if ((bitMap%2 ) !=0 ) - { +for (j =0 ; j <8 ; j++) { +if ((bitMap%2 ) !=0 ) {if (HTMcsToDataRate (ieee, (8 *i+j)) >HTMcsToDataRate (ieee, mcsRate)) mcsRate = (8 *i+j); }
@@ -910,7 +867,7 @@ static u8 HTFilterMCSRate(struct ieee80211_device *ieee, u8 *pSupportMCS,u8 i =0 ;// filter out operational rate set not supported by AP, the length of it is 16 -for (i =0 ; i <=15 ; i++){ +for (i =0 ; i <=15 ; i++) { pOperateMCS[i] = ieee->Regdot11HTOperationalRateSet[i]&pSupportMCS[i]; }
@@ -1008,7 +965,7 @@ void HTOnAssocRsp(struct ieee80211_device *ieee) nMaxAMSDUSize = (pPeerHTCap->MaxAMSDUSize ==0 ) ?3839 :7935 ; -if (pHTInfo->nAMSDU_MaxSize > nMaxAMSDUSize ) +if (pHTInfo->nAMSDU_MaxSize > nMaxAMSDUSize) pHTInfo->nCurrent_AMSDU_MaxSize = nMaxAMSDUSize;else pHTInfo->nCurrent_AMSDU_MaxSize = pHTInfo->nAMSDU_MaxSize;
@@ -1022,27 +979,23 @@ void HTOnAssocRsp(struct ieee80211_device *ieee)// <1> Decide AMPDU Factor // By Emily -if (!pHTInfo->bRegRT2RTAggregation) - { +if (!pHTInfo->bRegRT2RTAggregation) {// Decide AMPDU Factor according to protocol handshake if (pHTInfo->AMPDU_Factor > pPeerHTCap->MaxRxAMPDUFactor) pHTInfo->CurrentAMPDUFactor = pPeerHTCap->MaxRxAMPDUFactor;else pHTInfo->CurrentAMPDUFactor = pHTInfo->AMPDU_Factor; - }else - { + }else {// Set MPDU density to 2 to Realtek AP, and set it to 0 for others // Replace MPDU factor declared in original association response frame format. 2007.08.20 by Emily -if (ieee->current_network.bssht.bdRT2RTAggregation) - { +if (ieee->current_network.bssht.bdRT2RTAggregation) {if (ieee->pairwise_key_type !=KEY_TYPE_NA )// Realtek may set 32k in security mode and 64k for others pHTInfo->CurrentAMPDUFactor = pPeerHTCap->MaxRxAMPDUFactor;else pHTInfo->CurrentAMPDUFactor =HT_AGG_SIZE_64K ; - }else - { + }else {if (pPeerHTCap->MaxRxAMPDUFactor <HT_AGG_SIZE_32K ) pHTInfo->CurrentAMPDUFactor = pPeerHTCap->MaxRxAMPDUFactor;else
@@ -1056,18 +1009,16 @@ void HTOnAssocRsp(struct ieee80211_device *ieee) pHTInfo->CurrentMPDUDensity = pHTInfo->MPDU_Density; else pHTInfo->CurrentMPDUDensity = pPeerHTCap->MPDUDensity; -if (ieee->pairwise_key_type !=KEY_TYPE_NA ) +if (ieee->pairwise_key_type !=KEY_TYPE_NA ) pHTInfo->CurrentMPDUDensity =7 ;// 8us // Force TX AMSDU // Lanhsin: mark for tmp to avoid deauth by ap from s3 //if(memcmp(pMgntInfo->Bssid, NETGEAR834Bv2_BROADCOM, 3)==0) -if (0 ) - { - - pHTInfo->bCurrentAMPDUEnable = false; - pHTInfo->ForcedAMSDUMode =HT_AGG_FORCE_ENABLE ; - pHTInfo->ForcedAMSDUMaxSize =7935 ; +if (0 ) { + pHTInfo->bCurrentAMPDUEnable = false; + pHTInfo->ForcedAMSDUMode =HT_AGG_FORCE_ENABLE ; + pHTInfo->ForcedAMSDUMaxSize =7935 ; pHTInfo->IOTAction |=HT_IOT_ACT_TX_USE_AMSDU_8K ; }
@@ -1224,8 +1175,7 @@ void HTResetSelfAndSavePeerSetting(struct ieee80211_device *ieee, struct ieee802IEEE80211_DEBUG (IEEE80211_DL_HT ,"==============>%s()\n" , __func__);/*unmark bEnableHT flag here is the same reason why unmarked in function ieee80211_softmac_new_net. WB 2008.09.10*/ // if( pHTInfo->bEnableHT && pNetwork->bssht.bdSupportHT) -if (pNetwork->bssht.bdSupportHT) - { +if (pNetwork->bssht.bdSupportHT) { pHTInfo->bCurrentHTSupport = true; pHTInfo->ePeerHTSpecVer = pNetwork->bssht.bdHTSpecVer;
@@ -1237,13 +1187,10 @@ void HTResetSelfAndSavePeerSetting(struct ieee80211_device *ieee, struct ieee802memcpy (pHTInfo->PeerHTInfoBuf, pNetwork->bssht.bdHTInfoBuf, pNetwork->bssht.bdHTInfoLen);// Check whether RT to RT aggregation mode is enabled -if (pHTInfo->bRegRT2RTAggregation) - { +if (pHTInfo->bRegRT2RTAggregation) { pHTInfo->bCurrentRT2RTAggregation = pNetwork->bssht.bdRT2RTAggregation; pHTInfo->bCurrentRT2RTLongSlotTime = pNetwork->bssht.bdRT2RTLongSlotTime; - } -else - { + }else { pHTInfo->bCurrentRT2RTAggregation = false; pHTInfo->bCurrentRT2RTLongSlotTime = false; }
@@ -1280,9 +1227,7 @@ void HTResetSelfAndSavePeerSetting(struct ieee80211_device *ieee, struct ieee802 pHTInfo->IOTAction |=HT_IOT_ACT_CDD_FSYNC ; - } -else - { + }else { pHTInfo->bCurrentHTSupport = false; pHTInfo->bCurrentRT2RTAggregation = false; pHTInfo->bCurrentRT2RTLongSlotTime = false;
@@ -1298,8 +1243,7 @@ void HTUpdateSelfAndPeerSetting(struct ieee80211_device *ieee, struct ieee80211_// PHT_CAPABILITY_ELE pPeerHTCap = (PHT_CAPABILITY_ELE)pNetwork->bssht.bdHTCapBuf; PHT_INFORMATION_ELE pPeerHTInfo = (PHT_INFORMATION_ELE )pNetwork->bssht.bdHTInfoBuf; -if (pHTInfo->bCurrentHTSupport) - { +if (pHTInfo->bCurrentHTSupport) {// // Config current operation mode. //
@@ -1323,8 +1267,7 @@ EXPORT_SYMBOL(HTUpdateSelfAndPeerSetting);********************************************************************************************************************/ u8 HTCCheck (struct ieee80211_device *ieee,u8 *pFrame) { -if (ieee->pHTInfo->bCurrentHTSupport) - { +if (ieee->pHTInfo->bCurrentHTSupport) {if ((IsQoSDataFrame (pFrame) &&Frame_Order (pFrame)) ==1 ) {IEEE80211_DEBUG (IEEE80211_DL_HT ,"HT CONTROL FILED EXIST!!\n" );return true;
@@ -1357,8 +1300,7 @@ void HTSetConnectBwMode(struct ieee80211_device *ieee, HT_CHANNEL_WIDTH Bandwidtreturn ; }//if in half N mode, set to 20M bandwidth please 09.08.2008 WB. -if (Bandwidth ==HT_CHANNEL_WIDTH_20_40 && (!ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev))) - { +if (Bandwidth ==HT_CHANNEL_WIDTH_20_40 && (!ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev))) {// Handle Illegal extension channel offset!! if (ieee->current_network.channel <2 && Offset ==HT_EXTCHNL_OFFSET_LOWER ) Offset =HT_EXTCHNL_OFFSET_NO_EXT ;
@@ -1391,8 +1333,7 @@ void HTSetConnectBwModeCallback(struct ieee80211_device *ieee)IEEE80211_DEBUG (IEEE80211_DL_HT ,"======>%s()\n" , __func__); -if (pHTInfo->bCurBW40MHz) - { +if (pHTInfo->bCurBW40MHz) {if (pHTInfo->CurSTAExtChnlOffset ==HT_EXTCHNL_OFFSET_UPPER ) ieee->set_chan (ieee->dev, ieee->current_network.channel+2 );else if (pHTInfo->CurSTAExtChnlOffset ==HT_EXTCHNL_OFFSET_LOWER )
Signature · 1 line
2.16.3
PATCH v4 3/5 staging: rtl8192e: Correct indentation of switch statements - coding style
In reply to John WhitmoreTwo switch statements had wrong indentation of 'case' options
--- .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 48 +++++++++++----------- 1 file changed, 24 insertions(+), 24 deletions(-)
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c +24 -24
@@ -177,35 +177,35 @@ void HTDebugHTInfo(u8 *InfoIE, u8 *TitleString)IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tPrimary channel = %d\n" , pHTInfoEle->ControlChl);IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tSecondary channel =" );switch (pHTInfoEle->ExtChlOffset) { -case 0 : -IEEE80211_DEBUG (IEEE80211_DL_HT ,"Not Present\n" ); -break ; -case 1 : -IEEE80211_DEBUG (IEEE80211_DL_HT ,"Upper channel\n" ); -break ; -case 2 : -IEEE80211_DEBUG (IEEE80211_DL_HT ,"Reserved. Eooro!!!\n" ); -break ; -case 3 : -IEEE80211_DEBUG (IEEE80211_DL_HT ,"Lower Channel\n" ); -break ; +case 0 : +IEEE80211_DEBUG (IEEE80211_DL_HT ,"Not Present\n" ); +break ; +case 1 : +IEEE80211_DEBUG (IEEE80211_DL_HT ,"Upper channel\n" ); +break ; +case 2 : +IEEE80211_DEBUG (IEEE80211_DL_HT ,"Reserved. Eooro!!!\n" ); +break ; +case 3 : +IEEE80211_DEBUG (IEEE80211_DL_HT ,"Lower Channel\n" ); +break ; }IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tRecommended channel width = %s\n" , (pHTInfoEle->RecommemdedTxWidth) ?"20Mhz" :"40Mhz" );IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tOperation mode for protection = " );switch (pHTInfoEle->OptMode) { -case 0 : -IEEE80211_DEBUG (IEEE80211_DL_HT ,"No Protection\n" ); -break ; -case 1 : -IEEE80211_DEBUG (IEEE80211_DL_HT ,"HT non-member protection mode\n" ); -break ; -case 2 : -IEEE80211_DEBUG (IEEE80211_DL_HT ,"Suggest to open protection\n" ); -break ; -case 3 : -IEEE80211_DEBUG (IEEE80211_DL_HT ,"HT mixed mode\n" ); -break ; +case 0 : +IEEE80211_DEBUG (IEEE80211_DL_HT ,"No Protection\n" ); +break ; +case 1 : +IEEE80211_DEBUG (IEEE80211_DL_HT ,"HT non-member protection mode\n" ); +break ; +case 2 : +IEEE80211_DEBUG (IEEE80211_DL_HT ,"Suggest to open protection\n" ); +break ; +case 3 : +IEEE80211_DEBUG (IEEE80211_DL_HT ,"HT mixed mode\n" ); +break ; } IEEE80211_DEBUG(IEEE80211_DL_HT, "\tBasic MCS Rate Set = [%x][%x][%x][%x][%x]\n", pHTInfoEle->BasicMSC[0],\
Signature · 1 line
2.16.3
PATCH v4 4/5 staging: rtl8192e: Add and remove blank lines - Coding style
In reply to John WhitmoreCoding style requires blank line after definitions and function definition, blank lines removed. Correction of some indentation.
--- .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 96 ++++++++-------------- 1 file changed, 34 insertions(+), 62 deletions(-)
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c +34 -62
@@ -111,9 +111,8 @@ void HTUpdateDefaultSetting(struct ieee80211_device *ieee) pHTInfo->UsbRxFwAggrPacketNum =8 ; pHTInfo->UsbRxFwAggrTimeout =16 ;////usb rx FW aggregation timeout threshold.It's in units of 64us #endif - - } +/******************************************************************************************************************** *function: This function print out each field on HT capability IE mainly from (Beacon/ProbeRsp/AssocReq) * input: u8* CapIE //Capability IE to be printed out
@@ -124,7 +123,6 @@ void HTUpdateDefaultSetting(struct ieee80211_device *ieee)* *****************************************************************************************************************/ void HTDebugHTCapability (u8 *CapIE,u8 *TitleString) { -static u8 EWC11NHTCap[] = {0x00 ,0x90 ,0x4c ,0x33 };// For 11n EWC definition, 2007.07.17, by Emily PHT_CAPABILITY_ELE pCapELE;
@@ -150,6 +148,7 @@ void HTDebugHTCapability(u8 *CapIE, u8 *TitleString)return ; } +/******************************************************************************************************************** *function: This function print out each field on HT Information IE mainly from (Beacon/ProbeRsp) * input: u8* InfoIE //Capability IE to be printed out
@@ -271,7 +270,6 @@ u16 HTHalfMcsToDataRate(struct ieee80211_device *ieee, u8 nMcsRate)return MCS_DATA_RATE [is40MHz][isShortGI][(nMcsRate&0x7f )]; } -u16 HTMcsToDataRate (struct ieee80211_device *ieee,u8 nMcsRate) {PRT_HIGH_THROUGHPUT pHTInfo = ieee->pHTInfo;
@@ -326,12 +324,11 @@ u16 TxCountToDataRate(struct ieee80211_device *ieee, u8 nDataRate) } } - -bool IsHTHalfNmodeAPs (struct ieee80211_device *ieee) {bool retValue = false;struct ieee80211_network *net = &ieee->current_network; + if ((memcmp(net->bssid, BELKINF5D8233V1_RALINK, 3) == 0) || (memcmp(net->bssid, BELKINF5D82334V3_RALINK, 3) == 0) || (memcmp(net->bssid, PCI_RALINK, 3) == 0) ||
@@ -364,6 +361,7 @@ static void HTIOTPeerDetermine(struct ieee80211_device *ieee) {PRT_HIGH_THROUGHPUT pHTInfo = ieee->pHTInfo;struct ieee80211_network *net = &ieee->current_network; +if (net->bssht.bdRT2RTAggregation) pHTInfo->IOTPeer =HT_IOT_PEER_REALTEK ;else if (net->broadcom_cap_exist)
@@ -389,6 +387,7 @@ static void HTIOTPeerDetermine(struct ieee80211_device *ieee)IEEE80211_DEBUG (IEEE80211_DL_IOT ,"Joseph debug!! IOTPEER: %x\n" , pHTInfo->IOTPeer); } +/******************************************************************************************************************** *function: Check whether driver should declare received rate up to MCS13 only since some chipset is not good * at receiving MCS14~15 frame from some AP.
@@ -400,8 +399,7 @@ static void HTIOTPeerDetermine(struct ieee80211_device *ieee)static u8 HTIOTActIsDisableMCS14 (struct ieee80211_device *ieee,u8 *PeerMacAddr) {return 0 ; - } - +}/** * Function: HTIOTActIsDisableMCS15
@@ -496,6 +494,7 @@ static u8 HTIOTActIsMgntUseCCK6M(struct ieee80211_network *network)static u8 HTIOTActIsCCDFsync (u8 *PeerMacAddr) {u8 retValue =0 ; +if ((memcmp (PeerMacAddr,UNKNOWN_BORADCOM ,3 ) ==0 ) || (memcmp (PeerMacAddr,LINKSYSWRT330_LINKSYSWRT300_BROADCOM ,3 ) ==0 ) || (memcmp (PeerMacAddr,LINKSYSWRT350_LINKSYSWRT150_BROADCOM ,3 ) ==0 ))
@@ -504,15 +503,12 @@ static u8 HTIOTActIsCCDFsync(u8 *PeerMacAddr)return retValue; } -void HTResetIOTSetting ( -PRT_HIGH_THROUGHPUT pHTInfo -) +void HTResetIOTSetting (PRT_HIGH_THROUGHPUT pHTInfo) { pHTInfo->IOTAction =0 ; pHTInfo->IOTPeer =HT_IOT_PEER_UNKNOWN ; } -/******************************************************************************************************************** *function: Construct Capablility Element in Beacon... if HTEnable is turned on * input: struct ieee80211_device* ieee
@@ -536,13 +532,13 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, umemset (posHTCap,0 , *len);if (pHT->ePeerHTSpecVer ==HT_SPEC_VER_EWC ) {u8 EWC11NHTCap[] = {0x00 ,0x90 ,0x4c ,0x33 };// For 11n EWC definition, 2007.07.17, by Emily +memcpy (posHTCap, EWC11NHTCap,sizeof (EWC11NHTCap)); pCapELE = (PHT_CAPABILITY_ELE )&(posHTCap[4 ]); }else { pCapELE = (PHT_CAPABILITY_ELE )posHTCap; } -//HT capability info pCapELE->AdvCoding =0 ;// This feature is not supported now!! if (ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev)) {
@@ -566,7 +562,6 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, u pCapELE->PSMP =0 ;// Do not support now!! pCapELE->LSigTxopProtect =0 ;// Do not support now!! -//MAC HT parameters info // TODO: Nedd to take care of this part IEEE80211_DEBUG (IEEE80211_DL_HT ,"TX HT cap/info ele BW=%d MaxAMSDUSize:%d DssCCk:%d\n" , pCapELE->ChlWidth, pCapELE->MaxAMSDUSize, pCapELE->DssCCk);
@@ -594,6 +589,7 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, u// For RTL819X, if pairwisekey = wep/tkip, ap is ralink, we support only MCS0~7. if (ieee->GetHalfNmodeSupportByAPsHandler (ieee->dev)) {int i; +for (i =1 ; i <16 ; i++) pCapELE->MCS[i] =0 ; }
@@ -601,7 +597,6 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, u//Extended HT Capability Info memset (&pCapELE->ExtHTCapInfo,0 ,2 ); -//TXBF Capabilities memset (pCapELE->TxBFCap,0 ,4 );
@@ -613,8 +608,6 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, uelse *len =26 +2 ; - -// IEEE80211_DEBUG_DATA(IEEE80211_DL_DATA | IEEE80211_DL_HT, posHTCap, *len -2); //Print each field in detail. Driver should not print out this message by default
@@ -622,6 +615,7 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, ureturn ; } +/******************************************************************************************************************** *function: Construct Information Element in Beacon... if HTEnable is turned on * input: struct ieee80211_device* ieee
@@ -636,6 +630,7 @@ void HTConstructInfoElement(struct ieee80211_device *ieee, u8 *posHTInfo, u8 *le {PRT_HIGH_THROUGHPUT pHT = ieee->pHTInfo;PHT_INFORMATION_ELE pHTInfoEle = (PHT_INFORMATION_ELE )posHTInfo; +if ((posHTInfo ==NULL ) || (pHTInfoEle ==NULL )) {IEEE80211_DEBUG (IEEE80211_DL_ERR ,"posHTInfo or pHTInfoEle can't be null in HTConstructInfoElement()\n" );return ;
@@ -661,7 +656,6 @@ void HTConstructInfoElement(struct ieee80211_device *ieee, u8 *posHTInfo, u8 *lememset (pHTInfoEle->BasicMSC,0 ,16 ); - *len =22 +2 ;//same above }else {
@@ -733,20 +727,14 @@ void HTConstructRT2RTAggElement(struct ieee80211_device *ieee, u8 *posRT2RTAgg,posRT2RTAgg->Octet[5] &= 0xfb; } */ -#else // Do Nothing #endif posRT2RTAgg->Length =6 ;#endif - - - - } -/******************************************************************************************************************** *function: Pick the right Rate Adaptive table to use * input: struct ieee80211_device* ieee
@@ -756,7 +744,8 @@ void HTConstructRT2RTAggElement(struct ieee80211_device *ieee, u8 *posRT2RTAgg,* *****************************************************************************************************************/ static u8 HT_PickMCSRate (struct ieee80211_device *ieee,u8 *pOperateMCS) { -u8 i; +u8 i; +if (pOperateMCS ==NULL ) {IEEE80211_DEBUG (IEEE80211_DL_ERR ,"pOperateMCS can't be null in HT_PickMCSRate()\n" );return false;
@@ -766,30 +755,28 @@ static u8 HT_PickMCSRate(struct ieee80211_device *ieee, u8 *pOperateMCS)case IEEE_A :case IEEE_B :case IEEE_G : -//legacy rate routine handled at selectedrate +//legacy rate routine handled at selectedrate -//no MCS rate -for (i =0 ; i <=15 ; i++) - pOperateMCS[i] =0 ; -break ; +//no MCS rate +for (i =0 ; i <=15 ; i++) + pOperateMCS[i] =0 ; +break ;case IEEE_N_24G ://assume CCK rate ok case IEEE_N_5G : -// Legacy part we only use 6, 5.5,2,1 for N_24G and 6 for N_5G. -// Legacy part shall be handled at SelectRateSet(). - -//HT part -// TODO: may be different if we have different number of antenna - pOperateMCS[0 ] &=RATE_ADPT_1SS_MASK ;//support MCS 0~7 - pOperateMCS[1 ] &=RATE_ADPT_2SS_MASK ; - pOperateMCS[3 ] &=RATE_ADPT_MCS32_MASK ; -break ; +// Legacy part we only use 6, 5.5,2,1 for N_24G and 6 for N_5G. +// Legacy part shall be handled at SelectRateSet(). + +//HT part +// TODO: may be different if we have different number of antenna + pOperateMCS[0 ] &=RATE_ADPT_1SS_MASK ;//support MCS 0~7 + pOperateMCS[1 ] &=RATE_ADPT_2SS_MASK ; + pOperateMCS[3 ] &=RATE_ADPT_MCS32_MASK ; +break ;//should never reach here default : - -break ; - +break ; }return true;
@@ -820,6 +807,7 @@ u8 HTGetHighestMCSRate(struct ieee80211_device *ieee, u8 *pMCSRateSet, u8 *pMCSFu8 bitMap;u8 mcsRate =0 ;u8 availableMcsRate[16 ]; +if (pMCSRateSet ==NULL || pMCSFilter ==NULL ) {IEEE80211_DEBUG (IEEE80211_DL_ERR ,"pMCSRateSet or pMCSFilter can't be null in HTGetHighestMCSRate()\n" );return false;
@@ -849,8 +837,6 @@ u8 HTGetHighestMCSRate(struct ieee80211_device *ieee, u8 *pMCSRateSet, u8 *pMCSFreturn (mcsRate|0x80 ); } - -/* ** **1.Filter our operation rate set with AP's rate set
@@ -863,7 +849,6 @@ u8 HTGetHighestMCSRate(struct ieee80211_device *ieee, u8 *pMCSRateSet, u8 *pMCSFstatic u8 HTFilterMCSRate (struct ieee80211_device *ieee,u8 *pSupportMCS,u8 *pOperateMCS) { -u8 i =0 ;// filter out operational rate set not supported by AP, the length of it is 16
@@ -871,7 +856,6 @@ static u8 HTFilterMCSRate(struct ieee80211_device *ieee, u8 *pSupportMCS, pOperateMCS[i] = ieee->Regdot11HTOperationalRateSet[i]&pSupportMCS[i]; } -// TODO: adjust our operational rate set according to our channel bandwidth, STBC and Antenna number // TODO: fill suggested rate adaptive rate index and give firmware info using Tx command packet
@@ -891,6 +875,7 @@ static u8 HTFilterMCSRate(struct ieee80211_device *ieee, u8 *pSupportMCS,return true; } +void HTSetConnectBwMode (struct ieee80211_device *ieee,HT_CHANNEL_WIDTH Bandwidth,HT_EXTCHNL_OFFSET Offset);void HTOnAssocRsp (struct ieee80211_device *ieee) {
@@ -924,7 +909,6 @@ void HTOnAssocRsp(struct ieee80211_device *ieee)else pPeerHTInfo = (PHT_INFORMATION_ELE )(pHTInfo->PeerHTInfoBuf); -//////////////////////////////////////////////////////// // Configurations: ////////////////////////////////////////////////////////
@@ -956,8 +940,6 @@ void HTOnAssocRsp(struct ieee80211_device *ieee)// TODO: pHTInfo->bCurSuppCCK = pHTInfo->bRegSuppCCK && (pPeerHTCap->DssCCk ==1 ); - -// // Config and configure A-MSDU setting //
@@ -969,8 +951,6 @@ void HTOnAssocRsp(struct ieee80211_device *ieee) pHTInfo->nCurrent_AMSDU_MaxSize = nMaxAMSDUSize; else pHTInfo->nCurrent_AMSDU_MaxSize = pHTInfo->nAMSDU_MaxSize; - -// // Config A-MPDU setting //
@@ -1056,9 +1036,6 @@ void HTOnAssocRsp(struct ieee80211_device *ieee)// Config current operation mode. // pHTInfo->CurrentOpMode = pPeerHTInfo->OptMode; - - - }void HTSetConnectBwModeCallback (struct ieee80211_device *ieee);
@@ -1103,8 +1080,6 @@ void HTInitializeHTInfo(struct ieee80211_device *ieee) pHTInfo->CurrentMPDUDensity = pHTInfo->MPDU_Density; pHTInfo->CurrentAMPDUFactor = pHTInfo->AMPDU_Factor; - -// Initialize all of the parameters related to 11n memset ((void *)(&(pHTInfo->SelfHTCap)),0 ,sizeof (pHTInfo->SelfHTCap));memset ((void *)(&(pHTInfo->SelfHTInfo)),0 ,sizeof (pHTInfo->SelfHTInfo));
@@ -1126,11 +1101,13 @@ void HTInitializeHTInfo(struct ieee80211_device *ieee)//MCS rate initialized here {u8 *RegHTSuppRateSets = &(ieee->RegHTSuppRateSet[0 ]); + RegHTSuppRateSets[0 ] =0xFF ;//support MCS 0~7 RegHTSuppRateSets[1 ] =0xFF ;//support MCS 8~15 RegHTSuppRateSets[4 ] =0x01 ;//support MCS 32 } } +/******************************************************************************************************************** *function: initialize Bss HT structure(struct PBSS_HT) * input: PBSS_HT pBssHT //to be initialized
@@ -1152,6 +1129,7 @@ void HTInitializeBssDesc(PBSS_HT pBssHT) pBssHT->bdRT2RTAggregation = false; pBssHT->bdRT2RTLongSlotTime = false; } +/******************************************************************************************************************** *function: initialize Bss HT structure(struct PBSS_HT) * input: struct ieee80211_device *ieee
@@ -1213,7 +1191,6 @@ void HTResetSelfAndSavePeerSetting(struct ieee80211_device *ieee, struct ieee802if (bIOTAction) pHTInfo->IOTAction |=HT_IOT_ACT_DISABLE_ALL_2SS ; - bIOTAction =HTIOTActIsDisableEDCATurbo (ieee, pNetwork->bssid);if (bIOTAction) pHTInfo->IOTAction |=HT_IOT_ACT_DISABLE_EDCA_TURBO ;
@@ -1225,8 +1202,6 @@ void HTResetSelfAndSavePeerSetting(struct ieee80211_device *ieee, struct ieee802 bIOTAction =HTIOTActIsCCDFsync (pNetwork->bssid);if (bIOTAction) pHTInfo->IOTAction |=HT_IOT_ACT_CDD_FSYNC ; - - }else { pHTInfo->bCurrentHTSupport = false; pHTInfo->bCurrentRT2RTAggregation = false;
@@ -1234,7 +1209,6 @@ void HTResetSelfAndSavePeerSetting(struct ieee80211_device *ieee, struct ieee802 pHTInfo->IOTAction =0 ; } - }void HTUpdateSelfAndPeerSetting (struct ieee80211_device *ieee,struct ieee80211_network *pNetwork)
@@ -1287,8 +1261,6 @@ void HTSetConnectBwMode(struct ieee80211_device *ieee, HT_CHANNEL_WIDTH Bandwidtif (!pHTInfo->bRegBW40MHz)return ; - -// To reduce dummy operation // if((pHTInfo->bCurBW40MHz==false && Bandwidth==HT_CHANNEL_WIDTH_20) || // (pHTInfo->bCurBW40MHz==true && Bandwidth==HT_CHANNEL_WIDTH_20_40 && Offset==pHTInfo->CurSTAExtChnlOffset))
Signature · 1 line
2.16.3
Greg KH
maintainerRe: [PATCH v4 4/5] staging: rtl8192e: Add and remove blank lines - Coding style
Quoted 2 lines · John Whitmore
Coding style requires blank line after definitions and function definition, blank lines removed. Correction of some indentation.
Do not change indentation in the same patch you should only be doing blank line removal or adding in. This needs to be fixed up.
greg k-h
PATCH v4 5/5 staging: rtl8192e: Remove unrequired blank lines - Coding Style
In reply to John WhitmoreMissed these in the last pass. Coding style does not require blank line after opening brace of a function.
--- drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 5 ----- 1 file changed, 5 deletions(-)
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c +0 -5
@@ -146,7 +146,6 @@ void HTDebugHTCapability(u8 *CapIE, u8 *TitleString)IEEE80211_DEBUG (IEEE80211_DL_HT ,"\tMCS Rate Set = [%x][%x][%x][%x][%x]\n" , pCapELE->MCS[0 ],\ pCapELE->MCS[1 ], pCapELE->MCS[2 ], pCapELE->MCS[3 ], pCapELE->MCS[4 ]);return ; - }/********************************************************************************************************************
@@ -159,7 +158,6 @@ void HTDebugHTCapability(u8 *CapIE, u8 *TitleString)* *****************************************************************************************************************/ void HTDebugHTInfo (u8 *InfoIE,u8 *TitleString) { -static u8 EWC11NHTInfo[] = {0x00 ,0x90 ,0x4c ,0x34 };// For 11n EWC definition, 2007.07.17, by Emily PHT_INFORMATION_ELE pHTInfoEle;
@@ -260,7 +258,6 @@ static bool IsHTHalfNmodeSGI(struct ieee80211_device *ieee, bool is40MHz)u16 HTHalfMcsToDataRate (struct ieee80211_device *ieee,u8 nMcsRate) { -u8 is40MHz;u8 isShortGI;
@@ -613,7 +610,6 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, u//Print each field in detail. Driver should not print out this message by default // HTDebugHTCapability(posHTCap, (u8*)"HTConstructCapability()"); return ; - }/********************************************************************************************************************
@@ -1117,7 +1113,6 @@ void HTInitializeHTInfo(struct ieee80211_device *ieee)********************************************************************************************************************/ void HTInitializeBssDesc (PBSS_HT pBssHT) { - pBssHT->bdSupportHT = false;memset (pBssHT->bdHTCapBuf,0 ,sizeof (pBssHT->bdHTCapBuf)); pBssHT->bdHTCapLen =0 ;
Signature · 1 line
2.16.3
Greg KH
maintainerRe: [PATCH v4 5/5] staging: rtl8192e: Remove unrequired blank lines - Coding Style
Quoted 2 lines · John Whitmore
Missed these in the last pass. Coding style does not require blank line after opening brace of a function.
This should have been merged with the previous patch :(