瀏覽代碼

qcacld-3.0: Fix HE Tx beamforming cfg parameter values

HE Tx beamforming config parameter setting fails due to incorrect
value range. Correct the HE Tx beamforming config parameter values

Change-Id: Idb0ada45417467122bd7473cca07d15cf54fd3cc
CRs-Fixed: 2244247
Kiran Kumar Lokere 6 年之前
父節點
當前提交
a48cae1f5f
共有 3 個文件被更改,包括 7 次插入7 次删除
  1. 2 2
      core/mac/inc/wni_cfg.h
  2. 5 4
      core/mac/src/cfg/cfg_api.c
  3. 0 1
      core/mac/src/cfg/cfg_proc_msg.c

+ 2 - 2
core/mac/inc/wni_cfg.h

@@ -1459,11 +1459,11 @@ enum {
 #define WNI_CFG_HE_MU_BEAMFORMER_STAMAX 1
 #define WNI_CFG_HE_MU_BEAMFORMER_STADEF 0
 
-#define WNI_CFG_HE_BFEE_STS_LT80_STAMIN 0x3
+#define WNI_CFG_HE_BFEE_STS_LT80_STAMIN 0
 #define WNI_CFG_HE_BFEE_STS_LT80_STAMAX 0x7
 #define WNI_CFG_HE_BFEE_STS_LT80_STADEF 0
 
-#define WNI_CFG_HE_BFEE_STS_GT80_STAMIN 0x3
+#define WNI_CFG_HE_BFEE_STS_GT80_STAMIN 0
 #define WNI_CFG_HE_BFEE_STS_GT80_STAMAX 0x7
 #define WNI_CFG_HE_BFEE_STS_GT80_STADEF 0
 

+ 5 - 4
core/mac/src/cfg/cfg_api.c

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2017 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2011-2018 The Linux Foundation. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for
  * any purpose with or without fee is hereby granted, provided that the
@@ -257,9 +257,10 @@ tSirRetStatus cfg_set_int(tpAniSirGlobal pMac, uint16_t cfgId, uint32_t value)
 
 	if ((pMac->cfg.gCfgIBufMin[index] > value) ||
 			(pMac->cfg.gCfgIBufMax[index] < value)) {
-		pe_warn("Value: %d out of range: [%d,%d] cfg id: %d", value,
-			       pMac->cfg.gCfgIBufMin[index],
-			       pMac->cfg.gCfgIBufMax[index], cfgId);
+		pe_warn("Value: %d out of range: [%d,%d] cfg id: %d, %s",
+				value, pMac->cfg.gCfgIBufMin[index],
+				pMac->cfg.gCfgIBufMax[index], cfgId,
+				cfg_get_string(cfgId));
 		return eSIR_CFG_INVALID_ID;
 	} else {
 		/* Write integer value */

+ 0 - 1
core/mac/src/cfg/cfg_proc_msg.c

@@ -1178,7 +1178,6 @@ cgstatic cfg_static[CFG_PARAM_MAX_NUM] = {
 	WNI_CFG_HE_MU_PPDU_STADEF},
 	{WNI_CFG_HE_SU_BEAMFORMER,
 	CFG_CTL_VALID | CFG_CTL_RE | CFG_CTL_WE | CFG_CTL_INT,
-	CFG_CTL_VALID | CFG_CTL_RE | CFG_CTL_WE | CFG_CTL_INT,
 	WNI_CFG_HE_SU_BEAMFORMER_STAMIN, WNI_CFG_HE_SU_BEAMFORMER_STAMAX,
 	WNI_CFG_HE_SU_BEAMFORMER_STADEF},
 	{WNI_CFG_HE_SU_BEAMFORMEE,