From 56acf4386f07c9b39a002d1faf9fdb4f9611ffbf Mon Sep 17 00:00:00 2001 From: Jeff Johnson Date: Mon, 28 Aug 2017 12:00:20 -0700 Subject: [PATCH] qcacld-3.0: Replace hdd_context_t in wlan_hdd_he.[ch] 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_context_t typedef does not meet any of those criteria, so replace references to it with a reference to the underlying struct. Change-Id: Ib8b11a064c448c1f1995bf192c6a70f49391cbd5 CRs-Fixed: 2100194 --- core/hdd/inc/wlan_hdd_he.h | 8 ++++---- core/hdd/src/wlan_hdd_he.c | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/core/hdd/inc/wlan_hdd_he.h b/core/hdd/inc/wlan_hdd_he.h index d7394858e652..f92d8df7c695 100644 --- a/core/hdd/inc/wlan_hdd_he.h +++ b/core/hdd/inc/wlan_hdd_he.h @@ -99,7 +99,7 @@ void wlan_hdd_check_11ax_support(struct hdd_beacon_data *beacon, * * Return: None */ -void hdd_he_print_ini_config(hdd_context_t *hdd_ctx); +void hdd_he_print_ini_config(struct hdd_context *hdd_ctx); /** * hdd_update_he_cap_in_cfg() - update HE cap in global CFG @@ -110,7 +110,7 @@ void hdd_he_print_ini_config(hdd_context_t *hdd_ctx); * * Return: 0 on success and errno on failure */ -int hdd_update_he_cap_in_cfg(hdd_context_t *hdd_ctx); +int hdd_update_he_cap_in_cfg(struct hdd_context *hdd_ctx); /** * hdd_he_set_sme_config() - set HE related SME config param @@ -154,11 +154,11 @@ static inline void wlan_hdd_check_11ax_support(struct hdd_beacon_data *beacon, { } -static inline void hdd_he_print_ini_config(hdd_context_t *hdd_ctx) +static inline void hdd_he_print_ini_config(struct hdd_context *hdd_ctx) { } -static inline int hdd_update_he_cap_in_cfg(hdd_context_t *hdd_ctx) +static inline int hdd_update_he_cap_in_cfg(struct hdd_context *hdd_ctx) { return 0; } diff --git a/core/hdd/src/wlan_hdd_he.c b/core/hdd/src/wlan_hdd_he.c index 6466a84060ec..74e008739a6c 100644 --- a/core/hdd/src/wlan_hdd_he.c +++ b/core/hdd/src/wlan_hdd_he.c @@ -252,7 +252,7 @@ void wlan_hdd_check_11ax_support(beacon_data_t *beacon, tsap_Config_t *config) config->SapHw_mode = eCSR_DOT11_MODE_11ax; } -void hdd_he_print_ini_config(hdd_context_t *hdd_ctx) +void hdd_he_print_ini_config(struct hdd_context *hdd_ctx) { hdd_info("Name = [%s] Value = [%d]", CFG_ENABLE_UL_MIMO_NAME, hdd_ctx->config->enable_ul_mimo); @@ -262,7 +262,7 @@ void hdd_he_print_ini_config(hdd_context_t *hdd_ctx) hdd_ctx->config->he_sta_obsspd); } -int hdd_update_he_cap_in_cfg(hdd_context_t *hdd_ctx) +int hdd_update_he_cap_in_cfg(struct hdd_context *hdd_ctx) { uint32_t val, val1 = 0; QDF_STATUS status; @@ -317,7 +317,7 @@ __wlan_hdd_cfg80211_get_he_cap(struct wiphy *wiphy, const void *data, int data_len) { - hdd_context_t *hdd_ctx = wiphy_priv(wiphy); + struct hdd_context *hdd_ctx = wiphy_priv(wiphy); int ret; QDF_STATUS status; struct sk_buff *reply_skb;