Skip to content

Commit 24bc49d

Browse files
committed
wifi: iwlwifi: remove support of several iwl_ppag_table_cmd versions
We only need to support version 1, 5 and 7. Remove versions 2, 3, 4 and 6. Reviewed-by: Pagadala Yesu Anjaneyulu <[email protected]> Signed-off-by: Miri Korenblit <[email protected]> Link: https://patch.msgid.link/20250711183056.10d91f675505.Idd3a6da568261ee738918f290168a2ddaa87196b@changeid
1 parent 493681d commit 24bc49d

File tree

2 files changed

+10
-30
lines changed

2 files changed

+10
-30
lines changed

drivers/net/wireless/intel/iwlwifi/fw/api/power.h

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -571,8 +571,7 @@ enum iwl_ppag_flags {
571571
/**
572572
* union iwl_ppag_table_cmd - union for all versions of PPAG command
573573
* @v1: command version 1 structure.
574-
* @v2: command version from 2 to 6 are same structure as v2.
575-
* but has a different format of the flags bitmap
574+
* @v2: command version 5 structure.
576575
* @v3: command version 7 structure.
577576
* @v1.flags: values from &enum iwl_ppag_flags
578577
* @v1.gain: table of antenna gain values per chain and sub-band
@@ -593,30 +592,19 @@ union iwl_ppag_table_cmd {
593592
__le32 flags;
594593
s8 gain[IWL_NUM_CHAIN_LIMITS][IWL_NUM_SUB_BANDS_V2];
595594
s8 reserved[2];
596-
} __packed v2; /* PER_PLAT_ANTENNA_GAIN_CMD_API_S_VER_2, VER3, VER4,
597-
* VER5, VER6
598-
*/
595+
} __packed v2; /* PER_PLAT_ANTENNA_GAIN_CMD_API_S_VER_5 */
599596
struct {
600597
struct bios_value_u32 ppag_config_info;
601598
s8 gain[IWL_NUM_CHAIN_LIMITS][IWL_NUM_SUB_BANDS_V2];
602599
s8 reserved[2];
603600
} __packed v3; /* PER_PLAT_ANTENNA_GAIN_CMD_API_S_VER_7 */
604601
} __packed;
605602

606-
#define IWL_PPAG_CMD_V4_MASK (IWL_PPAG_ETSI_MASK | IWL_PPAG_CHINA_MASK)
607-
#define IWL_PPAG_CMD_V5_MASK (IWL_PPAG_CMD_V4_MASK | \
603+
#define IWL_PPAG_CMD_V1_MASK (IWL_PPAG_ETSI_MASK | IWL_PPAG_CHINA_MASK)
604+
#define IWL_PPAG_CMD_V5_MASK (IWL_PPAG_CMD_V1_MASK | \
608605
IWL_PPAG_ETSI_LPI_UHB_MASK | \
609606
IWL_PPAG_USA_LPI_UHB_MASK)
610607

611-
#define IWL_PPAG_CMD_V6_MASK (IWL_PPAG_CMD_V5_MASK | \
612-
IWL_PPAG_ETSI_VLP_UHB_MASK | \
613-
IWL_PPAG_ETSI_SP_UHB_MASK | \
614-
IWL_PPAG_USA_VLP_UHB_MASK | \
615-
IWL_PPAG_USA_SP_UHB_MASK | \
616-
IWL_PPAG_CANADA_LPI_UHB_MASK | \
617-
IWL_PPAG_CANADA_VLP_UHB_MASK | \
618-
IWL_PPAG_CANADA_SP_UHB_MASK)
619-
620608
#define MCC_TO_SAR_OFFSET_TABLE_ROW_SIZE 26
621609
#define MCC_TO_SAR_OFFSET_TABLE_COL_SIZE 13
622610

drivers/net/wireless/intel/iwlwifi/fw/regulatory.c

Lines changed: 6 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -344,18 +344,18 @@ int iwl_fill_ppag_table(struct iwl_fw_runtime *fwrt,
344344
num_sub_bands = IWL_NUM_SUB_BANDS_V1;
345345
gain = cmd->v1.gain[0];
346346
*cmd_size = sizeof(cmd->v1);
347-
cmd->v1.flags = cpu_to_le32(fwrt->ppag_flags);
347+
cmd->v1.flags = cpu_to_le32(fwrt->ppag_flags & IWL_PPAG_CMD_V1_MASK);
348348
if (fwrt->ppag_bios_rev >= 1) {
349349
/* in this case FW supports revision 0 */
350350
IWL_DEBUG_RADIO(fwrt,
351351
"PPAG table rev is %d, send truncated table\n",
352352
fwrt->ppag_bios_rev);
353353
}
354-
} else if (cmd_ver >= 2 && cmd_ver <= 6) {
354+
} else if (cmd_ver == 5) {
355355
num_sub_bands = IWL_NUM_SUB_BANDS_V2;
356356
gain = cmd->v2.gain[0];
357357
*cmd_size = sizeof(cmd->v2);
358-
cmd->v2.flags = cpu_to_le32(fwrt->ppag_flags);
358+
cmd->v2.flags = cpu_to_le32(fwrt->ppag_flags & IWL_PPAG_CMD_V5_MASK);
359359
if (fwrt->ppag_bios_rev == 0) {
360360
/* in this case FW supports revisions 1,2 or 3 */
361361
IWL_DEBUG_RADIO(fwrt,
@@ -378,17 +378,9 @@ int iwl_fill_ppag_table(struct iwl_fw_runtime *fwrt,
378378
"PPAG MODE bits were read from bios: %d\n",
379379
fwrt->ppag_flags);
380380

381-
if (cmd_ver == 6)
382-
cmd->v1.flags &= cpu_to_le32(IWL_PPAG_CMD_V6_MASK);
383-
else if (cmd_ver == 5)
384-
cmd->v1.flags &= cpu_to_le32(IWL_PPAG_CMD_V5_MASK);
385-
else if (cmd_ver < 5)
386-
cmd->v1.flags &= cpu_to_le32(IWL_PPAG_CMD_V4_MASK);
387-
388-
if ((cmd_ver == 1 &&
389-
!fw_has_capa(&fwrt->fw->ucode_capa,
390-
IWL_UCODE_TLV_CAPA_PPAG_CHINA_BIOS_SUPPORT)) ||
391-
(cmd_ver == 2 && fwrt->ppag_bios_rev >= 2)) {
381+
if (cmd_ver == 1 &&
382+
!fw_has_capa(&fwrt->fw->ucode_capa,
383+
IWL_UCODE_TLV_CAPA_PPAG_CHINA_BIOS_SUPPORT)) {
392384
cmd->v1.flags &= cpu_to_le32(IWL_PPAG_ETSI_MASK);
393385
IWL_DEBUG_RADIO(fwrt, "masking ppag China bit\n");
394386
} else {

0 commit comments

Comments
 (0)