Skip to content

Commit 8196b5f

Browse files
Ivan Veceraanguy11
authored andcommitted
i40e: Refactor I40E_MDIO_CLAUSE* macros
The macros I40E_MDIO_CLAUSE22* and I40E_MDIO_CLAUSE45* are using I40E_MASK together with the same values I40E_GLGEN_MSCA_STCODE_SHIFT and I40E_GLGEN_MSCA_OPCODE_SHIFT to define masks. Introduce I40E_GLGEN_MSCA_OPCODE_MASK and I40E_GLGEN_MSCA_STCODE_MASK for both shifts in i40e_register.h and use them to refactor the macros mentioned above. Signed-off-by: Ivan Vecera <[email protected]> Reviewed-by: Przemek Kitszel <[email protected]> Reviewed-by: Jesse Brandeburg <[email protected]> Reviewed-by: Aleksandr Loktionov <[email protected]> Tested-by: Pucha Himasekhar Reddy <[email protected]> (A Contingent worker at Intel) Signed-off-by: Tony Nguyen <[email protected]>
1 parent 9d84f73 commit 8196b5f

File tree

2 files changed

+10
-15
lines changed

2 files changed

+10
-15
lines changed

drivers/net/ethernet/intel/i40e/i40e_register.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,9 @@
205205
#define I40E_GLGEN_MSCA_DEVADD_SHIFT 16
206206
#define I40E_GLGEN_MSCA_PHYADD_SHIFT 21
207207
#define I40E_GLGEN_MSCA_OPCODE_SHIFT 26
208+
#define I40E_GLGEN_MSCA_OPCODE_MASK(_i) I40E_MASK(_i, I40E_GLGEN_MSCA_OPCODE_SHIFT)
208209
#define I40E_GLGEN_MSCA_STCODE_SHIFT 28
210+
#define I40E_GLGEN_MSCA_STCODE_MASK I40E_MASK(0x1, I40E_GLGEN_MSCA_STCODE_SHIFT)
209211
#define I40E_GLGEN_MSCA_MDICMD_SHIFT 30
210212
#define I40E_GLGEN_MSCA_MDICMD_MASK I40E_MASK(0x1, I40E_GLGEN_MSCA_MDICMD_SHIFT)
211213
#define I40E_GLGEN_MSCA_MDIINPROGEN_SHIFT 31

drivers/net/ethernet/intel/i40e/i40e_type.h

Lines changed: 8 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -67,21 +67,14 @@ enum i40e_debug_mask {
6767
I40E_DEBUG_ALL = 0xFFFFFFFF
6868
};
6969

70-
#define I40E_MDIO_CLAUSE22_STCODE_MASK I40E_MASK(1, \
71-
I40E_GLGEN_MSCA_STCODE_SHIFT)
72-
#define I40E_MDIO_CLAUSE22_OPCODE_WRITE_MASK I40E_MASK(1, \
73-
I40E_GLGEN_MSCA_OPCODE_SHIFT)
74-
#define I40E_MDIO_CLAUSE22_OPCODE_READ_MASK I40E_MASK(2, \
75-
I40E_GLGEN_MSCA_OPCODE_SHIFT)
76-
77-
#define I40E_MDIO_CLAUSE45_STCODE_MASK I40E_MASK(0, \
78-
I40E_GLGEN_MSCA_STCODE_SHIFT)
79-
#define I40E_MDIO_CLAUSE45_OPCODE_ADDRESS_MASK I40E_MASK(0, \
80-
I40E_GLGEN_MSCA_OPCODE_SHIFT)
81-
#define I40E_MDIO_CLAUSE45_OPCODE_WRITE_MASK I40E_MASK(1, \
82-
I40E_GLGEN_MSCA_OPCODE_SHIFT)
83-
#define I40E_MDIO_CLAUSE45_OPCODE_READ_MASK I40E_MASK(3, \
84-
I40E_GLGEN_MSCA_OPCODE_SHIFT)
70+
#define I40E_MDIO_CLAUSE22_STCODE_MASK I40E_GLGEN_MSCA_STCODE_MASK
71+
#define I40E_MDIO_CLAUSE22_OPCODE_WRITE_MASK I40E_GLGEN_MSCA_OPCODE_MASK(1)
72+
#define I40E_MDIO_CLAUSE22_OPCODE_READ_MASK I40E_GLGEN_MSCA_OPCODE_MASK(2)
73+
74+
#define I40E_MDIO_CLAUSE45_STCODE_MASK I40E_GLGEN_MSCA_STCODE_MASK
75+
#define I40E_MDIO_CLAUSE45_OPCODE_ADDRESS_MASK I40E_GLGEN_MSCA_OPCODE_MASK(0)
76+
#define I40E_MDIO_CLAUSE45_OPCODE_WRITE_MASK I40E_GLGEN_MSCA_OPCODE_MASK(1)
77+
#define I40E_MDIO_CLAUSE45_OPCODE_READ_MASK I40E_GLGEN_MSCA_OPCODE_MASK(3)
8578

8679
#define I40E_PHY_COM_REG_PAGE 0x1E
8780
#define I40E_PHY_LED_LINK_MODE_MASK 0xF0

0 commit comments

Comments
 (0)