if (size > 15)
                size = 15;
 
-       nvm->word_size = 1 << size;
+       nvm->word_size = BIT(size);
        nvm->opcode_bits = 8;
        nvm->delay_usec = 1;
 
                                    16 : 8;
                break;
        }
-       if (nvm->word_size == (1 << 15))
+       if (nvm->word_size == BIT(15))
                nvm->page_size = 128;
 
        nvm->type = e1000_nvm_eeprom_spi;
        nvm->ops.write = igb_write_nvm_spi;
        nvm->ops.validate = igb_validate_nvm_checksum;
        nvm->ops.update = igb_update_nvm_checksum;
-       if (nvm->word_size < (1 << 15))
+       if (nvm->word_size < BIT(15))
                nvm->ops.read = igb_read_nvm_eerd;
        else
                nvm->ops.read = igb_read_nvm_spi;
                /* The PF can spoof - it has to in order to
                 * support emulation mode NICs
                 */
-               reg_val ^= (1 << pf | 1 << (pf + MAX_NUM_VFS));
+               reg_val ^= (BIT(pf) | BIT(pf + MAX_NUM_VFS));
        } else {
                reg_val &= ~(E1000_DTXSWC_MAC_SPOOF_MASK |
                             E1000_DTXSWC_VLAN_SPOOF_MASK);
 
 #define E1000_DCA_CTRL_DCA_MODE_CB2     0x02 /* DCA Mode CB2 */
 
 #define E1000_DCA_RXCTRL_CPUID_MASK 0x0000001F /* Rx CPUID Mask */
-#define E1000_DCA_RXCTRL_DESC_DCA_EN (1 << 5) /* DCA Rx Desc enable */
-#define E1000_DCA_RXCTRL_HEAD_DCA_EN (1 << 6) /* DCA Rx Desc header enable */
-#define E1000_DCA_RXCTRL_DATA_DCA_EN (1 << 7) /* DCA Rx Desc payload enable */
-#define E1000_DCA_RXCTRL_DESC_RRO_EN (1 << 9) /* DCA Rx rd Desc Relax Order */
+#define E1000_DCA_RXCTRL_DESC_DCA_EN BIT(5) /* DCA Rx Desc enable */
+#define E1000_DCA_RXCTRL_HEAD_DCA_EN BIT(6) /* DCA Rx Desc header enable */
+#define E1000_DCA_RXCTRL_DATA_DCA_EN BIT(7) /* DCA Rx Desc payload enable */
+#define E1000_DCA_RXCTRL_DESC_RRO_EN BIT(9) /* DCA Rx rd Desc Relax Order */
 
 #define E1000_DCA_TXCTRL_CPUID_MASK 0x0000001F /* Tx CPUID Mask */
-#define E1000_DCA_TXCTRL_DESC_DCA_EN (1 << 5) /* DCA Tx Desc enable */
-#define E1000_DCA_TXCTRL_DESC_RRO_EN (1 << 9) /* Tx rd Desc Relax Order */
-#define E1000_DCA_TXCTRL_TX_WB_RO_EN (1 << 11) /* Tx Desc writeback RO bit */
-#define E1000_DCA_TXCTRL_DATA_RRO_EN (1 << 13) /* Tx rd data Relax Order */
+#define E1000_DCA_TXCTRL_DESC_DCA_EN BIT(5) /* DCA Tx Desc enable */
+#define E1000_DCA_TXCTRL_DESC_RRO_EN BIT(9) /* Tx rd Desc Relax Order */
+#define E1000_DCA_TXCTRL_TX_WB_RO_EN BIT(11) /* Tx Desc writeback RO bit */
+#define E1000_DCA_TXCTRL_DATA_RRO_EN BIT(13) /* Tx rd data Relax Order */
 
 /* Additional DCA related definitions, note change in position of CPUID */
 #define E1000_DCA_TXCTRL_CPUID_MASK_82576 0xFF000000 /* Tx CPUID Mask */
 #define E1000_DCA_RXCTRL_CPUID_SHIFT 24 /* Rx CPUID now in the last byte */
 
 /* ETQF register bit definitions */
-#define E1000_ETQF_FILTER_ENABLE   (1 << 26)
-#define E1000_ETQF_1588            (1 << 30)
+#define E1000_ETQF_FILTER_ENABLE   BIT(26)
+#define E1000_ETQF_1588            BIT(30)
 
 /* FTQF register bit definitions */
 #define E1000_FTQF_VF_BP               0x00008000
 #define E1000_DTXSWC_VLAN_SPOOF_MASK  0x0000FF00 /* Per VF VLAN spoof control */
 #define E1000_DTXSWC_LLE_MASK         0x00FF0000 /* Per VF Local LB enables */
 #define E1000_DTXSWC_VLAN_SPOOF_SHIFT 8
-#define E1000_DTXSWC_VMDQ_LOOPBACK_EN (1 << 31)  /* global VF LB enable */
+#define E1000_DTXSWC_VMDQ_LOOPBACK_EN BIT(31)  /* global VF LB enable */
 
 /* Easy defines for setting default pool, would normally be left a zero */
 #define E1000_VT_CTL_DEFAULT_POOL_SHIFT 7
 #define E1000_VT_CTL_DEFAULT_POOL_MASK  (0x7 << E1000_VT_CTL_DEFAULT_POOL_SHIFT)
 
 /* Other useful VMD_CTL register defines */
-#define E1000_VT_CTL_IGNORE_MAC         (1 << 28)
-#define E1000_VT_CTL_DISABLE_DEF_POOL   (1 << 29)
-#define E1000_VT_CTL_VM_REPL_EN         (1 << 30)
+#define E1000_VT_CTL_IGNORE_MAC         BIT(28)
+#define E1000_VT_CTL_DISABLE_DEF_POOL   BIT(29)
+#define E1000_VT_CTL_VM_REPL_EN         BIT(30)
 
 /* Per VM Offload register setup */
 #define E1000_VMOLR_RLPML_MASK 0x00003FFF /* Long Packet Maximum Length mask */
 #define E1000_DTXCTL_MDP_EN     0x0020
 #define E1000_DTXCTL_SPOOF_INT  0x0040
 
-#define E1000_EEPROM_PCS_AUTONEG_DISABLE_BIT   (1 << 14)
+#define E1000_EEPROM_PCS_AUTONEG_DISABLE_BIT   BIT(14)
 
 #define ALL_QUEUES   0xFFFF
 
 
 
 /* Time Sync Interrupt Cause/Mask Register Bits */
 
-#define TSINTR_SYS_WRAP  (1 << 0) /* SYSTIM Wrap around. */
-#define TSINTR_TXTS      (1 << 1) /* Transmit Timestamp. */
-#define TSINTR_RXTS      (1 << 2) /* Receive Timestamp. */
-#define TSINTR_TT0       (1 << 3) /* Target Time 0 Trigger. */
-#define TSINTR_TT1       (1 << 4) /* Target Time 1 Trigger. */
-#define TSINTR_AUTT0     (1 << 5) /* Auxiliary Timestamp 0 Taken. */
-#define TSINTR_AUTT1     (1 << 6) /* Auxiliary Timestamp 1 Taken. */
-#define TSINTR_TADJ      (1 << 7) /* Time Adjust Done. */
+#define TSINTR_SYS_WRAP  BIT(0) /* SYSTIM Wrap around. */
+#define TSINTR_TXTS      BIT(1) /* Transmit Timestamp. */
+#define TSINTR_RXTS      BIT(2) /* Receive Timestamp. */
+#define TSINTR_TT0       BIT(3) /* Target Time 0 Trigger. */
+#define TSINTR_TT1       BIT(4) /* Target Time 1 Trigger. */
+#define TSINTR_AUTT0     BIT(5) /* Auxiliary Timestamp 0 Taken. */
+#define TSINTR_AUTT1     BIT(6) /* Auxiliary Timestamp 1 Taken. */
+#define TSINTR_TADJ      BIT(7) /* Time Adjust Done. */
 
 #define TSYNC_INTERRUPTS TSINTR_TXTS
 #define E1000_TSICR_TXTS TSINTR_TXTS
 
 /* TSAUXC Configuration Bits */
-#define TSAUXC_EN_TT0    (1 << 0)  /* Enable target time 0. */
-#define TSAUXC_EN_TT1    (1 << 1)  /* Enable target time 1. */
-#define TSAUXC_EN_CLK0   (1 << 2)  /* Enable Configurable Frequency Clock 0. */
-#define TSAUXC_SAMP_AUT0 (1 << 3)  /* Latch SYSTIML/H into AUXSTMPL/0. */
-#define TSAUXC_ST0       (1 << 4)  /* Start Clock 0 Toggle on Target Time 0. */
-#define TSAUXC_EN_CLK1   (1 << 5)  /* Enable Configurable Frequency Clock 1. */
-#define TSAUXC_SAMP_AUT1 (1 << 6)  /* Latch SYSTIML/H into AUXSTMPL/1. */
-#define TSAUXC_ST1       (1 << 7)  /* Start Clock 1 Toggle on Target Time 1. */
-#define TSAUXC_EN_TS0    (1 << 8)  /* Enable hardware timestamp 0. */
-#define TSAUXC_AUTT0     (1 << 9)  /* Auxiliary Timestamp Taken. */
-#define TSAUXC_EN_TS1    (1 << 10) /* Enable hardware timestamp 0. */
-#define TSAUXC_AUTT1     (1 << 11) /* Auxiliary Timestamp Taken. */
-#define TSAUXC_PLSG      (1 << 17) /* Generate a pulse. */
-#define TSAUXC_DISABLE   (1 << 31) /* Disable SYSTIM Count Operation. */
+#define TSAUXC_EN_TT0    BIT(0)  /* Enable target time 0. */
+#define TSAUXC_EN_TT1    BIT(1)  /* Enable target time 1. */
+#define TSAUXC_EN_CLK0   BIT(2)  /* Enable Configurable Frequency Clock 0. */
+#define TSAUXC_SAMP_AUT0 BIT(3)  /* Latch SYSTIML/H into AUXSTMPL/0. */
+#define TSAUXC_ST0       BIT(4)  /* Start Clock 0 Toggle on Target Time 0. */
+#define TSAUXC_EN_CLK1   BIT(5)  /* Enable Configurable Frequency Clock 1. */
+#define TSAUXC_SAMP_AUT1 BIT(6)  /* Latch SYSTIML/H into AUXSTMPL/1. */
+#define TSAUXC_ST1       BIT(7)  /* Start Clock 1 Toggle on Target Time 1. */
+#define TSAUXC_EN_TS0    BIT(8)  /* Enable hardware timestamp 0. */
+#define TSAUXC_AUTT0     BIT(9)  /* Auxiliary Timestamp Taken. */
+#define TSAUXC_EN_TS1    BIT(10) /* Enable hardware timestamp 0. */
+#define TSAUXC_AUTT1     BIT(11) /* Auxiliary Timestamp Taken. */
+#define TSAUXC_PLSG      BIT(17) /* Generate a pulse. */
+#define TSAUXC_DISABLE   BIT(31) /* Disable SYSTIM Count Operation. */
 
 /* SDP Configuration Bits */
-#define AUX0_SEL_SDP0    (0 << 0)  /* Assign SDP0 to auxiliary time stamp 0. */
-#define AUX0_SEL_SDP1    (1 << 0)  /* Assign SDP1 to auxiliary time stamp 0. */
-#define AUX0_SEL_SDP2    (2 << 0)  /* Assign SDP2 to auxiliary time stamp 0. */
-#define AUX0_SEL_SDP3    (3 << 0)  /* Assign SDP3 to auxiliary time stamp 0. */
-#define AUX0_TS_SDP_EN   (1 << 2)  /* Enable auxiliary time stamp trigger 0. */
-#define AUX1_SEL_SDP0    (0 << 3)  /* Assign SDP0 to auxiliary time stamp 1. */
-#define AUX1_SEL_SDP1    (1 << 3)  /* Assign SDP1 to auxiliary time stamp 1. */
-#define AUX1_SEL_SDP2    (2 << 3)  /* Assign SDP2 to auxiliary time stamp 1. */
-#define AUX1_SEL_SDP3    (3 << 3)  /* Assign SDP3 to auxiliary time stamp 1. */
-#define AUX1_TS_SDP_EN   (1 << 5)  /* Enable auxiliary time stamp trigger 1. */
-#define TS_SDP0_SEL_TT0  (0 << 6)  /* Target time 0 is output on SDP0. */
-#define TS_SDP0_SEL_TT1  (1 << 6)  /* Target time 1 is output on SDP0. */
-#define TS_SDP0_SEL_FC0  (2 << 6)  /* Freq clock  0 is output on SDP0. */
-#define TS_SDP0_SEL_FC1  (3 << 6)  /* Freq clock  1 is output on SDP0. */
-#define TS_SDP0_EN       (1 << 8)  /* SDP0 is assigned to Tsync. */
-#define TS_SDP1_SEL_TT0  (0 << 9)  /* Target time 0 is output on SDP1. */
-#define TS_SDP1_SEL_TT1  (1 << 9)  /* Target time 1 is output on SDP1. */
-#define TS_SDP1_SEL_FC0  (2 << 9)  /* Freq clock  0 is output on SDP1. */
-#define TS_SDP1_SEL_FC1  (3 << 9)  /* Freq clock  1 is output on SDP1. */
-#define TS_SDP1_EN       (1 << 11) /* SDP1 is assigned to Tsync. */
-#define TS_SDP2_SEL_TT0  (0 << 12) /* Target time 0 is output on SDP2. */
-#define TS_SDP2_SEL_TT1  (1 << 12) /* Target time 1 is output on SDP2. */
-#define TS_SDP2_SEL_FC0  (2 << 12) /* Freq clock  0 is output on SDP2. */
-#define TS_SDP2_SEL_FC1  (3 << 12) /* Freq clock  1 is output on SDP2. */
-#define TS_SDP2_EN       (1 << 14) /* SDP2 is assigned to Tsync. */
-#define TS_SDP3_SEL_TT0  (0 << 15) /* Target time 0 is output on SDP3. */
-#define TS_SDP3_SEL_TT1  (1 << 15) /* Target time 1 is output on SDP3. */
-#define TS_SDP3_SEL_FC0  (2 << 15) /* Freq clock  0 is output on SDP3. */
-#define TS_SDP3_SEL_FC1  (3 << 15) /* Freq clock  1 is output on SDP3. */
-#define TS_SDP3_EN       (1 << 17) /* SDP3 is assigned to Tsync. */
+#define AUX0_SEL_SDP0    (0u << 0)  /* Assign SDP0 to auxiliary time stamp 0. */
+#define AUX0_SEL_SDP1    (1u << 0)  /* Assign SDP1 to auxiliary time stamp 0. */
+#define AUX0_SEL_SDP2    (2u << 0)  /* Assign SDP2 to auxiliary time stamp 0. */
+#define AUX0_SEL_SDP3    (3u << 0)  /* Assign SDP3 to auxiliary time stamp 0. */
+#define AUX0_TS_SDP_EN   (1u << 2)  /* Enable auxiliary time stamp trigger 0. */
+#define AUX1_SEL_SDP0    (0u << 3)  /* Assign SDP0 to auxiliary time stamp 1. */
+#define AUX1_SEL_SDP1    (1u << 3)  /* Assign SDP1 to auxiliary time stamp 1. */
+#define AUX1_SEL_SDP2    (2u << 3)  /* Assign SDP2 to auxiliary time stamp 1. */
+#define AUX1_SEL_SDP3    (3u << 3)  /* Assign SDP3 to auxiliary time stamp 1. */
+#define AUX1_TS_SDP_EN   (1u << 5)  /* Enable auxiliary time stamp trigger 1. */
+#define TS_SDP0_SEL_TT0  (0u << 6)  /* Target time 0 is output on SDP0. */
+#define TS_SDP0_SEL_TT1  (1u << 6)  /* Target time 1 is output on SDP0. */
+#define TS_SDP0_SEL_FC0  (2u << 6)  /* Freq clock  0 is output on SDP0. */
+#define TS_SDP0_SEL_FC1  (3u << 6)  /* Freq clock  1 is output on SDP0. */
+#define TS_SDP0_EN       (1u << 8)  /* SDP0 is assigned to Tsync. */
+#define TS_SDP1_SEL_TT0  (0u << 9)  /* Target time 0 is output on SDP1. */
+#define TS_SDP1_SEL_TT1  (1u << 9)  /* Target time 1 is output on SDP1. */
+#define TS_SDP1_SEL_FC0  (2u << 9)  /* Freq clock  0 is output on SDP1. */
+#define TS_SDP1_SEL_FC1  (3u << 9)  /* Freq clock  1 is output on SDP1. */
+#define TS_SDP1_EN       (1u << 11) /* SDP1 is assigned to Tsync. */
+#define TS_SDP2_SEL_TT0  (0u << 12) /* Target time 0 is output on SDP2. */
+#define TS_SDP2_SEL_TT1  (1u << 12) /* Target time 1 is output on SDP2. */
+#define TS_SDP2_SEL_FC0  (2u << 12) /* Freq clock  0 is output on SDP2. */
+#define TS_SDP2_SEL_FC1  (3u << 12) /* Freq clock  1 is output on SDP2. */
+#define TS_SDP2_EN       (1u << 14) /* SDP2 is assigned to Tsync. */
+#define TS_SDP3_SEL_TT0  (0u << 15) /* Target time 0 is output on SDP3. */
+#define TS_SDP3_SEL_TT1  (1u << 15) /* Target time 1 is output on SDP3. */
+#define TS_SDP3_SEL_FC0  (2u << 15) /* Freq clock  0 is output on SDP3. */
+#define TS_SDP3_SEL_FC1  (3u << 15) /* Freq clock  1 is output on SDP3. */
+#define TS_SDP3_EN       (1u << 17) /* SDP3 is assigned to Tsync. */
 
 #define E1000_MDICNFG_EXT_MDIO    0x80000000      /* MDI ext/int destination */
 #define E1000_MDICNFG_COM_MDIO    0x40000000      /* MDI shared w/ lan 0 */
 #define E1000_M88E1543_FIBER_CTRL      0x0
 #define E1000_EEE_ADV_DEV_I354         7
 #define E1000_EEE_ADV_ADDR_I354                60
-#define E1000_EEE_ADV_100_SUPPORTED    (1 << 1)   /* 100BaseTx EEE Supported */
-#define E1000_EEE_ADV_1000_SUPPORTED   (1 << 2)   /* 1000BaseT EEE Supported */
+#define E1000_EEE_ADV_100_SUPPORTED    BIT(1)   /* 100BaseTx EEE Supported */
+#define E1000_EEE_ADV_1000_SUPPORTED   BIT(2)   /* 1000BaseT EEE Supported */
 #define E1000_PCS_STATUS_DEV_I354      3
 #define E1000_PCS_STATUS_ADDR_I354     1
 #define E1000_PCS_STATUS_TX_LPI_IND    0x0200     /* Tx in LPI state */
 
         *    bits[4-0]:  which bit in the register
         */
        regidx = vlan / 32;
-       vfta_delta = 1 << (vlan % 32);
+       vfta_delta = BIT(vlan % 32);
        vfta = adapter->shadow_vfta[regidx];
 
        /* vfta_delta represents the difference between the current value
        bits = rd32(E1000_VLVF(vlvf_index));
 
        /* set the pool bit */
-       bits |= 1 << (E1000_VLVF_POOLSEL_SHIFT + vind);
+       bits |= BIT(E1000_VLVF_POOLSEL_SHIFT + vind);
        if (vlan_on)
                goto vlvf_update;
 
        /* clear the pool bit */
-       bits ^= 1 << (E1000_VLVF_POOLSEL_SHIFT + vind);
+       bits ^= BIT(E1000_VLVF_POOLSEL_SHIFT + vind);
 
        if (!(bits & E1000_VLVF_POOLSEL_MASK)) {
                /* Clear VFTA first, then disable VLVF.  Otherwise
 
        mta = array_rd32(E1000_MTA, hash_reg);
 
-       mta |= (1 << hash_bit);
+       mta |= BIT(hash_bit);
 
        array_wr32(E1000_MTA, hash_reg, mta);
        wrfl();
                hash_reg = (hash_value >> 5) & (hw->mac.mta_reg_count - 1);
                hash_bit = hash_value & 0x1F;
 
-               hw->mac.mta_shadow[hash_reg] |= (1 << hash_bit);
+               hw->mac.mta_shadow[hash_reg] |= BIT(hash_bit);
                mc_addr_list += (ETH_ALEN);
        }
 
 
        u32 vflre = rd32(E1000_VFLRE);
        s32 ret_val = -E1000_ERR_MBX;
 
-       if (vflre & (1 << vf_number)) {
+       if (vflre & BIT(vf_number)) {
                ret_val = 0;
-               wr32(E1000_VFLRE, (1 << vf_number));
+               wr32(E1000_VFLRE, BIT(vf_number));
                hw->mbx.stats.rsts++;
        }
 
 
        u32 eecd = rd32(E1000_EECD);
        u32 mask;
 
-       mask = 0x01 << (count - 1);
+       mask = 1u << (count - 1);
        if (nvm->type == e1000_nvm_eeprom_spi)
                eecd |= E1000_EECD_DO;
 
 
 
 #define I82580_ADDR_REG                   16
 #define I82580_CFG_REG                    22
-#define I82580_CFG_ASSERT_CRS_ON_TX       (1 << 15)
-#define I82580_CFG_ENABLE_DOWNSHIFT       (3 << 10) /* auto downshift 100/10 */
+#define I82580_CFG_ASSERT_CRS_ON_TX       BIT(15)
+#define I82580_CFG_ENABLE_DOWNSHIFT       (3u << 10) /* auto downshift 100/10 */
 #define I82580_CTRL_REG                   23
-#define I82580_CTRL_DOWNSHIFT_MASK        (7 << 10)
+#define I82580_CTRL_DOWNSHIFT_MASK        (7u << 10)
 
 /* 82580 specific PHY registers */
 #define I82580_PHY_CTRL_2            18
 
  * maintain a power of two alignment we have to limit ourselves to 32K.
  */
 #define IGB_MAX_TXD_PWR        15
-#define IGB_MAX_DATA_PER_TXD   (1 << IGB_MAX_TXD_PWR)
+#define IGB_MAX_DATA_PER_TXD   (1u << IGB_MAX_TXD_PWR)
 
 /* Tx Descriptors needed, worst case */
 #define TXD_USE_COUNT(S) DIV_ROUND_UP((S), IGB_MAX_DATA_PER_TXD)
        u16 eee_advert;
 };
 
-#define IGB_FLAG_HAS_MSI               (1 << 0)
-#define IGB_FLAG_DCA_ENABLED           (1 << 1)
-#define IGB_FLAG_QUAD_PORT_A           (1 << 2)
-#define IGB_FLAG_QUEUE_PAIRS           (1 << 3)
-#define IGB_FLAG_DMAC                  (1 << 4)
-#define IGB_FLAG_PTP                   (1 << 5)
-#define IGB_FLAG_RSS_FIELD_IPV4_UDP    (1 << 6)
-#define IGB_FLAG_RSS_FIELD_IPV6_UDP    (1 << 7)
-#define IGB_FLAG_WOL_SUPPORTED         (1 << 8)
-#define IGB_FLAG_NEED_LINK_UPDATE      (1 << 9)
-#define IGB_FLAG_MEDIA_RESET           (1 << 10)
-#define IGB_FLAG_MAS_CAPABLE           (1 << 11)
-#define IGB_FLAG_MAS_ENABLE            (1 << 12)
-#define IGB_FLAG_HAS_MSIX              (1 << 13)
-#define IGB_FLAG_EEE                   (1 << 14)
+#define IGB_FLAG_HAS_MSI               BIT(0)
+#define IGB_FLAG_DCA_ENABLED           BIT(1)
+#define IGB_FLAG_QUAD_PORT_A           BIT(2)
+#define IGB_FLAG_QUEUE_PAIRS           BIT(3)
+#define IGB_FLAG_DMAC                  BIT(4)
+#define IGB_FLAG_PTP                   BIT(5)
+#define IGB_FLAG_RSS_FIELD_IPV4_UDP    BIT(6)
+#define IGB_FLAG_RSS_FIELD_IPV6_UDP    BIT(7)
+#define IGB_FLAG_WOL_SUPPORTED         BIT(8)
+#define IGB_FLAG_NEED_LINK_UPDATE      BIT(9)
+#define IGB_FLAG_MEDIA_RESET           BIT(10)
+#define IGB_FLAG_MAS_CAPABLE           BIT(11)
+#define IGB_FLAG_MAS_ENABLE            BIT(12)
+#define IGB_FLAG_HAS_MSIX              BIT(13)
+#define IGB_FLAG_EEE                   BIT(14)
 #define IGB_FLAG_VLAN_PROMISC          BIT(15)
 
 /* Media Auto Sense */
 
 
        memset(p, 0, IGB_REGS_LEN * sizeof(u32));
 
-       regs->version = (1 << 24) | (hw->revision_id << 16) | hw->device_id;
+       regs->version = (1u << 24) | (hw->revision_id << 16) | hw->device_id;
 
        /* General Registers */
        regs_buff[0] = rd32(E1000_CTRL);
        /* Test each interrupt */
        for (; i < 31; i++) {
                /* Interrupt to test */
-               mask = 1 << i;
+               mask = BIT(i);
 
                if (!(mask & ics_mask))
                        continue;
                        SOF_TIMESTAMPING_RAW_HARDWARE;
 
                info->tx_types =
-                       (1 << HWTSTAMP_TX_OFF) |
-                       (1 << HWTSTAMP_TX_ON);
+                       BIT(HWTSTAMP_TX_OFF) |
+                       BIT(HWTSTAMP_TX_ON);
 
-               info->rx_filters = 1 << HWTSTAMP_FILTER_NONE;
+               info->rx_filters = BIT(HWTSTAMP_FILTER_NONE);
 
                /* 82576 does not support timestamping all packets. */
                if (adapter->hw.mac.type >= e1000_82580)
-                       info->rx_filters |= 1 << HWTSTAMP_FILTER_ALL;
+                       info->rx_filters |= BIT(HWTSTAMP_FILTER_ALL);
                else
                        info->rx_filters |=
-                               (1 << HWTSTAMP_FILTER_PTP_V1_L4_SYNC) |
-                               (1 << HWTSTAMP_FILTER_PTP_V1_L4_DELAY_REQ) |
-                               (1 << HWTSTAMP_FILTER_PTP_V2_EVENT);
+                               BIT(HWTSTAMP_FILTER_PTP_V1_L4_SYNC) |
+                               BIT(HWTSTAMP_FILTER_PTP_V1_L4_DELAY_REQ) |
+                               BIT(HWTSTAMP_FILTER_PTP_V2_EVENT);
 
                return 0;
        default:
 
                        igb_write_ivar(hw, msix_vector,
                                       tx_queue & 0x7,
                                       ((tx_queue & 0x8) << 1) + 8);
-               q_vector->eims_value = 1 << msix_vector;
+               q_vector->eims_value = BIT(msix_vector);
                break;
        case e1000_82580:
        case e1000_i350:
                        igb_write_ivar(hw, msix_vector,
                                       tx_queue >> 1,
                                       ((tx_queue & 0x1) << 4) + 8);
-               q_vector->eims_value = 1 << msix_vector;
+               q_vector->eims_value = BIT(msix_vector);
                break;
        default:
                BUG();
                     E1000_GPIE_NSICR);
 
                /* enable msix_other interrupt */
-               adapter->eims_other = 1 << vector;
+               adapter->eims_other = BIT(vector);
                tmp = (vector++ | E1000_IVAR_VALID) << 8;
 
                wr32(E1000_IVAR_MISC, tmp);
        for (i = E1000_VLVF_ARRAY_SIZE; --i;) {
                u32 vlvf = rd32(E1000_VLVF(i));
 
-               vlvf |= 1 << pf_id;
+               vlvf |= BIT(pf_id);
                wr32(E1000_VLVF(i), vlvf);
        }
 
        /* guarantee that we don't scrub out management VLAN */
        vid = adapter->mng_vlan_id;
        if (vid >= vid_start && vid < vid_end)
-               vfta[(vid - vid_start) / 32] |= 1 << (vid % 32);
+               vfta[(vid - vid_start) / 32] |= BIT(vid % 32);
 
        if (!adapter->vfs_allocated_count)
                goto set_vfta;
 
                if (vlvf & E1000_VLVF_VLANID_ENABLE) {
                        /* record VLAN ID in VFTA */
-                       vfta[(vid - vid_start) / 32] |= 1 << (vid % 32);
+                       vfta[(vid - vid_start) / 32] |= BIT(vid % 32);
 
                        /* if PF is part of this then continue */
                        if (test_bit(vid, adapter->active_vlans))
                }
 
                /* remove PF from the pool */
-               bits = ~(1 << pf_id);
+               bits = ~BIT(pf_id);
                bits &= rd32(E1000_VLVF(i));
                wr32(E1000_VLVF(i), bits);
        }
                return;
 
        for (j = 0; j < adapter->vfs_allocated_count; j++) {
-               if (adapter->wvbr & (1 << j) ||
-                   adapter->wvbr & (1 << (j + IGB_STAGGERED_QUEUE_OFFSET))) {
+               if (adapter->wvbr & BIT(j) ||
+                   adapter->wvbr & BIT(j + IGB_STAGGERED_QUEUE_OFFSET)) {
                        dev_warn(&adapter->pdev->dev,
                                "Spoof event(s) detected on VF %d\n", j);
                        adapter->wvbr &=
-                               ~((1 << j) |
-                                 (1 << (j + IGB_STAGGERED_QUEUE_OFFSET)));
+                               ~(BIT(j) |
+                                 BIT(j + IGB_STAGGERED_QUEUE_OFFSET));
                }
        }
 }
 
        /* create mask for VF and other pools */
        pool_mask = E1000_VLVF_POOLSEL_MASK;
-       vlvf_mask = 1 << (E1000_VLVF_POOLSEL_SHIFT + vf);
+       vlvf_mask = BIT(E1000_VLVF_POOLSEL_SHIFT + vf);
 
        /* drop PF from pool bits */
-       pool_mask &= ~(1 << (E1000_VLVF_POOLSEL_SHIFT +
-                            adapter->vfs_allocated_count));
+       pool_mask &= ~BIT(E1000_VLVF_POOLSEL_SHIFT +
+                            adapter->vfs_allocated_count);
 
        /* Find the vlan filter for this id */
        for (i = E1000_VLVF_ARRAY_SIZE; i--;) {
                        goto update_vlvf;
 
                vid = vlvf & E1000_VLVF_VLANID_MASK;
-               vfta_mask = 1 << (vid % 32);
+               vfta_mask = BIT(vid % 32);
 
                /* clear bit from VFTA */
                vfta = adapter->shadow_vfta[vid / 32];
         * entry other than the PF.
         */
        pf_id = adapter->vfs_allocated_count + E1000_VLVF_POOLSEL_SHIFT;
-       bits = ~(1 << pf_id) & E1000_VLVF_POOLSEL_MASK;
+       bits = ~BIT(pf_id) & E1000_VLVF_POOLSEL_MASK;
        bits &= rd32(E1000_VLVF(idx));
 
        /* Disable the filter so this falls into the default pool. */
        if (!bits) {
                if (adapter->flags & IGB_FLAG_VLAN_PROMISC)
-                       wr32(E1000_VLVF(idx), 1 << pf_id);
+                       wr32(E1000_VLVF(idx), BIT(pf_id));
                else
                        wr32(E1000_VLVF(idx), 0);
        }
 
        /* enable transmit and receive for vf */
        reg = rd32(E1000_VFTE);
-       wr32(E1000_VFTE, reg | (1 << vf));
+       wr32(E1000_VFTE, reg | BIT(vf));
        reg = rd32(E1000_VFRE);
-       wr32(E1000_VFRE, reg | (1 << vf));
+       wr32(E1000_VFRE, reg | BIT(vf));
 
        adapter->vf_data[vf].flags |= IGB_VF_FLAG_CTS;
 
                /* Calculate the rate factor values to set */
                rf_int = link_speed / tx_rate;
                rf_dec = (link_speed - (rf_int * tx_rate));
-               rf_dec = (rf_dec * (1 << E1000_RTTBCNRC_RF_INT_SHIFT)) /
+               rf_dec = (rf_dec * BIT(E1000_RTTBCNRC_RF_INT_SHIFT)) /
                         tx_rate;
 
                bcnrc_val = E1000_RTTBCNRC_RS_ENA;
        reg_offset = (hw->mac.type == e1000_82576) ? E1000_DTXSWC : E1000_TXSWC;
        reg_val = rd32(reg_offset);
        if (setting)
-               reg_val |= ((1 << vf) |
-                           (1 << (vf + E1000_DTXSWC_VLAN_SPOOF_SHIFT)));
+               reg_val |= (BIT(vf) |
+                           BIT(vf + E1000_DTXSWC_VLAN_SPOOF_SHIFT));
        else
-               reg_val &= ~((1 << vf) |
-                            (1 << (vf + E1000_DTXSWC_VLAN_SPOOF_SHIFT)));
+               reg_val &= ~(BIT(vf) |
+                            BIT(vf + E1000_DTXSWC_VLAN_SPOOF_SHIFT));
        wr32(reg_offset, reg_val);
 
        adapter->vf_data[vf].spoofchk_enabled = setting;
 
 
 #define IGB_SYSTIM_OVERFLOW_PERIOD     (HZ * 60 * 9)
 #define IGB_PTP_TX_TIMEOUT             (HZ * 15)
-#define INCPERIOD_82576                        (1 << E1000_TIMINCA_16NS_SHIFT)
-#define INCVALUE_82576_MASK            ((1 << E1000_TIMINCA_16NS_SHIFT) - 1)
-#define INCVALUE_82576                 (16 << IGB_82576_TSYNC_SHIFT)
+#define INCPERIOD_82576                        BIT(E1000_TIMINCA_16NS_SHIFT)
+#define INCVALUE_82576_MASK            GENMASK(E1000_TIMINCA_16NS_SHIFT - 1, 0)
+#define INCVALUE_82576                 (16u << IGB_82576_TSYNC_SHIFT)
 #define IGB_NBITS_82580                        40
 
 static void igb_ptp_tx_hwtstamp(struct igb_adapter *adapter);