staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
Subsystems (3)
cover staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
This patchset performs code cleanup in qlge driver as per linux coding style and may be applied in any sequence.
Nandha Kumar Singaram (2):
staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
drivers/staging/qlge/qlge.h | 388 +++++++++++++++---------------- drivers/staging/qlge/qlge_main.c | 8 +- drivers/staging/qlge/qlge_mpi.c | 2 +- 3 files changed, 199 insertions(+), 199 deletions(-)
Signature · 1 line
2.25.1
PATCH 1/2 staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
Adhere to linux coding style. Replace the occurrences of (1<<x) by BIT(x) in qlge.h
--- drivers/staging/qlge/qlge.h | 388 ++++++++++++++++++------------------ 1 file changed, 194 insertions(+), 194 deletions(-)
drivers/staging/qlge/qlge.h +194 -194
@@ -96,16 +96,16 @@ enum {MPI_TEST_FUNC_RST_STS =0x100a ,MPI_TEST_FUNC_RST_FRC =0x00000003 ,MPI_TEST_NIC_FUNC_MASK =0x00000007 , -MPI_TEST_NIC1_FUNCTION_ENABLE = (1 <<0 ), +MPI_TEST_NIC1_FUNCTION_ENABLE =BIT (0 ),MPI_TEST_NIC1_FUNCTION_MASK =0x0000000e ,MPI_TEST_NIC1_FUNC_SHIFT =1 , -MPI_TEST_NIC2_FUNCTION_ENABLE = (1 <<4 ), +MPI_TEST_NIC2_FUNCTION_ENABLE =BIT (4 ),MPI_TEST_NIC2_FUNCTION_MASK =0x000000e0 ,MPI_TEST_NIC2_FUNC_SHIFT =5 , -MPI_TEST_FC1_FUNCTION_ENABLE = (1 <<8 ), +MPI_TEST_FC1_FUNCTION_ENABLE =BIT (8 ),MPI_TEST_FC1_FUNCTION_MASK =0x00000e00 ,MPI_TEST_FC1_FUNCTION_SHIFT =9 , -MPI_TEST_FC2_FUNCTION_ENABLE = (1 <<12 ), +MPI_TEST_FC2_FUNCTION_ENABLE =BIT (12 ),MPI_TEST_FC2_FUNCTION_MASK =0x0000e000 , MPI_TEST_FC2_FUNCTION_SHIFT = 13,
@@ -122,10 +122,10 @@ enum {MAILBOX_COUNT =16 ,MAILBOX_TIMEOUT =5 , -PROC_ADDR_RDY = (1 <<31 ), -PROC_ADDR_R = (1 <<30 ), -PROC_ADDR_ERR = (1 <<29 ), -PROC_ADDR_DA = (1 <<28 ), +PROC_ADDR_RDY =BIT (31 ), +PROC_ADDR_R =BIT (30 ), +PROC_ADDR_ERR =BIT (29 ), +PROC_ADDR_DA =BIT (28 ),PROC_ADDR_FUNC0_MBI =0x00001180 ,PROC_ADDR_FUNC0_MBO = (PROC_ADDR_FUNC0_MBI +MAILBOX_COUNT ), PROC_ADDR_FUNC0_CTL = 0x000011a1,
@@ -142,33 +142,33 @@ enum {* System Register (SYS) bit definitions. */ enum { -SYS_EFE = (1 <<0 ), -SYS_FAE = (1 <<1 ), -SYS_MDC = (1 <<2 ), -SYS_DST = (1 <<3 ), -SYS_DWC = (1 <<4 ), -SYS_EVW = (1 <<5 ), +SYS_EFE =BIT (0 ), +SYS_FAE =BIT (1 ), +SYS_MDC =BIT (2 ), +SYS_DST =BIT (3 ), +SYS_DWC =BIT (4 ), +SYS_EVW =BIT (5 ),SYS_OMP_DLY_MASK =0x3f000000 ,/* * There are no values defined as of edit #15. */ -SYS_ODI = (1 <<14 ), +SYS_ODI =BIT (14 ), };/* * Reset/Failover Register (RST_FO) bit definitions. */ enum { -RST_FO_TFO = (1 <<0 ), +RST_FO_TFO =BIT (0 ),RST_FO_RR_MASK =0x00060000 ,RST_FO_RR_CQ_CAM =0x00000000 ,RST_FO_RR_DROP =0x00000002 ,RST_FO_RR_DQ =0x00000004 ,RST_FO_RR_RCV_FUNC_CQ =0x00000006 , -RST_FO_FRB = (1 <<12 ), -RST_FO_MOP = (1 <<13 ), -RST_FO_REG = (1 <<14 ), -RST_FO_FR = (1 <<15 ), +RST_FO_FRB =BIT (12 ), +RST_FO_MOP =BIT (13 ), +RST_FO_REG =BIT (14 ), +RST_FO_FR =BIT (15 ), };/*
@@ -185,19 +185,19 @@ enum {FSC_DBL_MAX_PLD =0x00000008 ,FSC_DBL_MAX_BRST =0x00000010 ,FSC_DBL_128_BYTES =0x00000018 , -FSC_EC = (1 <<5 ), +FSC_EC =BIT (5 ),FSC_EPC_MASK =0x00c00000 , -FSC_EPC_INBOUND = (1 <<6 ), -FSC_EPC_OUTBOUND = (1 <<7 ), +FSC_EPC_INBOUND =BIT (6 ), +FSC_EPC_OUTBOUND =BIT (7 ),FSC_VM_PAGESIZE_MASK =0x07000000 ,FSC_VM_PAGE_2K =0x00000100 ,FSC_VM_PAGE_4K =0x00000200 ,FSC_VM_PAGE_8K =0x00000300 ,FSC_VM_PAGE_64K =0x00000600 , -FSC_SH = (1 <<11 ), -FSC_DSB = (1 <<12 ), -FSC_STE = (1 <<13 ), - FSC_FE = (1 << 15), +FSC_SH =BIT (11 ), +FSC_DSB =BIT (12 ), +FSC_STE =BIT (13 ), + FSC_FE = BIT(15), };/*
@@ -208,9 +208,9 @@ enum {/* * There are no valued defined as of edit #15. */ -CSR_RR = (1 <<8 ), -CSR_HRI = (1 <<9 ), -CSR_RP = (1 <<10 ), +CSR_RR =BIT (8 ), +CSR_HRI =BIT (9 ), +CSR_RP =BIT (10 ),CSR_CMD_PARM_SHIFT =22 ,CSR_CMD_NOP =0x00000000 , CSR_CMD_SET_RST = 0x10000000,
@@ -229,13 +229,13 @@ enum {* Configuration Register (CFG) bit definitions. */ enum { -CFG_LRQ = (1 <<0 ), -CFG_DRQ = (1 <<1 ), -CFG_LR = (1 <<2 ), -CFG_DR = (1 <<3 ), -CFG_LE = (1 <<5 ), -CFG_LCQ = (1 <<6 ), -CFG_DCQ = (1 <<7 ), +CFG_LRQ =BIT (0 ), +CFG_DRQ =BIT (1 ), +CFG_LR =BIT (2 ), +CFG_DR =BIT (3 ), +CFG_LE =BIT (5 ), +CFG_LCQ =BIT (6 ), +CFG_DCQ =BIT (7 ),CFG_Q_SHIFT =8 ,CFG_Q_MASK =0x7f000000 , };
@@ -244,19 +244,19 @@ enum {* Status Register (STS) bit definitions. */ enum { -STS_FE = (1 <<0 ), -STS_PI = (1 <<1 ), -STS_PL0 = (1 <<2 ), -STS_PL1 = (1 <<3 ), -STS_PI0 = (1 <<4 ), -STS_PI1 = (1 <<5 ), +STS_FE =BIT (0 ), +STS_PI =BIT (1 ), +STS_PL0 =BIT (2 ), +STS_PL1 =BIT (3 ), +STS_PI0 =BIT (4 ), +STS_PI1 =BIT (5 ),STS_FUNC_ID_MASK =0x000000c0 ,STS_FUNC_ID_SHIFT =6 , -STS_F0E = (1 <<8 ), -STS_F1E = (1 <<9 ), -STS_F2E = (1 <<10 ), -STS_F3E = (1 <<11 ), -STS_NFE = (1 <<12 ), +STS_F0E =BIT (8 ), +STS_F1E =BIT (9 ), +STS_F2E =BIT (10 ), +STS_F3E =BIT (11 ), +STS_NFE =BIT (12 ), };/*
@@ -268,24 +268,24 @@ enum {INTR_EN_TYPE_ENABLE =0x00000100 ,INTR_EN_TYPE_DISABLE =0x00000200 ,INTR_EN_TYPE_READ =0x00000300 , -INTR_EN_IHD = (1 <<13 ), +INTR_EN_IHD =BIT (13 ),INTR_EN_IHD_MASK = (INTR_EN_IHD <<16 ), -INTR_EN_EI = (1 <<14 ), - INTR_EN_EN = (1 << 15), +INTR_EN_EI =BIT (14 ), + INTR_EN_EN = BIT(15), };/* * Interrupt Mask Register (INTR_MASK) bit definitions. */ enum { -INTR_MASK_PI = (1 <<0 ), -INTR_MASK_HL0 = (1 <<1 ), -INTR_MASK_LH0 = (1 <<2 ), -INTR_MASK_HL1 = (1 <<3 ), -INTR_MASK_LH1 = (1 <<4 ), -INTR_MASK_SE = (1 <<5 ), -INTR_MASK_LSC = (1 <<6 ), -INTR_MASK_MC = (1 <<7 ), +INTR_MASK_PI =BIT (0 ), +INTR_MASK_HL0 =BIT (1 ), +INTR_MASK_LH0 =BIT (2 ), +INTR_MASK_HL1 =BIT (3 ), +INTR_MASK_LH1 =BIT (4 ), +INTR_MASK_SE =BIT (5 ), +INTR_MASK_LSC =BIT (6 ), +INTR_MASK_MC =BIT (7 ),INTR_MASK_LINK_IRQS =INTR_MASK_LSC |INTR_MASK_SE |INTR_MASK_MC , };
@@ -305,38 +305,38 @@ enum {* Force ECC Error Register (FRC_ECC_ERR) bit definitions. */ enum { -FRC_ECC_ERR_VW = (1 <<12 ), -FRC_ECC_ERR_VB = (1 <<13 ), -FRC_ECC_ERR_NI = (1 <<14 ), -FRC_ECC_ERR_NO = (1 <<15 ), +FRC_ECC_ERR_VW =BIT (12 ), +FRC_ECC_ERR_VB =BIT (13 ), +FRC_ECC_ERR_NI =BIT (14 ), +FRC_ECC_ERR_NO =BIT (15 ),FRC_ECC_PFE_SHIFT =16 , -FRC_ECC_ERR_DO = (1 <<18 ), -FRC_ECC_P14 = (1 <<19 ), +FRC_ECC_ERR_DO =BIT (18 ), +FRC_ECC_P14 =BIT (19 ), };/* * Error Status Register (ERR_STS) bit definitions. */ enum { -ERR_STS_NOF = (1 <<0 ), -ERR_STS_NIF = (1 <<1 ), -ERR_STS_DRP = (1 <<2 ), -ERR_STS_XGP = (1 <<3 ), -ERR_STS_FOU = (1 <<4 ), -ERR_STS_FOC = (1 <<5 ), -ERR_STS_FOF = (1 <<6 ), -ERR_STS_FIU = (1 <<7 ), -ERR_STS_FIC = (1 <<8 ), -ERR_STS_FIF = (1 <<9 ), -ERR_STS_MOF = (1 <<10 ), -ERR_STS_TA = (1 <<11 ), -ERR_STS_MA = (1 <<12 ), -ERR_STS_MPE = (1 <<13 ), -ERR_STS_SCE = (1 <<14 ), -ERR_STS_STE = (1 <<15 ), -ERR_STS_FOW = (1 <<16 ), -ERR_STS_UE = (1 <<17 ), -ERR_STS_MCH = (1 <<26 ), +ERR_STS_NOF =BIT (0 ), +ERR_STS_NIF =BIT (1 ), +ERR_STS_DRP =BIT (2 ), +ERR_STS_XGP =BIT (3 ), +ERR_STS_FOU =BIT (4 ), +ERR_STS_FOC =BIT (5 ), +ERR_STS_FOF =BIT (6 ), +ERR_STS_FIU =BIT (7 ), +ERR_STS_FIC =BIT (8 ), +ERR_STS_FIF =BIT (9 ), +ERR_STS_MOF =BIT (10 ), +ERR_STS_TA =BIT (11 ), +ERR_STS_MA =BIT (12 ), +ERR_STS_MPE =BIT (13 ), +ERR_STS_SCE =BIT (14 ), +ERR_STS_STE =BIT (15 ), +ERR_STS_FOW =BIT (16 ), +ERR_STS_UE =BIT (17 ), +ERR_STS_MCH =BIT (26 ),ERR_STS_LOC_SHIFT =27 , };
@@ -344,8 +344,8 @@ enum {* RAM Debug Address Register (RAM_DBG_ADDR) bit definitions. */ enum { -RAM_DBG_ADDR_FW = (1 <<30 ), -RAM_DBG_ADDR_FR = (1 <<31 ), +RAM_DBG_ADDR_FW =BIT (30 ), +RAM_DBG_ADDR_FR =BIT (31 ), };/*
@@ -381,33 +381,33 @@ enum {* 10G MAC Address Register (XGMAC_ADDR) bit definitions. */ enum { -XGMAC_ADDR_RDY = (1 <<31 ), -XGMAC_ADDR_R = (1 <<30 ), -XGMAC_ADDR_XME = (1 <<29 ), +XGMAC_ADDR_RDY =BIT (31 ), +XGMAC_ADDR_R =BIT (30 ), +XGMAC_ADDR_XME =BIT (29 ),/* XGMAC control registers */ PAUSE_SRC_LO =0x00000100 ,PAUSE_SRC_HI =0x00000104 ,GLOBAL_CFG =0x00000108 , -GLOBAL_CFG_RESET = (1 <<0 ), -GLOBAL_CFG_JUMBO = (1 <<6 ), -GLOBAL_CFG_TX_STAT_EN = (1 <<10 ), -GLOBAL_CFG_RX_STAT_EN = (1 <<11 ), +GLOBAL_CFG_RESET =BIT (0 ), +GLOBAL_CFG_JUMBO =BIT (6 ), +GLOBAL_CFG_TX_STAT_EN =BIT (10 ), +GLOBAL_CFG_RX_STAT_EN =BIT (11 ),TX_CFG =0x0000010c , -TX_CFG_RESET = (1 <<0 ), -TX_CFG_EN = (1 <<1 ), -TX_CFG_PREAM = (1 <<2 ), +TX_CFG_RESET =BIT (0 ), +TX_CFG_EN =BIT (1 ), +TX_CFG_PREAM =BIT (2 ),RX_CFG =0x00000110 , -RX_CFG_RESET = (1 <<0 ), -RX_CFG_EN = (1 <<1 ), -RX_CFG_PREAM = (1 <<2 ), +RX_CFG_RESET =BIT (0 ), +RX_CFG_EN =BIT (1 ), +RX_CFG_PREAM =BIT (2 ),FLOW_CTL =0x0000011c ,PAUSE_OPCODE =0x00000120 ,PAUSE_TIMER =0x00000124 ,PAUSE_FRM_DEST_LO =0x00000128 ,PAUSE_FRM_DEST_HI =0x0000012c ,MAC_TX_PARAMS =0x00000134 , -MAC_TX_PARAMS_JUMBO = (1 <<31 ), +MAC_TX_PARAMS_JUMBO =BIT (31 ),MAC_TX_PARAMS_SIZE_SHIFT =16 ,MAC_RX_PARAMS =0x00000138 ,MAC_SYS_INT =0x00000144 ,
@@ -444,8 +444,8 @@ enum {RX_OVERFLOW_MASK =0x000002b8 ,TX_OVERFLOW_MASK =0x000002bc ,STAT_CNT_CTL =0x000002c0 , -STAT_CNT_CTL_CLEAR_TX = (1 <<0 ), -STAT_CNT_CTL_CLEAR_RX = (1 <<1 ), +STAT_CNT_CTL_CLEAR_TX =BIT (0 ), +STAT_CNT_CTL_CLEAR_RX =BIT (1 ),AUX_RX_HALF_FULL_DET =0x000002d0 ,AUX_TX_HALF_FULL_DET =0x000002d4 , AUX_RX_OVERFLOW_DET = 0x000002d8,
@@ -499,9 +499,9 @@ enum {*/ enum {ETS_QUEUE_SHIFT =29 , -ETS_REF = (1 <<26 ), -ETS_RS = (1 <<27 ), -ETS_P = (1 <<28 ), +ETS_REF =BIT (26 ), +ETS_RS =BIT (27 ), +ETS_P =BIT (28 ),ETS_FC_COS_SHIFT =23 , };
@@ -509,9 +509,9 @@ enum {* Flash Address Register (FLASH_ADDR) bit definitions. */ enum { -FLASH_ADDR_RDY = (1 <<31 ), -FLASH_ADDR_R = (1 <<30 ), -FLASH_ADDR_ERR = (1 <<29 ), +FLASH_ADDR_RDY =BIT (31 ), +FLASH_ADDR_R =BIT (30 ), +FLASH_ADDR_ERR =BIT (29 ), };/*
@@ -523,7 +523,7 @@ enum {CQ_STOP_TYPE_START =0x00000100 ,CQ_STOP_TYPE_STOP =0x00000200 ,CQ_STOP_TYPE_READ =0x00000300 , - CQ_STOP_EN = (1 << 15), + CQ_STOP_EN = BIT(15), };/*
@@ -544,11 +544,11 @@ enum {MAC_ADDR_TYPE_MGMT_V4 =0x00070000 ,MAC_ADDR_TYPE_MGMT_V6 =0x00080000 ,MAC_ADDR_TYPE_MGMT_TU_DP =0x00090000 , -MAC_ADDR_ADR = (1 <<25 ), -MAC_ADDR_RS = (1 <<26 ), -MAC_ADDR_E = (1 <<27 ), -MAC_ADDR_MR = (1 <<30 ), -MAC_ADDR_MW = (1 <<31 ), +MAC_ADDR_ADR =BIT (25 ), +MAC_ADDR_RS =BIT (26 ), +MAC_ADDR_E =BIT (27 ), +MAC_ADDR_MR =BIT (30 ), +MAC_ADDR_MW =BIT (31 ),MAX_MULTICAST_ENTRIES =32 ,/* Entry count and words per entry
@@ -580,34 +580,34 @@ enum {* MAC Protocol Address Index Register (SPLT_HDR) bit definitions. */ enum { -SPLT_HDR_EP = (1 <<31 ), +SPLT_HDR_EP =BIT (31 ), };/* * FCoE Receive Configuration Register (FC_RCV_CFG) bit definitions. */ enum { -FC_RCV_CFG_ECT = (1 <<15 ), -FC_RCV_CFG_DFH = (1 <<20 ), -FC_RCV_CFG_DVF = (1 <<21 ), -FC_RCV_CFG_RCE = (1 <<27 ), -FC_RCV_CFG_RFE = (1 <<28 ), -FC_RCV_CFG_TEE = (1 <<29 ), -FC_RCV_CFG_TCE = (1 <<30 ), -FC_RCV_CFG_TFE = (1 <<31 ), +FC_RCV_CFG_ECT =BIT (15 ), +FC_RCV_CFG_DFH =BIT (20 ), +FC_RCV_CFG_DVF =BIT (21 ), +FC_RCV_CFG_RCE =BIT (27 ), +FC_RCV_CFG_RFE =BIT (28 ), +FC_RCV_CFG_TEE =BIT (29 ), +FC_RCV_CFG_TCE =BIT (30 ), +FC_RCV_CFG_TFE =BIT (31 ), };/* * NIC Receive Configuration Register (NIC_RCV_CFG) bit definitions. */ enum { -NIC_RCV_CFG_PPE = (1 <<0 ), +NIC_RCV_CFG_PPE =BIT (0 ),NIC_RCV_CFG_VLAN_MASK =0x00060000 ,NIC_RCV_CFG_VLAN_ALL =0x00000000 ,NIC_RCV_CFG_VLAN_MATCH_ONLY =0x00000002 ,NIC_RCV_CFG_VLAN_MATCH_AND_NON =0x00000004 ,NIC_RCV_CFG_VLAN_NONE_AND_NON =0x00000006 , -NIC_RCV_CFG_RV = (1 <<3 ), +NIC_RCV_CFG_RV =BIT (3 ),NIC_RCV_CFG_DFQ_MASK = (0x7f000000 ),NIC_RCV_CFG_DFQ_SHIFT =8 ,NIC_RCV_CFG_DFQ =0 ,/* HARDCODE default queue to 0. */
@@ -617,20 +617,20 @@ enum {* Mgmt Receive Configuration Register (MGMT_RCV_CFG) bit definitions. */ enum { -MGMT_RCV_CFG_ARP = (1 <<0 ), -MGMT_RCV_CFG_DHC = (1 <<1 ), -MGMT_RCV_CFG_DHS = (1 <<2 ), -MGMT_RCV_CFG_NP = (1 <<3 ), -MGMT_RCV_CFG_I6N = (1 <<4 ), -MGMT_RCV_CFG_I6R = (1 <<5 ), -MGMT_RCV_CFG_DH6 = (1 <<6 ), -MGMT_RCV_CFG_UD1 = (1 <<7 ), -MGMT_RCV_CFG_UD0 = (1 <<8 ), -MGMT_RCV_CFG_BCT = (1 <<9 ), -MGMT_RCV_CFG_MCT = (1 <<10 ), -MGMT_RCV_CFG_DM = (1 <<11 ), -MGMT_RCV_CFG_RM = (1 <<12 ), -MGMT_RCV_CFG_STL = (1 <<13 ), +MGMT_RCV_CFG_ARP =BIT (0 ), +MGMT_RCV_CFG_DHC =BIT (1 ), +MGMT_RCV_CFG_DHS =BIT (2 ), +MGMT_RCV_CFG_NP =BIT (3 ), +MGMT_RCV_CFG_I6N =BIT (4 ), +MGMT_RCV_CFG_I6R =BIT (5 ), +MGMT_RCV_CFG_DH6 =BIT (6 ), +MGMT_RCV_CFG_UD1 =BIT (7 ), +MGMT_RCV_CFG_UD0 =BIT (8 ), +MGMT_RCV_CFG_BCT =BIT (9 ), +MGMT_RCV_CFG_MCT =BIT (10 ), +MGMT_RCV_CFG_DM =BIT (11 ), +MGMT_RCV_CFG_RM =BIT (12 ), +MGMT_RCV_CFG_STL =BIT (13 ),MGMT_RCV_CFG_VLAN_MASK =0xc0000000 ,MGMT_RCV_CFG_VLAN_ALL =0x00000000 ,MGMT_RCV_CFG_VLAN_MATCH_ONLY =0x00004000 ,
@@ -655,44 +655,44 @@ enum {RT_IDX_DST_COS_Q =0x00200000 ,RT_IDX_DST_DFLT_Q =0x00300000 ,RT_IDX_DST_DEST_Q =0x00400000 , -RT_IDX_RS = (1 <<26 ), -RT_IDX_E = (1 <<27 ), -RT_IDX_MR = (1 <<30 ), -RT_IDX_MW = (1 <<31 ), +RT_IDX_RS =BIT (26 ), +RT_IDX_E =BIT (27 ), +RT_IDX_MR =BIT (30 ), +RT_IDX_MW =BIT (31 ),/* Nic Queue format - type 2 bits */ -RT_IDX_BCAST = (1 <<0 ), -RT_IDX_MCAST = (1 <<1 ), -RT_IDX_MCAST_MATCH = (1 <<2 ), -RT_IDX_MCAST_REG_MATCH = (1 <<3 ), -RT_IDX_MCAST_HASH_MATCH = (1 <<4 ), -RT_IDX_FC_MACH = (1 <<5 ), -RT_IDX_ETH_FCOE = (1 <<6 ), -RT_IDX_CAM_HIT = (1 <<7 ), -RT_IDX_CAM_BIT0 = (1 <<8 ), -RT_IDX_CAM_BIT1 = (1 <<9 ), -RT_IDX_VLAN_TAG = (1 <<10 ), -RT_IDX_VLAN_MATCH = (1 <<11 ), -RT_IDX_VLAN_FILTER = (1 <<12 ), -RT_IDX_ETH_SKIP1 = (1 <<13 ), -RT_IDX_ETH_SKIP2 = (1 <<14 ), -RT_IDX_BCAST_MCAST_MATCH = (1 <<15 ), -RT_IDX_802_3 = (1 <<16 ), -RT_IDX_LLDP = (1 <<17 ), -RT_IDX_UNUSED018 = (1 <<18 ), -RT_IDX_UNUSED019 = (1 <<19 ), -RT_IDX_UNUSED20 = (1 <<20 ), -RT_IDX_UNUSED21 = (1 <<21 ), -RT_IDX_ERR = (1 <<22 ), -RT_IDX_VALID = (1 <<23 ), -RT_IDX_TU_CSUM_ERR = (1 <<24 ), -RT_IDX_IP_CSUM_ERR = (1 <<25 ), -RT_IDX_MAC_ERR = (1 <<26 ), -RT_IDX_RSS_TCP6 = (1 <<27 ), -RT_IDX_RSS_TCP4 = (1 <<28 ), -RT_IDX_RSS_IPV6 = (1 <<29 ), -RT_IDX_RSS_IPV4 = (1 <<30 ), -RT_IDX_RSS_MATCH = (1 <<31 ), +RT_IDX_BCAST =BIT (0 ), +RT_IDX_MCAST =BIT (1 ), +RT_IDX_MCAST_MATCH =BIT (2 ), +RT_IDX_MCAST_REG_MATCH =BIT (3 ), +RT_IDX_MCAST_HASH_MATCH =BIT (4 ), +RT_IDX_FC_MACH =BIT (5 ), +RT_IDX_ETH_FCOE =BIT (6 ), +RT_IDX_CAM_HIT =BIT (7 ), +RT_IDX_CAM_BIT0 =BIT (8 ), +RT_IDX_CAM_BIT1 =BIT (9 ), +RT_IDX_VLAN_TAG =BIT (10 ), +RT_IDX_VLAN_MATCH =BIT (11 ), +RT_IDX_VLAN_FILTER =BIT (12 ), +RT_IDX_ETH_SKIP1 =BIT (13 ), +RT_IDX_ETH_SKIP2 =BIT (14 ), +RT_IDX_BCAST_MCAST_MATCH =BIT (15 ), +RT_IDX_802_3 =BIT (16 ), +RT_IDX_LLDP =BIT (17 ), +RT_IDX_UNUSED018 =BIT (18 ), +RT_IDX_UNUSED019 =BIT (19 ), +RT_IDX_UNUSED20 =BIT (20 ), +RT_IDX_UNUSED21 =BIT (21 ), +RT_IDX_ERR =BIT (22 ), +RT_IDX_VALID =BIT (23 ), +RT_IDX_TU_CSUM_ERR =BIT (24 ), +RT_IDX_IP_CSUM_ERR =BIT (25 ), +RT_IDX_MAC_ERR =BIT (26 ), +RT_IDX_RSS_TCP6 =BIT (27 ), +RT_IDX_RSS_TCP4 =BIT (28 ), +RT_IDX_RSS_IPV6 =BIT (29 ), +RT_IDX_RSS_IPV4 =BIT (30 ), +RT_IDX_RSS_MATCH =BIT (31 ),/* Hierarchy for the NIC Queue Mask */ RT_IDX_ALL_ERR_SLOT = 0,
@@ -721,8 +721,8 @@ enum {* Serdes Address Register (XG_SERDES_ADDR) bit definitions. */ enum { -XG_SERDES_ADDR_RDY = (1 <<31 ), -XG_SERDES_ADDR_R = (1 <<30 ), +XG_SERDES_ADDR_RDY =BIT (31 ), +XG_SERDES_ADDR_R =BIT (30 ),XG_SERDES_ADDR_STS =0x00001E06 ,XG_SERDES_ADDR_XFI1_PWR_UP =0x00000005 ,
@@ -752,9 +752,9 @@ enum {* NIC Probe Mux Address Register (PRB_MX_ADDR) bit definitions. */ enum { -PRB_MX_ADDR_ARE = (1 <<16 ), -PRB_MX_ADDR_UP = (1 <<15 ), -PRB_MX_ADDR_SWP = (1 <<14 ), +PRB_MX_ADDR_ARE =BIT (16 ), +PRB_MX_ADDR_UP =BIT (15 ), +PRB_MX_ADDR_SWP =BIT (14 ),/* Module select values. */ PRB_MX_ADDR_MAX_MODS =21 ,
@@ -890,8 +890,8 @@ enum {CAM_OUT_ROUTE_FC =0 ,CAM_OUT_ROUTE_NIC =1 ,CAM_OUT_FUNC_SHIFT =2 , -CAM_OUT_RV = (1 <<4 ), -CAM_OUT_SH = (1 <<15 ), +CAM_OUT_RV =BIT (4 ), +CAM_OUT_SH =BIT (15 ), CAM_OUT_CQ_ID_SHIFT = 5, };
@@ -936,14 +936,14 @@ enum {MB_CMD_IDC_ACK =0x00000101 ,/* Inter-Driver Communication */ MB_CMD_SET_WOL_MODE =0x00000110 ,/* Wake On Lan */ MB_WOL_DISABLE =0 , -MB_WOL_MAGIC_PKT = (1 <<1 ), -MB_WOL_FLTR = (1 <<2 ), -MB_WOL_UCAST = (1 <<3 ), -MB_WOL_MCAST = (1 <<4 ), -MB_WOL_BCAST = (1 <<5 ), -MB_WOL_LINK_UP = (1 <<6 ), -MB_WOL_LINK_DOWN = (1 <<7 ), -MB_WOL_MODE_ON = (1 <<16 ),/* Wake on Lan Mode on */ +MB_WOL_MAGIC_PKT =BIT (1 ), +MB_WOL_FLTR =BIT (2 ), +MB_WOL_UCAST =BIT (3 ), +MB_WOL_MCAST =BIT (4 ), +MB_WOL_BCAST =BIT (5 ), +MB_WOL_LINK_UP =BIT (6 ), +MB_WOL_LINK_DOWN =BIT (7 ), +MB_WOL_MODE_ON =BIT (16 ),/* Wake on Lan Mode on */ MB_CMD_SET_WOL_FLTR =0x00000111 ,/* Wake On Lan Filter */ MB_CMD_CLEAR_WOL_FLTR =0x00000112 ,/* Wake On Lan Filter */ MB_CMD_SET_WOL_MAGIC = 0x00000113, /* Wake On Lan Magic Packet */
@@ -957,11 +957,11 @@ enum {QL_LED_BLINK =0x03e803e8 ,MB_CMD_GET_LED_CFG =0x00000126 ,/* Get LED Configuration Register */ MB_CMD_SET_MGMNT_TFK_CTL =0x00000160 ,/* Set Mgmnt Traffic Control */ -MB_SET_MPI_TFK_STOP = (1 <<0 ), -MB_SET_MPI_TFK_RESUME = (1 <<1 ), +MB_SET_MPI_TFK_STOP =BIT (0 ), +MB_SET_MPI_TFK_RESUME =BIT (1 ),MB_CMD_GET_MGMNT_TFK_CTL =0x00000161 ,/* Get Mgmnt Traffic Control */ -MB_GET_MPI_TFK_STOPPED = (1 <<0 ), -MB_GET_MPI_TFK_FIFO_EMPTY = (1 <<1 ), +MB_GET_MPI_TFK_STOPPED =BIT (0 ), +MB_GET_MPI_TFK_FIFO_EMPTY =BIT (1 ),/* Sub-commands for IDC request. * This describes the reason for the * IDC request.
Signature · 1 line
2.25.1
PATCH 2/2 staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
In reply to Nandha Kumar SingaramAdhere to linux coding style. Replace the occurrences of (1<<x) by BIT(x)
--- drivers/staging/qlge/qlge_main.c | 8 ++++---- drivers/staging/qlge/qlge_mpi.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-)
drivers/staging/qlge/qlge_main.c +4 -4
@@ -2191,7 +2191,7 @@ static int qlge_napi_poll_msix(struct napi_struct *napi, int budget)/* If this TX completion ring belongs to this vector and * it's not empty then service it. */ -if ((ctx->irq_mask & (1 << trx_ring->cq_id)) && +if ((ctx->irq_mask &BIT (trx_ring->cq_id)) && (qlge_read_sh_reg (trx_ring->prod_idx_sh_reg) != trx_ring->cnsmr_idx)) { netif_printk(qdev, intr, KERN_DEBUG, qdev->ndev,
@@ -3222,13 +3222,13 @@ static void qlge_set_irq_mask(struct qlge_adapter *qdev, struct intr_context *ct/* Add the RSS ring serviced by this vector * to the mask. */ - ctx->irq_mask = (1 << qdev->rx_ring[vect].cq_id); + ctx->irq_mask =BIT (qdev->rx_ring[vect].cq_id);/* Add the TX ring(s) serviced by this vector * to the mask. */ for (j =0 ; j < tx_rings_per_vector; j++) { ctx->irq_mask |= - (1 << qdev->rx_ring[qdev->rss_ring_count + +BIT (qdev->rx_ring[qdev->rss_ring_count + (vect * tx_rings_per_vector) + j].cq_id); } }else {
@@ -3236,7 +3236,7 @@ static void qlge_set_irq_mask(struct qlge_adapter *qdev, struct intr_context *ct* ID into the mask. */ for (j =0 ; j < qdev->rx_ring_count; j++) - ctx->irq_mask |= (1 << qdev->rx_ring[j].cq_id); + ctx->irq_mask |=BIT (qdev->rx_ring[j].cq_id); } }
drivers/staging/qlge/qlge_mpi.c +1 -1
@@ -113,7 +113,7 @@ int qlge_own_firmware(struct qlge_adapter *qdev)* core dump and firmware reset after an error. */ temp =qlge_read32 (qdev,STS ); -if (!(temp & (1 << (8 + qdev->alt_func)))) +if (!(temp &BIT ((8 + qdev->alt_func))))return 1 ;return 0 ;
Signature · 1 line
2.25.1
Greg Kroah-Hartman
maintainerQuoted 6 lines · Nandha Kumar Singaram
This patchset performs code cleanup in qlge driver as per linux coding style and may be applied in any sequence.
Nandha Kumar Singaram (2):
staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
You have two different patches doing different things yet they have the same identical subject lines. That's not ok, sorry, please make unique subject lines as obviously you are doing different things.
thanks,
greg k-h
Quoted 8 lines · Greg Kroah-Hartman
Quoted 6 lines · Nandha Kumar Singaram
This patchset performs code cleanup in qlge driver as per linux coding style and may be applied in any sequence.
Nandha Kumar Singaram (2):
staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
staging: qlge: Replace the occurrences of (1<<x) by BIT(x)
Quoted 7 lines
You have two different patches doing different things yet they have the same identical subject lines. That's not ok, sorry, please make unique subject lines as obviously you are doing different things.
thanks,
greg k-h
Sorry greg, thanks for your feedback. Will update the patch
Regards,
Nandha Kumar Singaram