From 1a71e634a81353e5047bae368c8a25a39dde1fc6 Mon Sep 17 00:00:00 2001 From: Vivek Date: Tue, 11 Sep 2018 12:06:16 +0530 Subject: [PATCH] qcacmn: Remove redundant check when moving cmd from pending to active The API wlan_serialization_is_active_cmd_allowed should used when a serialization request is received and we need to decide if a command can be moved to active queue or pending queue. However when moving pending to active, the logic to move pending to active is implemented in the function itself and is different from what is implemented in wlan_serialization_is_active_cmd_allowed and thus should not be used. Change-Id: If04edf4a0012d9ae6cbbea9606b0e6b962f10b65 CRs-Fixed: 2312991 --- .../serialization/src/wlan_serialization_non_scan.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/umac/cmn_services/serialization/src/wlan_serialization_non_scan.c b/umac/cmn_services/serialization/src/wlan_serialization_non_scan.c index 91551663209d..83a24070ea1b 100644 --- a/umac/cmn_services/serialization/src/wlan_serialization_non_scan.c +++ b/umac/cmn_services/serialization/src/wlan_serialization_non_scan.c @@ -285,12 +285,6 @@ wlan_ser_move_multiple_non_scan_pending_to_active( qdf_mem_copy(&cmd_to_remove, &pending_cmd_list->cmd, sizeof(struct wlan_serialization_command)); - if (!wlan_serialization_is_active_cmd_allowed(&cmd_to_remove)) { - ser_debug("Can't add to active queue cmd_id:%d", - cmd_to_remove.cmd_id); - continue; - } - qdf_status = wlan_ser_remove_non_scan_cmd(ser_pdev_obj, &pending_cmd_list, &cmd_to_remove,