Browse Source

qcacld-3.0: Cleanup unnecessary log in roaming

When vdev is null in both mlme_get_roam_state() and
mlme_is_roam_invoke_in_progress(), error log is getting printed
redundantly.

Remove prints from both function as there is no additional info.

Change-Id: Ibc96ad957d947857777dad9df9ff766e68562dae
CRs-Fixed: 2785775
Jyoti Kumari 4 years ago
parent
commit
2fc8cced81
1 changed files with 2 additions and 6 deletions
  1. 2 6
      components/mlme/core/src/wlan_mlme_main.c

+ 2 - 6
components/mlme/core/src/wlan_mlme_main.c

@@ -89,10 +89,8 @@ bool mlme_is_roam_invoke_in_progress(struct wlan_objmgr_psoc *psoc,
 
 	vdev = wlan_objmgr_get_vdev_by_id_from_psoc(psoc, vdev_id,
 						    WLAN_MLME_NB_ID);
-	if (!vdev) {
-		mlme_legacy_err("Vdev object is NULL");
+	if (!vdev)
 		return false;
-	}
 
 	vdev_roam_params = mlme_get_roam_invoke_params(vdev);
 	if (!vdev_roam_params) {
@@ -2866,10 +2864,8 @@ mlme_get_roam_state(struct wlan_objmgr_psoc *psoc, uint8_t vdev_id)
 	vdev = wlan_objmgr_get_vdev_by_id_from_psoc(psoc, vdev_id,
 						    WLAN_MLME_OBJMGR_ID);
 
-	if (!vdev) {
-		mlme_legacy_err("vdev object is NULL");
+	if (!vdev)
 		return WLAN_ROAM_DEINIT;
-	}
 
 	mlme_priv = wlan_vdev_mlme_get_ext_hdl(vdev);
 	if (!mlme_priv) {