qcacld-3.0: Fix kernel checkpatch warnings in wlan_hdd_power.c
Fix kernel checkpatch warnings in wlan_hdd_power.c Change-Id: I0a22c2c1ebc86ada69ff271cbd68f2166e08b541 CRs-Fixed: 2024274
This commit is contained in:

committed by
snandini

parent
8d8d9fe485
commit
cf11bae9c0
@@ -233,7 +233,7 @@ void hdd_disable_and_flush_mc_addr_list(hdd_adapter_t *adapter,
|
|||||||
enum pmo_offload_trigger trigger);
|
enum pmo_offload_trigger trigger);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* wlan_hdd_cfg80211_update_replay_counter_callback() - replay counter callback
|
* wlan_hdd_cfg80211_update_replay_counter_cb() - replay counter callback
|
||||||
* @cb_ctx: Callback context as void* as PMO do not about HDD adapter type
|
* @cb_ctx: Callback context as void* as PMO do not about HDD adapter type
|
||||||
* @gtk_rsp_param: Pointer to gtk offload response parameter
|
* @gtk_rsp_param: Pointer to gtk offload response parameter
|
||||||
*
|
*
|
||||||
@@ -241,7 +241,7 @@ void hdd_disable_and_flush_mc_addr_list(hdd_adapter_t *adapter,
|
|||||||
*
|
*
|
||||||
* Return: none
|
* Return: none
|
||||||
*/
|
*/
|
||||||
void wlan_hdd_cfg80211_update_replay_counter_callback(
|
void wlan_hdd_cfg80211_update_replay_counter_cb(
|
||||||
void *cb_ctx,
|
void *cb_ctx,
|
||||||
struct pmo_gtk_rsp_params *gtk_rsp_param);
|
struct pmo_gtk_rsp_params *gtk_rsp_param);
|
||||||
|
|
||||||
|
@@ -17037,7 +17037,7 @@ wlan_hdd_cfg80211_update_ft_ies(struct wiphy *wiphy,
|
|||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
void wlan_hdd_cfg80211_update_replay_counter_callback(
|
void wlan_hdd_cfg80211_update_replay_counter_cb(
|
||||||
void *cb_ctx, struct pmo_gtk_rsp_params *gtk_rsp_param)
|
void *cb_ctx, struct pmo_gtk_rsp_params *gtk_rsp_param)
|
||||||
|
|
||||||
{
|
{
|
||||||
|
@@ -217,6 +217,7 @@ static const struct wiphy_wowlan_support wowlan_support_reg_init = {
|
|||||||
#endif
|
#endif
|
||||||
|
|
||||||
/* internal function declaration */
|
/* internal function declaration */
|
||||||
|
struct notifier_block hdd_netdev_notifier;
|
||||||
|
|
||||||
struct sock *cesium_nl_srv_sock;
|
struct sock *cesium_nl_srv_sock;
|
||||||
#ifdef FEATURE_WLAN_AUTO_SHUTDOWN
|
#ifdef FEATURE_WLAN_AUTO_SHUTDOWN
|
||||||
|
@@ -166,7 +166,7 @@ static void hdd_disable_gtk_offload(hdd_adapter_t *adapter)
|
|||||||
|
|
||||||
/* ensure to get gtk rsp first before disable it*/
|
/* ensure to get gtk rsp first before disable it*/
|
||||||
gtk_rsp_request.callback =
|
gtk_rsp_request.callback =
|
||||||
wlan_hdd_cfg80211_update_replay_counter_callback;
|
wlan_hdd_cfg80211_update_replay_counter_cb;
|
||||||
/* Passing as void* as PMO does not know legacy HDD adapter type */
|
/* Passing as void* as PMO does not know legacy HDD adapter type */
|
||||||
gtk_rsp_request.callback_context =
|
gtk_rsp_request.callback_context =
|
||||||
(void *)adapter;
|
(void *)adapter;
|
||||||
@@ -397,8 +397,7 @@ void hdd_enable_ns_offload(hdd_adapter_t *adapter,
|
|||||||
ns_req->ipv6_addr_type, &ns_req->count);
|
ns_req->ipv6_addr_type, &ns_req->count);
|
||||||
if (err) {
|
if (err) {
|
||||||
hdd_disable_ns_offload(adapter, trigger);
|
hdd_disable_ns_offload(adapter, trigger);
|
||||||
hdd_debug("Reached max supported addresses and not enabling "
|
hdd_debug("Max supported addresses: disabling NS offload");
|
||||||
"NS offload");
|
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -407,8 +406,7 @@ void hdd_enable_ns_offload(hdd_adapter_t *adapter,
|
|||||||
ns_req->ipv6_addr_type, &ns_req->count);
|
ns_req->ipv6_addr_type, &ns_req->count);
|
||||||
if (err) {
|
if (err) {
|
||||||
hdd_disable_ns_offload(adapter, trigger);
|
hdd_disable_ns_offload(adapter, trigger);
|
||||||
hdd_debug("Reached max supported addresses and not enabling "
|
hdd_debug("Max supported addresses: disabling NS offload");
|
||||||
"NS offload");
|
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1046,7 +1044,9 @@ hdd_update_conn_state_mask(hdd_adapter_t *adapter, uint32_t *conn_state_mask)
|
|||||||
|
|
||||||
eConnectionState connState;
|
eConnectionState connState;
|
||||||
hdd_station_ctx_t *sta_ctx;
|
hdd_station_ctx_t *sta_ctx;
|
||||||
|
|
||||||
sta_ctx = WLAN_HDD_GET_STATION_CTX_PTR(adapter);
|
sta_ctx = WLAN_HDD_GET_STATION_CTX_PTR(adapter);
|
||||||
|
|
||||||
connState = sta_ctx->conn_info.connState;
|
connState = sta_ctx->conn_info.connState;
|
||||||
|
|
||||||
if (connState == eConnectionState_Associated ||
|
if (connState == eConnectionState_Associated ||
|
||||||
@@ -1314,12 +1314,12 @@ QDF_STATUS hdd_wlan_shutdown(void)
|
|||||||
|
|
||||||
#ifdef FEATURE_WLAN_DIAG_SUPPORT
|
#ifdef FEATURE_WLAN_DIAG_SUPPORT
|
||||||
/**
|
/**
|
||||||
* hdd_wlan_ssr_reinit_event()- send ssr reinit state
|
* hdd_wlan_ssr_reinit_event()- send ssr reinit state
|
||||||
*
|
*
|
||||||
* This Function send send ssr reinit state diag event
|
* This Function send send ssr reinit state diag event
|
||||||
*
|
*
|
||||||
* Return: void.
|
* Return: void.
|
||||||
*/
|
*/
|
||||||
static void hdd_wlan_ssr_reinit_event(void)
|
static void hdd_wlan_ssr_reinit_event(void)
|
||||||
{
|
{
|
||||||
WLAN_HOST_DIAG_EVENT_DEF(ssr_reinit, struct host_event_wlan_ssr_reinit);
|
WLAN_HOST_DIAG_EVENT_DEF(ssr_reinit, struct host_event_wlan_ssr_reinit);
|
||||||
@@ -1377,9 +1377,9 @@ QDF_STATUS hdd_wlan_re_init(void)
|
|||||||
pAdapter = hdd_get_adapter(pHddCtx, QDF_SAP_MODE);
|
pAdapter = hdd_get_adapter(pHddCtx, QDF_SAP_MODE);
|
||||||
if (!pAdapter) {
|
if (!pAdapter) {
|
||||||
pAdapter = hdd_get_adapter(pHddCtx, QDF_IBSS_MODE);
|
pAdapter = hdd_get_adapter(pHddCtx, QDF_IBSS_MODE);
|
||||||
if (!pAdapter) {
|
if (!pAdapter)
|
||||||
hdd_err("Failed to get Adapter!");
|
hdd_err("Failed to get Adapter!");
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1518,6 +1518,7 @@ int wlan_hdd_set_powersave(hdd_adapter_t *adapter,
|
|||||||
static void wlan_hdd_print_suspend_fail_stats(hdd_context_t *hdd_ctx)
|
static void wlan_hdd_print_suspend_fail_stats(hdd_context_t *hdd_ctx)
|
||||||
{
|
{
|
||||||
struct suspend_resume_stats *stats = &hdd_ctx->suspend_resume_stats;
|
struct suspend_resume_stats *stats = &hdd_ctx->suspend_resume_stats;
|
||||||
|
|
||||||
hdd_err("ipa:%d, radar:%d, roam:%d, scan:%d, initial_wakeup:%d",
|
hdd_err("ipa:%d, radar:%d, roam:%d, scan:%d, initial_wakeup:%d",
|
||||||
stats->suspend_fail[SUSPEND_FAIL_IPA],
|
stats->suspend_fail[SUSPEND_FAIL_IPA],
|
||||||
stats->suspend_fail[SUSPEND_FAIL_RADAR],
|
stats->suspend_fail[SUSPEND_FAIL_RADAR],
|
||||||
@@ -2063,7 +2064,8 @@ static int __wlan_hdd_cfg80211_set_txpower(struct wiphy *wiphy,
|
|||||||
/* Automatically determine transmit power */
|
/* Automatically determine transmit power */
|
||||||
case NL80211_TX_POWER_AUTOMATIC:
|
case NL80211_TX_POWER_AUTOMATIC:
|
||||||
/* Fall through */
|
/* Fall through */
|
||||||
case NL80211_TX_POWER_LIMITED: /* Limit TX power by the mBm parameter */
|
case NL80211_TX_POWER_LIMITED:
|
||||||
|
/* Limit TX power by the mBm parameter */
|
||||||
if (sme_set_max_tx_power(hHal, bssid, selfMac, dbm) !=
|
if (sme_set_max_tx_power(hHal, bssid, selfMac, dbm) !=
|
||||||
QDF_STATUS_SUCCESS) {
|
QDF_STATUS_SUCCESS) {
|
||||||
hdd_err("Setting maximum tx power failed");
|
hdd_err("Setting maximum tx power failed");
|
||||||
@@ -2074,7 +2076,6 @@ static int __wlan_hdd_cfg80211_set_txpower(struct wiphy *wiphy,
|
|||||||
case NL80211_TX_POWER_FIXED: /* Fix TX power to the mBm parameter */
|
case NL80211_TX_POWER_FIXED: /* Fix TX power to the mBm parameter */
|
||||||
hdd_err("NL80211_TX_POWER_FIXED not supported");
|
hdd_err("NL80211_TX_POWER_FIXED not supported");
|
||||||
return -EOPNOTSUPP;
|
return -EOPNOTSUPP;
|
||||||
break;
|
|
||||||
|
|
||||||
default:
|
default:
|
||||||
hdd_err("Invalid power setting type %d", type);
|
hdd_err("Invalid power setting type %d", type);
|
||||||
@@ -2100,6 +2101,7 @@ int wlan_hdd_cfg80211_set_txpower(struct wiphy *wiphy,
|
|||||||
int dbm)
|
int dbm)
|
||||||
{
|
{
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
cds_ssr_protect(__func__);
|
cds_ssr_protect(__func__);
|
||||||
ret = __wlan_hdd_cfg80211_set_txpower(wiphy,
|
ret = __wlan_hdd_cfg80211_set_txpower(wiphy,
|
||||||
wdev,
|
wdev,
|
||||||
|
Reference in New Issue
Block a user