From e7e4e1fbecfa921af2ca0cdf743291b6407db97a Mon Sep 17 00:00:00 2001 From: Jianmin Zhu Date: Mon, 19 Apr 2021 08:56:03 +0800 Subject: [PATCH] qcacld-3.0: Return 0 if rx aggregation missed In QCA_NL80211_VENDOR_SUBCMD_SET_WIFI_CONFIGURATION cmd parameters, If attribute QCA_WLAN_VENDOR_ATTR_CONFIG_RX_MPDU_AGGREGATION is missed, should return 0 instead of failed. Change-Id: I5e570b9237768800e06d839814919d058a5e1d1f CRs-Fixed: 2926104 --- core/hdd/src/wlan_hdd_cfg80211.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/core/hdd/src/wlan_hdd_cfg80211.c b/core/hdd/src/wlan_hdd_cfg80211.c index e406d0f8e08f..55cab62f85c3 100644 --- a/core/hdd/src/wlan_hdd_cfg80211.c +++ b/core/hdd/src/wlan_hdd_cfg80211.c @@ -7617,10 +7617,8 @@ static int hdd_config_mpdu_aggregation(struct hdd_adapter *adapter, uint8_t rx_size; QDF_STATUS status; - if (!rx_attr) { - hdd_err("Missing attribute for RX"); - return -EINVAL; - } + if (!rx_attr) + return 0; rx_size = nla_get_u8(rx_attr); if (!cfg_in_range(CFG_RX_AGGREGATION_SIZE, rx_size)) { @@ -7645,10 +7643,8 @@ static int hdd_config_msdu_aggregation(struct hdd_adapter *adapter, uint8_t rx_size; QDF_STATUS status; - if (!rx_attr) { - hdd_err("Missing attribute for RX"); - return -EINVAL; - } + if (!rx_attr) + return 0; rx_size = nla_get_u8(rx_attr); if (!cfg_in_range(CFG_RX_AGGREGATION_SIZE, rx_size)) {