Skip to content

Commit 69ef969

Browse files
chaithrika@ti.comdavem330
authored andcommitted
TI DaVinci EMAC: Minor macro related updates
Use BIT for macro definitions wherever possible, remove unused and redundant macros. Signed-off-by: Chaithrika U S <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent 0835acf commit 69ef969

File tree

1 file changed

+11
-15
lines changed

1 file changed

+11
-15
lines changed

drivers/net/davinci_emac.c

Lines changed: 11 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -164,16 +164,14 @@ static const char emac_version_string[] = "TI DaVinci EMAC Linux v6.1";
164164
# define EMAC_MBP_MCASTCHAN(ch) ((ch) & 0x7)
165165

166166
/* EMAC mac_control register */
167-
#define EMAC_MACCONTROL_TXPTYPE (0x200)
168-
#define EMAC_MACCONTROL_TXPACEEN (0x40)
169-
#define EMAC_MACCONTROL_MIIEN (0x20)
170-
#define EMAC_MACCONTROL_GIGABITEN (0x80)
171-
#define EMAC_MACCONTROL_GIGABITEN_SHIFT (7)
172-
#define EMAC_MACCONTROL_FULLDUPLEXEN (0x1)
167+
#define EMAC_MACCONTROL_TXPTYPE BIT(9)
168+
#define EMAC_MACCONTROL_TXPACEEN BIT(6)
169+
#define EMAC_MACCONTROL_GMIIEN BIT(5)
170+
#define EMAC_MACCONTROL_GIGABITEN BIT(7)
171+
#define EMAC_MACCONTROL_FULLDUPLEXEN BIT(0)
173172
#define EMAC_MACCONTROL_RMIISPEED_MASK BIT(15)
174173

175174
/* GIGABIT MODE related bits */
176-
#define EMAC_DM646X_MACCONTORL_GMIIEN BIT(5)
177175
#define EMAC_DM646X_MACCONTORL_GIG BIT(7)
178176
#define EMAC_DM646X_MACCONTORL_GIGFORCE BIT(17)
179177

@@ -192,18 +190,17 @@ static const char emac_version_string[] = "TI DaVinci EMAC Linux v6.1";
192190
#define EMAC_RX_BUFFER_OFFSET_MASK (0xFFFF)
193191

194192
/* MAC_IN_VECTOR (0x180) register bit fields */
195-
#define EMAC_DM644X_MAC_IN_VECTOR_HOST_INT (0x20000)
196-
#define EMAC_DM644X_MAC_IN_VECTOR_STATPEND_INT (0x10000)
197-
#define EMAC_DM644X_MAC_IN_VECTOR_RX_INT_VEC (0x0100)
198-
#define EMAC_DM644X_MAC_IN_VECTOR_TX_INT_VEC (0x01)
193+
#define EMAC_DM644X_MAC_IN_VECTOR_HOST_INT BIT(17)
194+
#define EMAC_DM644X_MAC_IN_VECTOR_STATPEND_INT BIT(16)
195+
#define EMAC_DM644X_MAC_IN_VECTOR_RX_INT_VEC BIT(8)
196+
#define EMAC_DM644X_MAC_IN_VECTOR_TX_INT_VEC BIT(0)
199197

200198
/** NOTE:: For DM646x the IN_VECTOR has changed */
201199
#define EMAC_DM646X_MAC_IN_VECTOR_RX_INT_VEC BIT(EMAC_DEF_RX_CH)
202200
#define EMAC_DM646X_MAC_IN_VECTOR_TX_INT_VEC BIT(16 + EMAC_DEF_TX_CH)
203201
#define EMAC_DM646X_MAC_IN_VECTOR_HOST_INT BIT(26)
204202
#define EMAC_DM646X_MAC_IN_VECTOR_STATPEND_INT BIT(27)
205203

206-
207204
/* CPPI bit positions */
208205
#define EMAC_CPPI_SOP_BIT BIT(31)
209206
#define EMAC_CPPI_EOP_BIT BIT(30)
@@ -747,8 +744,7 @@ static void emac_update_phystatus(struct emac_priv *priv)
747744

748745
if (priv->speed == SPEED_1000 && (priv->version == EMAC_VERSION_2)) {
749746
mac_control = emac_read(EMAC_MACCONTROL);
750-
mac_control |= (EMAC_DM646X_MACCONTORL_GMIIEN |
751-
EMAC_DM646X_MACCONTORL_GIG |
747+
mac_control |= (EMAC_DM646X_MACCONTORL_GIG |
752748
EMAC_DM646X_MACCONTORL_GIGFORCE);
753749
} else {
754750
/* Clear the GIG bit and GIGFORCE bit */
@@ -2105,7 +2101,7 @@ static int emac_hw_enable(struct emac_priv *priv)
21052101

21062102
/* Enable MII */
21072103
val = emac_read(EMAC_MACCONTROL);
2108-
val |= (EMAC_MACCONTROL_MIIEN);
2104+
val |= (EMAC_MACCONTROL_GMIIEN);
21092105
emac_write(EMAC_MACCONTROL, val);
21102106

21112107
/* Enable NAPI and interrupts */

0 commit comments

Comments
 (0)