From a49c0ac4653da04561d93d46b57165d022409d4e Mon Sep 17 00:00:00 2001 From: Jeff Johnson Date: Tue, 15 Aug 2017 13:52:08 -0700 Subject: [PATCH] qcacld-3.0: Replace typedef hdd_pmf_stats_t The Linux Coding Style enumerates a few special cases where typedefs are useful, but stresses "NEVER EVER use a typedef unless you can clearly match one of those rules." The hdd_pmf_stats_t typedef does not meet any of those criteria, so replace it with a well named struct. Change-Id: I6e1f205f36ca35d58f5f8846d95d76976266a8bd CRs-Fixed: 2093258 --- core/hdd/inc/wlan_hdd_main.h | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/core/hdd/inc/wlan_hdd_main.h b/core/hdd/inc/wlan_hdd_main.h index 984a06d68f04..ce61aaa3716f 100644 --- a/core/hdd/inc/wlan_hdd_main.h +++ b/core/hdd/inc/wlan_hdd_main.h @@ -425,10 +425,15 @@ typedef struct hdd_tx_rx_stats { } hdd_tx_rx_stats_t; #ifdef WLAN_FEATURE_11W -typedef struct hdd_pmf_stats_s { +/** + * struct hdd_pmf_stats - Protected Management Frame statistics + * @numUnprotDeauthRx: Number of unprotected deauth frames received + * @numUnprotDisassocRx: Number of unprotected disassoc frames received + */ +struct hdd_pmf_stats { uint8_t numUnprotDeauthRx; uint8_t numUnprotDisassocRx; -} hdd_pmf_stats_t; +}; #endif /** @@ -447,7 +452,7 @@ struct hdd_stats { struct csr_per_chain_rssi_stats_info per_chain_rssi_stats; hdd_tx_rx_stats_t hddTxRxStats; #ifdef WLAN_FEATURE_11W - hdd_pmf_stats_t hddPmfStats; + struct hdd_pmf_stats hddPmfStats; #endif };