From c4c0d15a2c4a45e73672ba5b9510bece24a7f3a6 Mon Sep 17 00:00:00 2001 From: Liangwei Dong Date: Tue, 18 Oct 2022 15:05:43 +0800 Subject: [PATCH] qcacmn: Allow to set zero count action oui entry Allow to set zero count action oui entry to clear the target setting. Change-Id: I76af4d7067a0764cfae0a1a4c378c3cbcb572402 CRs-Fixed: 3316588 --- wmi/src/wmi_unified_action_oui_tlv.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/wmi/src/wmi_unified_action_oui_tlv.c b/wmi/src/wmi_unified_action_oui_tlv.c index 1bc8a3d425..6462679dd6 100644 --- a/wmi/src/wmi_unified_action_oui_tlv.c +++ b/wmi/src/wmi_unified_action_oui_tlv.c @@ -219,8 +219,7 @@ send_action_oui_cmd_tlv(wmi_unified_t wmi_handle, len = sizeof(*cmd); len += WMI_TLV_HDR_SIZE; /* Array of wmi_vendor_oui_ext structures */ - if (!no_oui_extns || - no_oui_extns > WMI_MAX_VENDOR_OUI_ACTION_SUPPORTED_PER_ACTION || + if (no_oui_extns > WMI_MAX_VENDOR_OUI_ACTION_SUPPORTED_PER_ACTION || (total_no_oui_extns > WMI_VENDOR_OUI_ACTION_MAX_ACTION_ID * WMI_MAX_VENDOR_OUI_ACTION_SUPPORTED_PER_ACTION)) { wmi_err("Invalid number of action oui extensions"); @@ -232,6 +231,8 @@ send_action_oui_cmd_tlv(wmi_unified_t wmi_handle, wmi_err("Invalid action id"); return QDF_STATUS_E_INVAL; } + wmi_debug("wmi action_id %d num %d total_num %d", action_id, + no_oui_extns, total_no_oui_extns); len += no_oui_extns * sizeof(*cmd_ext); len += WMI_TLV_HDR_SIZE; /* Variable length buffer */ @@ -286,6 +287,7 @@ send_action_oui_cmd_tlv(wmi_unified_t wmi_handle, if (!QDF_IS_STATUS_SUCCESS(status)) { wmi_buf_free(wmi_buf); wmi_buf = NULL; + wmi_err("failed to fill oui ext status %d", status); return QDF_STATUS_E_INVAL; }