From 05bc2c2396ab3e3c2ec7d9f38c2d345510a41bef Mon Sep 17 00:00:00 2001 From: Srinivas Dasari Date: Fri, 23 Aug 2019 17:54:53 +0530 Subject: [PATCH] qcacld-3.0: Correct the order of arguments to roam_triggers_to_sme Vdev_id and ENABLE_ROAM_TRIGGERS_ALL are passed to hdd_send_roam_triggers_to_sme in reverse order. Pass them in correct the order. Change-Id: I79d2a843ee9f237620d31dbf91e3c57750e66e88 CRs-Fixed: 2514476 --- core/hdd/src/wlan_hdd_cfg80211.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/core/hdd/src/wlan_hdd_cfg80211.c b/core/hdd/src/wlan_hdd_cfg80211.c index 4d2fcc470a71..d9187c842115 100644 --- a/core/hdd/src/wlan_hdd_cfg80211.c +++ b/core/hdd/src/wlan_hdd_cfg80211.c @@ -4441,6 +4441,7 @@ hdd_clear_roam_control_config(struct hdd_context *hdd_ctx, QDF_STATUS status; struct nlattr *tb2[QCA_ATTR_ROAM_CONTROL_MAX + 1]; mac_handle_t mac_handle = hdd_ctx->mac_handle; + uint32_t value; /* The command must carry PARAM_ROAM_CONTROL_CONFIG */ if (!tb[PARAM_ROAM_CONTROL_CONFIG]) { @@ -4465,11 +4466,9 @@ hdd_clear_roam_control_config(struct hdd_context *hdd_ctx, return qdf_status_to_os_return(status); } - hdd_debug("Reset roam trigger bitmap to 0x%x", - ENABLE_ROAM_TRIGGERS_ALL); - status = hdd_send_roam_triggers_to_sme(hdd_ctx, - ENABLE_ROAM_TRIGGERS_ALL, - vdev_id); + value = ENABLE_ROAM_TRIGGERS_ALL; + hdd_debug("Reset roam trigger bitmap to 0x%x", value); + status = hdd_send_roam_triggers_to_sme(hdd_ctx, vdev_id, value); if (QDF_IS_STATUS_ERROR(status)) { hdd_err("failed to restore roam trigger bitmap"); return qdf_status_to_os_return(status);