Skip to content

Commit db5cce1

Browse files
anderskKalle Valo
authored andcommitted
Revert "iwlwifi: assign directly to iwl_trans->cfg in QuZ detection"
This reverts commit 968dcfb. Both that commit and commit 809805a attempted to fix the same bug (dead assignments to the local variable cfg), but they did so in incompatible ways. When they were both merged, independently of each other, the combination actually caused the bug to reappear, leading to a firmware crash on boot for some cards. https://bugzilla.kernel.org/show_bug.cgi?id=205719 Signed-off-by: Anders Kaseorg <andersk@mit.edu> Acked-by: Luca Coelho <luciano.coelho@intel.com> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
1 parent 1e58252 commit db5cce1

File tree

1 file changed

+12
-12
lines changed
  • drivers/net/wireless/intel/iwlwifi/pcie

1 file changed

+12
-12
lines changed

drivers/net/wireless/intel/iwlwifi/pcie/drv.c

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1111,18 +1111,18 @@ static int iwl_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
11111111

11121112
/* same thing for QuZ... */
11131113
if (iwl_trans->hw_rev == CSR_HW_REV_TYPE_QUZ) {
1114-
if (iwl_trans->cfg == &iwl_ax101_cfg_qu_hr)
1115-
iwl_trans->cfg = &iwl_ax101_cfg_quz_hr;
1116-
else if (iwl_trans->cfg == &iwl_ax201_cfg_qu_hr)
1117-
iwl_trans->cfg = &iwl_ax201_cfg_quz_hr;
1118-
else if (iwl_trans->cfg == &iwl9461_2ac_cfg_qu_b0_jf_b0)
1119-
iwl_trans->cfg = &iwl9461_2ac_cfg_quz_a0_jf_b0_soc;
1120-
else if (iwl_trans->cfg == &iwl9462_2ac_cfg_qu_b0_jf_b0)
1121-
iwl_trans->cfg = &iwl9462_2ac_cfg_quz_a0_jf_b0_soc;
1122-
else if (iwl_trans->cfg == &iwl9560_2ac_cfg_qu_b0_jf_b0)
1123-
iwl_trans->cfg = &iwl9560_2ac_cfg_quz_a0_jf_b0_soc;
1124-
else if (iwl_trans->cfg == &iwl9560_2ac_160_cfg_qu_b0_jf_b0)
1125-
iwl_trans->cfg = &iwl9560_2ac_160_cfg_quz_a0_jf_b0_soc;
1114+
if (cfg == &iwl_ax101_cfg_qu_hr)
1115+
cfg = &iwl_ax101_cfg_quz_hr;
1116+
else if (cfg == &iwl_ax201_cfg_qu_hr)
1117+
cfg = &iwl_ax201_cfg_quz_hr;
1118+
else if (cfg == &iwl9461_2ac_cfg_qu_b0_jf_b0)
1119+
cfg = &iwl9461_2ac_cfg_quz_a0_jf_b0_soc;
1120+
else if (cfg == &iwl9462_2ac_cfg_qu_b0_jf_b0)
1121+
cfg = &iwl9462_2ac_cfg_quz_a0_jf_b0_soc;
1122+
else if (cfg == &iwl9560_2ac_cfg_qu_b0_jf_b0)
1123+
cfg = &iwl9560_2ac_cfg_quz_a0_jf_b0_soc;
1124+
else if (cfg == &iwl9560_2ac_160_cfg_qu_b0_jf_b0)
1125+
cfg = &iwl9560_2ac_160_cfg_quz_a0_jf_b0_soc;
11261126
}
11271127

11281128
#endif

0 commit comments

Comments
 (0)