diff options
author | Quytelda Kahja <quytelda@tamalin.org> | 2018-03-27 04:41:13 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-03-29 05:59:29 -0400 |
commit | 3df3602ac74b641f54149599c78d3606b467f58e (patch) | |
tree | a577967a94b804eb156061b3618e4819399b4692 | |
parent | 9f534f7ee8558dd327f63fe07a9127eb32789065 (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.h | 16 | ||||
-rw-r--r-- | drivers/staging/rtl8723bs/hal/odm_CfoTracking.c | 4 | ||||
-rw-r--r-- | drivers/staging/rtl8723bs/hal/odm_HWConfig.c | 26 | ||||
-rw-r--r-- | drivers/staging/rtl8723bs/hal/odm_HWConfig.h | 2 | ||||
-rw-r--r-- | drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c | 37 |
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 | ||
265 | typedef struct _ODM_Per_Pkt_Info_ { | 265 | struct 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 | ||
276 | typedef struct _ODM_Phy_Dbg_Info_ { | 274 | typedef 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) | |||
316 | void ODM_ParsingCFO(void *pDM_VOID, void *pPktinfo_VOID, s8 *pcfotail) | 316 | void 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 | ||
269 | static void odm_Process_RSSIForDM( | 269 | static 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 | ||
139 | HAL_STATUS ODM_ConfigRFWithTxPwrTrackHeaderFile(PDM_ODM_T pDM_Odm); | 139 | HAL_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) |