wifi: iwlwifi: fix the rf step and flavor bits range
authorMukesh Sisodiya <mukesh.sisodiya@intel.com>
Wed, 4 Oct 2023 09:36:33 +0000 (12:36 +0300)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 23 Oct 2023 10:21:48 +0000 (12:21 +0200)
The macros used to get the RF step and flavour are using
wrong bit range. Update the bit range for both macros.

Signed-off-by: Mukesh Sisodiya <mukesh.sisodiya@intel.com>
Signed-off-by: Gregory Greenman <gregory.greenman@intel.com>
Link: https://lore.kernel.org/r/20231004123422.cfd0aaaa5eb3.Ie2dd6c3a3062647f19cb5e888c46f0fdca103484@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
drivers/net/wireless/intel/iwlwifi/iwl-prph.h

index c1a94b01ce1441faafa2260080339685114cfaca..da035dbfbdb089923b735723619e129b1db43127 100644 (file)
@@ -423,14 +423,14 @@ enum {
  * reserved: bits 12-18
  * slave_exist: bit 19
  * dash: bits 20-23
- * step: bits 24-26
- * flavor: bits 27-31
+ * step: bits 24-27
+ * flavor: bits 28-31
  */
 #define REG_CRF_ID_TYPE(val)           (((val) & 0x00000FFF) >> 0)
 #define REG_CRF_ID_SLAVE(val)          (((val) & 0x00080000) >> 19)
 #define REG_CRF_ID_DASH(val)           (((val) & 0x00F00000) >> 20)
-#define REG_CRF_ID_STEP(val)           (((val) & 0x07000000) >> 24)
-#define REG_CRF_ID_FLAVOR(val)         (((val) & 0xF8000000) >> 27)
+#define REG_CRF_ID_STEP(val)           (((val) & 0x0F000000) >> 24)
+#define REG_CRF_ID_FLAVOR(val)         (((val) & 0xF0000000) >> 28)
 
 #define UREG_CHICK             (0xA05C00)
 #define UREG_CHICK_MSI_ENABLE  BIT(24)