Browse Source

qcacld-3.0: Return correct status from csr_post_roam_state_change

Currently, sme_config_fast_roaming is called to set the
supplicant disabled roaming flag in the driver, which is
received over the vendor command
QCA_NL80211_VENDOR_SUBCMD_ROAMING. This supplicant disabled
roaming flag is used to toggle between RSO start and RSO stop.
When the driver is in disconnected state, the roaming module
would be in de-init state already. So the driver should return
success whenever a state change is received in disconnected
state. But currently the driver sends failure when its in
disconnected state. This results in sending failure status to
userspace.

Return QDF_SUCCESS status if any roam state change is requested
in disconnected state.

Change-Id: I520ab92b014eb1239e0164a39e0135e1bb4c2d70
CRs-Fixed: 2533019
Pragaspathi Thilagaraj 5 years ago
parent
commit
872bf8404f
1 changed files with 1 additions and 1 deletions
  1. 1 1
      core/sme/src/csr/csr_api_roam.c

+ 1 - 1
core/sme/src/csr/csr_api_roam.c

@@ -19427,7 +19427,7 @@ csr_handle_roam_state_change(struct mac_context *mac, uint8_t vdev_id,
 			     enum roam_offload_state requested_state,
 			     uint8_t reason)
 {
-	QDF_STATUS status = QDF_STATUS_E_INVAL;
+	QDF_STATUS status = QDF_STATUS_SUCCESS;
 
 	if (requested_state != ROAM_DEINIT &&
 	    !csr_is_conn_state_connected_infra(mac, vdev_id)) {