aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorQuytelda Kahja <quytelda@tamalin.org>2018-03-27 04:41:13 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2018-03-29 05:59:29 -0400
commit3df3602ac74b641f54149599c78d3606b467f58e (patch)
treea577967a94b804eb156061b3618e4819399b4692
parent9f534f7ee8558dd327f63fe07a9127eb32789065 (diff)
staging: rtl8723bs: Rework 'struct _ODM_Per_Pkt_Info_' coding style.
Change the typedef'd 'struct _ODM_Per_Pkt_Info_' into 'struct odm_packet_info' and change the members to snake case in order to meet the coding style guidelines. Members: * u8 DataRate -> data_rate * u8 StationID -> station_id * bool bPacketMatchBSSID -> bssid_match * bool bPacketToSelf -> to_self * bool bPacketBeacon -> is_beacon * bool bToSelf -> (removed because it isn't used) Signed-off-by: Quytelda Kahja <quytelda@tamalin.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/staging/rtl8723bs/hal/odm.h16
-rw-r--r--drivers/staging/rtl8723bs/hal/odm_CfoTracking.c4
-rw-r--r--drivers/staging/rtl8723bs/hal/odm_HWConfig.c26
-rw-r--r--drivers/staging/rtl8723bs/hal/odm_HWConfig.h2
-rw-r--r--drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c37
5 files changed, 43 insertions, 42 deletions
diff --git a/drivers/staging/rtl8723bs/hal/odm.h b/drivers/staging/rtl8723bs/hal/odm.h
index 87a76bafecb3..7c13a9314a22 100644
--- a/drivers/staging/rtl8723bs/hal/odm.h
+++ b/drivers/staging/rtl8723bs/hal/odm.h
@@ -262,15 +262,13 @@ typedef struct _ODM_Phy_Status_Info_ {
262} ODM_PHY_INFO_T, *PODM_PHY_INFO_T; 262} ODM_PHY_INFO_T, *PODM_PHY_INFO_T;
263 263
264 264
265typedef struct _ODM_Per_Pkt_Info_ { 265struct odm_packet_info {
266 /* u8 Rate; */ 266 u8 data_rate;
267 u8 DataRate; 267 u8 station_id;
268 u8 StationID; 268 bool bssid_match;
269 bool bPacketMatchBSSID; 269 bool to_self;
270 bool bPacketToSelf; 270 bool is_beacon;
271 bool bPacketBeacon; 271};
272 bool bToSelf;
273} ODM_PACKET_INFO_T, *PODM_PACKET_INFO_T;
274 272
275 273
276typedef struct _ODM_Phy_Dbg_Info_ { 274typedef struct _ODM_Phy_Dbg_Info_ {
diff --git a/drivers/staging/rtl8723bs/hal/odm_CfoTracking.c b/drivers/staging/rtl8723bs/hal/odm_CfoTracking.c
index 71853e6f7106..178aaab3f76c 100644
--- a/drivers/staging/rtl8723bs/hal/odm_CfoTracking.c
+++ b/drivers/staging/rtl8723bs/hal/odm_CfoTracking.c
@@ -316,14 +316,14 @@ void ODM_CfoTracking(void *pDM_VOID)
316void ODM_ParsingCFO(void *pDM_VOID, void *pPktinfo_VOID, s8 *pcfotail) 316void ODM_ParsingCFO(void *pDM_VOID, void *pPktinfo_VOID, s8 *pcfotail)
317{ 317{
318 PDM_ODM_T pDM_Odm = (PDM_ODM_T)pDM_VOID; 318 PDM_ODM_T pDM_Odm = (PDM_ODM_T)pDM_VOID;
319 PODM_PACKET_INFO_T pPktinfo = (PODM_PACKET_INFO_T)pPktinfo_VOID; 319 struct odm_packet_info *pPktinfo = (struct odm_packet_info *)pPktinfo_VOID;
320 PCFO_TRACKING pCfoTrack = &pDM_Odm->DM_CfoTrack; 320 PCFO_TRACKING pCfoTrack = &pDM_Odm->DM_CfoTrack;
321 u8 i; 321 u8 i;
322 322
323 if (!(pDM_Odm->SupportAbility & ODM_BB_CFO_TRACKING)) 323 if (!(pDM_Odm->SupportAbility & ODM_BB_CFO_TRACKING))
324 return; 324 return;
325 325
326 if (pPktinfo->StationID != 0) { 326 if (pPktinfo->station_id != 0) {
327 /* 3 Update CFO report for path-A & path-B */ 327 /* 3 Update CFO report for path-A & path-B */
328 /* Only paht-A and path-B have CFO tail and short CFO */ 328 /* Only paht-A and path-B have CFO tail and short CFO */
329 for (i = ODM_RF_PATH_A; i <= ODM_RF_PATH_B; i++) 329 for (i = ODM_RF_PATH_A; i <= ODM_RF_PATH_B; i++)
diff --git a/drivers/staging/rtl8723bs/hal/odm_HWConfig.c b/drivers/staging/rtl8723bs/hal/odm_HWConfig.c
index 8dd6da8a4e26..b3f4c237e903 100644
--- a/drivers/staging/rtl8723bs/hal/odm_HWConfig.c
+++ b/drivers/staging/rtl8723bs/hal/odm_HWConfig.c
@@ -93,7 +93,7 @@ static void odm_RxPhyStatus92CSeries_Parsing(
93 PDM_ODM_T pDM_Odm, 93 PDM_ODM_T pDM_Odm,
94 PODM_PHY_INFO_T pPhyInfo, 94 PODM_PHY_INFO_T pPhyInfo,
95 u8 *pPhyStatus, 95 u8 *pPhyStatus,
96 PODM_PACKET_INFO_T pPktinfo 96 struct odm_packet_info *pPktinfo
97) 97)
98{ 98{
99 u8 i, Max_spatial_stream; 99 u8 i, Max_spatial_stream;
@@ -106,7 +106,7 @@ static void odm_RxPhyStatus92CSeries_Parsing(
106 u8 LNA_idx, VGA_idx; 106 u8 LNA_idx, VGA_idx;
107 PPHY_STATUS_RPT_8192CD_T pPhyStaRpt = (PPHY_STATUS_RPT_8192CD_T)pPhyStatus; 107 PPHY_STATUS_RPT_8192CD_T pPhyStaRpt = (PPHY_STATUS_RPT_8192CD_T)pPhyStatus;
108 108
109 isCCKrate = pPktinfo->DataRate <= DESC_RATE11M; 109 isCCKrate = pPktinfo->data_rate <= DESC_RATE11M;
110 pPhyInfo->RxMIMOSignalQuality[ODM_RF_PATH_A] = -1; 110 pPhyInfo->RxMIMOSignalQuality[ODM_RF_PATH_A] = -1;
111 pPhyInfo->RxMIMOSignalQuality[ODM_RF_PATH_B] = -1; 111 pPhyInfo->RxMIMOSignalQuality[ODM_RF_PATH_B] = -1;
112 112
@@ -215,7 +215,7 @@ static void odm_RxPhyStatus92CSeries_Parsing(
215 /* */ 215 /* */
216 /* (3)EVM of HT rate */ 216 /* (3)EVM of HT rate */
217 /* */ 217 /* */
218 if (pPktinfo->DataRate >= DESC_RATEMCS8 && pPktinfo->DataRate <= DESC_RATEMCS15) 218 if (pPktinfo->data_rate >= DESC_RATEMCS8 && pPktinfo->data_rate <= DESC_RATEMCS15)
219 Max_spatial_stream = 2; /* both spatial stream make sense */ 219 Max_spatial_stream = 2; /* both spatial stream make sense */
220 else 220 else
221 Max_spatial_stream = 1; /* only spatial stream 1 makes sense */ 221 Max_spatial_stream = 1; /* only spatial stream 1 makes sense */
@@ -267,7 +267,7 @@ static void odm_RxPhyStatus92CSeries_Parsing(
267} 267}
268 268
269static void odm_Process_RSSIForDM( 269static void odm_Process_RSSIForDM(
270 PDM_ODM_T pDM_Odm, PODM_PHY_INFO_T pPhyInfo, PODM_PACKET_INFO_T pPktinfo 270 PDM_ODM_T pDM_Odm, PODM_PHY_INFO_T pPhyInfo, struct odm_packet_info *pPktinfo
271) 271)
272{ 272{
273 273
@@ -279,22 +279,22 @@ static void odm_Process_RSSIForDM(
279 PSTA_INFO_T pEntry; 279 PSTA_INFO_T pEntry;
280 280
281 281
282 if (pPktinfo->StationID == 0xFF) 282 if (pPktinfo->station_id == 0xFF)
283 return; 283 return;
284 284
285 pEntry = pDM_Odm->pODM_StaInfo[pPktinfo->StationID]; 285 pEntry = pDM_Odm->pODM_StaInfo[pPktinfo->station_id];
286 286
287 if (!IS_STA_VALID(pEntry)) 287 if (!IS_STA_VALID(pEntry))
288 return; 288 return;
289 289
290 if ((!pPktinfo->bPacketMatchBSSID)) 290 if ((!pPktinfo->bssid_match))
291 return; 291 return;
292 292
293 if (pPktinfo->bPacketBeacon) 293 if (pPktinfo->is_beacon)
294 pDM_Odm->PhyDbgInfo.NumQryBeaconPkt++; 294 pDM_Odm->PhyDbgInfo.NumQryBeaconPkt++;
295 295
296 isCCKrate = ((pPktinfo->DataRate <= DESC_RATE11M)) ? true : false; 296 isCCKrate = ((pPktinfo->data_rate <= DESC_RATE11M)) ? true : false;
297 pDM_Odm->RxRate = pPktinfo->DataRate; 297 pDM_Odm->RxRate = pPktinfo->data_rate;
298 298
299 /* Statistic for antenna/path diversity------------------ */ 299 /* Statistic for antenna/path diversity------------------ */
300 if (pDM_Odm->SupportAbility & ODM_BB_ANT_DIV) { 300 if (pDM_Odm->SupportAbility & ODM_BB_ANT_DIV) {
@@ -307,7 +307,7 @@ static void odm_Process_RSSIForDM(
307 UndecoratedSmoothedOFDM = pEntry->rssi_stat.UndecoratedSmoothedOFDM; 307 UndecoratedSmoothedOFDM = pEntry->rssi_stat.UndecoratedSmoothedOFDM;
308 UndecoratedSmoothedPWDB = pEntry->rssi_stat.UndecoratedSmoothedPWDB; 308 UndecoratedSmoothedPWDB = pEntry->rssi_stat.UndecoratedSmoothedPWDB;
309 309
310 if (pPktinfo->bPacketToSelf || pPktinfo->bPacketBeacon) { 310 if (pPktinfo->to_self || pPktinfo->is_beacon) {
311 311
312 if (!isCCKrate) { /* ofdm rate */ 312 if (!isCCKrate) { /* ofdm rate */
313 if (pPhyInfo->RxMIMOSignalStrength[ODM_RF_PATH_B] == 0) { 313 if (pPhyInfo->RxMIMOSignalStrength[ODM_RF_PATH_B] == 0) {
@@ -424,7 +424,7 @@ static void ODM_PhyStatusQuery_92CSeries(
424 PDM_ODM_T pDM_Odm, 424 PDM_ODM_T pDM_Odm,
425 PODM_PHY_INFO_T pPhyInfo, 425 PODM_PHY_INFO_T pPhyInfo,
426 u8 *pPhyStatus, 426 u8 *pPhyStatus,
427 PODM_PACKET_INFO_T pPktinfo 427 struct odm_packet_info *pPktinfo
428) 428)
429{ 429{
430 430
@@ -438,7 +438,7 @@ void ODM_PhyStatusQuery(
438 PDM_ODM_T pDM_Odm, 438 PDM_ODM_T pDM_Odm,
439 PODM_PHY_INFO_T pPhyInfo, 439 PODM_PHY_INFO_T pPhyInfo,
440 u8 *pPhyStatus, 440 u8 *pPhyStatus,
441 PODM_PACKET_INFO_T pPktinfo 441 struct odm_packet_info *pPktinfo
442) 442)
443{ 443{
444 444
diff --git a/drivers/staging/rtl8723bs/hal/odm_HWConfig.h b/drivers/staging/rtl8723bs/hal/odm_HWConfig.h
index f029922d12f0..ff0373eba495 100644
--- a/drivers/staging/rtl8723bs/hal/odm_HWConfig.h
+++ b/drivers/staging/rtl8723bs/hal/odm_HWConfig.h
@@ -133,7 +133,7 @@ void ODM_PhyStatusQuery(
133 PDM_ODM_T pDM_Odm, 133 PDM_ODM_T pDM_Odm,
134 PODM_PHY_INFO_T pPhyInfo, 134 PODM_PHY_INFO_T pPhyInfo,
135 u8 *pPhyStatus, 135 u8 *pPhyStatus,
136 PODM_PACKET_INFO_T pPktinfo 136 struct odm_packet_info *pPktinfo
137); 137);
138 138
139HAL_STATUS ODM_ConfigRFWithTxPwrTrackHeaderFile(PDM_ODM_T pDM_Odm); 139HAL_STATUS ODM_ConfigRFWithTxPwrTrackHeaderFile(PDM_ODM_T pDM_Odm);
diff --git a/drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c b/drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c
index 7a294102b63b..6280a2e21dc2 100644
--- a/drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c
+++ b/drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c
@@ -97,45 +97,48 @@ static void update_recvframe_phyinfo(union recv_frame *precvframe,
97 u8 *rx_bssid; 97 u8 *rx_bssid;
98 u8 *rx_ra; 98 u8 *rx_ra;
99 u8 *my_hwaddr; 99 u8 *my_hwaddr;
100
101 ODM_PACKET_INFO_T pkt_info;
102 u8 *sa = NULL; 100 u8 *sa = NULL;
101
102 struct odm_packet_info pkt_info = {
103 .data_rate = 0x00,
104 .station_id = 0x00,
105 .bssid_match = false,
106 .to_self = false,
107 .is_beacon = false,
108 };
109
103 /* _irqL irqL; */ 110 /* _irqL irqL; */
104 struct sta_priv *pstapriv; 111 struct sta_priv *pstapriv;
105 struct sta_info *psta; 112 struct sta_info *psta;
106 113
107 pkt_info.bPacketMatchBSSID = false;
108 pkt_info.bPacketToSelf = false;
109 pkt_info.bPacketBeacon = false;
110
111 wlanhdr = get_recvframe_data(precvframe); 114 wlanhdr = get_recvframe_data(precvframe);
112 my_bssid = get_bssid(&padapter->mlmepriv); 115 my_bssid = get_bssid(&padapter->mlmepriv);
113 rx_bssid = get_hdr_bssid(wlanhdr); 116 rx_bssid = get_hdr_bssid(wlanhdr);
114 pkt_info.bPacketMatchBSSID = ((!IsFrameTypeCtrl(wlanhdr)) && 117 pkt_info.bssid_match = ((!IsFrameTypeCtrl(wlanhdr)) &&
115 !pattrib->icv_err && !pattrib->crc_err && 118 !pattrib->icv_err && !pattrib->crc_err &&
116 !ether_addr_equal(rx_bssid, my_bssid)); 119 !ether_addr_equal(rx_bssid, my_bssid));
117 120
118 rx_ra = get_ra(wlanhdr); 121 rx_ra = get_ra(wlanhdr);
119 my_hwaddr = myid(&padapter->eeprompriv); 122 my_hwaddr = myid(&padapter->eeprompriv);
120 pkt_info.bPacketToSelf = pkt_info.bPacketMatchBSSID && 123 pkt_info.to_self = pkt_info.bssid_match &&
121 !ether_addr_equal(rx_ra, my_hwaddr); 124 !ether_addr_equal(rx_ra, my_hwaddr);
122 125
123 126
124 pkt_info.bPacketBeacon = pkt_info.bPacketMatchBSSID && 127 pkt_info.is_beacon = pkt_info.bssid_match &&
125 (GetFrameSubType(wlanhdr) == WIFI_BEACON); 128 (GetFrameSubType(wlanhdr) == WIFI_BEACON);
126 129
127 sa = get_ta(wlanhdr); 130 sa = get_ta(wlanhdr);
128 131
129 pkt_info.StationID = 0xFF; 132 pkt_info.station_id = 0xFF;
130 133
131 pstapriv = &padapter->stapriv; 134 pstapriv = &padapter->stapriv;
132 psta = rtw_get_stainfo(pstapriv, sa); 135 psta = rtw_get_stainfo(pstapriv, sa);
133 if (psta) { 136 if (psta) {
134 pkt_info.StationID = psta->mac_id; 137 pkt_info.station_id = psta->mac_id;
135 /* DBG_8192C("%s ==> StationID(%d)\n", 138 /* DBG_8192C("%s ==> StationID(%d)\n",
136 * __func__, pkt_info.StationID); */ 139 * __func__, pkt_info.station_id); */
137 } 140 }
138 pkt_info.DataRate = pattrib->data_rate; 141 pkt_info.data_rate = pattrib->data_rate;
139 142
140 /* rtl8723b_query_rx_phy_status(precvframe, pphy_status); */ 143 /* rtl8723b_query_rx_phy_status(precvframe, pphy_status); */
141 /* spin_lock_bh(&p_hal_data->odm_stainfo_lock); */ 144 /* spin_lock_bh(&p_hal_data->odm_stainfo_lock); */
@@ -146,14 +149,14 @@ static void update_recvframe_phyinfo(union recv_frame *precvframe,
146 /* spin_unlock_bh(&p_hal_data->odm_stainfo_lock); */ 149 /* spin_unlock_bh(&p_hal_data->odm_stainfo_lock); */
147 precvframe->u.hdr.psta = NULL; 150 precvframe->u.hdr.psta = NULL;
148 if ( 151 if (
149 pkt_info.bPacketMatchBSSID && 152 pkt_info.bssid_match &&
150 (check_fwstate(&padapter->mlmepriv, WIFI_AP_STATE) == true) 153 (check_fwstate(&padapter->mlmepriv, WIFI_AP_STATE) == true)
151 ) { 154 ) {
152 if (psta) { 155 if (psta) {
153 precvframe->u.hdr.psta = psta; 156 precvframe->u.hdr.psta = psta;
154 rtl8723b_process_phy_info(padapter, precvframe); 157 rtl8723b_process_phy_info(padapter, precvframe);
155 } 158 }
156 } else if (pkt_info.bPacketToSelf || pkt_info.bPacketBeacon) { 159 } else if (pkt_info.to_self || pkt_info.is_beacon) {
157 u32 adhoc_state = WIFI_ADHOC_STATE | WIFI_ADHOC_MASTER_STATE; 160 u32 adhoc_state = WIFI_ADHOC_STATE | WIFI_ADHOC_MASTER_STATE;
158 if (check_fwstate(&padapter->mlmepriv, adhoc_state)) 161 if (check_fwstate(&padapter->mlmepriv, adhoc_state))
159 if (psta) 162 if (psta)