From 2fc8cced817099448ba577fa6b4b3acd159db7de Mon Sep 17 00:00:00 2001 From: Jyoti Kumari Date: Fri, 25 Sep 2020 20:29:45 +0530 Subject: [PATCH] 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 --- components/mlme/core/src/wlan_mlme_main.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/components/mlme/core/src/wlan_mlme_main.c b/components/mlme/core/src/wlan_mlme_main.c index 932aca077936..accb9ddd1817 100644 --- a/components/mlme/core/src/wlan_mlme_main.c +++ b/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) {