From 6eeb76ce926fcc6e0d50c0747a751070fd659ffb Mon Sep 17 00:00:00 2001 From: Vignesh Mohan Date: Tue, 11 Dec 2018 13:47:48 +0530 Subject: [PATCH] qcacmn: Remove subchannel marking fail prints in PO The cfg tool command to enable/disable subchannel marking called a WMI to be sent to FW without checking if dfs is offloaded to FW. This added fail prints in log for partial offload and direct attach even if subchannel marking was set. Fix the cfg command to send a WMI to FW only if dfs is offloaded. Change-Id: Ifae574ed6aeb49c4058d89374fdcc81b8d43a5bc CRs-Fixed: 2334258 --- umac/dfs/core/src/misc/dfs_process_radar_found_ind.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/umac/dfs/core/src/misc/dfs_process_radar_found_ind.c b/umac/dfs/core/src/misc/dfs_process_radar_found_ind.c index 18e57f6f2751..33b6159d2959 100644 --- a/umac/dfs/core/src/misc/dfs_process_radar_found_ind.c +++ b/umac/dfs/core/src/misc/dfs_process_radar_found_ind.c @@ -123,7 +123,7 @@ int dfs_set_nol_subchannel_marking(struct wlan_dfs *dfs, bool nol_subchannel_marking) { - QDF_STATUS status; + QDF_STATUS status = QDF_STATUS_SUCCESS; if (!dfs) return -EIO; @@ -131,8 +131,9 @@ int dfs_set_nol_subchannel_marking(struct wlan_dfs *dfs, dfs->dfs_use_nol_subchannel_marking = nol_subchannel_marking; dfs_info(dfs, WLAN_DEBUG_DFS_ALWAYS, "NOL subchannel marking is %s ", (nol_subchannel_marking) ? "set" : "disabled"); - status = tgt_dfs_send_subchan_marking(dfs->dfs_pdev_obj, - nol_subchannel_marking); + if (dfs->dfs_is_offload_enabled) + status = tgt_dfs_send_subchan_marking(dfs->dfs_pdev_obj, + nol_subchannel_marking); return qdf_status_to_os_return(status); }