Skip to contentSkip to conversation

arm: mx6: Change defines ENET_xxMHz to ENET_xxMHZ (no CamelCase)

PATCH Superseded Stefan Roese 2 participants 2 messages Nov 19, 2014 – Nov 21, 2014 Arm (32-bit)

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 arm: mx6: Change defines ENET_xxMHz to ENET_xxMHZ (no CamelCase)

As checkpatch complaines about these camel-case defines, lets change them to only use upper-case characters.

Cc Heiko Schocher <[email protected]>
Cc Fabio Estevam <[email protected]>
Cc Jon Nettleton <[email protected]>
Cc Stefano Babic <[email protected]>
---
 arch/arm/cpu/armv7/mx6/clock.c              | 2 +-
 arch/arm/include/asm/arch-mx6/clock.h       | 8 ++++----
 board/aristainetos/aristainetos.c           | 2 +-
 board/freescale/mx6slevk/mx6slevk.c         | 2 +-
 board/freescale/mx6sxsabresd/mx6sxsabresd.c | 2 +-
 board/solidrun/hummingboard/hummingboard.c  | 2 +-
 6 files changed, 9 insertions(+), 9 deletions(-)
arch/arm/cpu/armv7/mx6/clock.c +1 -1
index 6c9c78c..bf2a6ec 100644
@@ -443,7 +443,7 @@ int enable_fec_anatop_clock(enum enet_freq freq)
 	struct anatop_regs __iomem *anatop =
 		(struct anatop_regs __iomem *)ANATOP_BASE_ADDR;
 
-	if (freq < ENET_25MHz || freq > ENET_125MHz)
+	if (freq < ENET_25MHZ || freq > ENET_125MHZ)
 		return -EINVAL;
 
 	reg = readl(&anatop->pll_enet);
arch/arm/include/asm/arch-mx6/clock.h +4 -4
index 3c58a0a..ec5d026 100644
@@ -43,10 +43,10 @@ enum mxc_clock {
 };
 
 enum enet_freq {
-	ENET_25MHz,
-	ENET_50MHz,
-	ENET_100MHz,
-	ENET_125MHz,
+	ENET_25MHZ,
+	ENET_50MHZ,
+	ENET_100MHZ,
+	ENET_125MHZ,
 };
 
 u32 imx_get_uartclk(void);
board/aristainetos/aristainetos.c +1 -1
index 06922c0..67ac260 100644
@@ -301,7 +301,7 @@ int board_eth_init(bd_t *bis)
 	/* clear gpr1[14], gpr1[18:17] to select anatop clock */
 	clrsetbits_le32(&iomuxc_regs->gpr[1], IOMUX_GPR1_FEC_MASK, 0);
 
-	ret = enable_fec_anatop_clock(ENET_50MHz);
+	ret = enable_fec_anatop_clock(ENET_50MHZ);
 	if (ret)
 		return ret;
 
board/freescale/mx6slevk/mx6slevk.c +1 -1
index e76c343..ffd1117 100644
@@ -235,7 +235,7 @@ static int setup_fec(void)
 	/* clear gpr1[14], gpr1[18:17] to select anatop clock */
 	clrsetbits_le32(&iomuxc_regs->gpr[1], IOMUX_GPR1_FEC_MASK, 0);
 
-	ret = enable_fec_anatop_clock(ENET_50MHz);
+	ret = enable_fec_anatop_clock(ENET_50MHZ);
 	if (ret)
 		return ret;
 
board/freescale/mx6sxsabresd/mx6sxsabresd.c +1 -1
index 68d3718..42928f8 100644
@@ -140,7 +140,7 @@ static int setup_fec(void)
 	reg |= BM_ANADIG_PLL_ENET_REF_25M_ENABLE;
 	writel(reg, &anatop->pll_enet);
 
-	ret = enable_fec_anatop_clock(ENET_125MHz);
+	ret = enable_fec_anatop_clock(ENET_125MHZ);
 	if (ret)
 		return ret;
 
board/solidrun/hummingboard/hummingboard.c +1 -1
index 6d204b3..52c384b 100644
@@ -146,7 +146,7 @@ int board_eth_init(bd_t *bis)
 {
 	struct iomuxc *const iomuxc_regs = (struct iomuxc *)IOMUXC_BASE_ADDR;
 
-	int ret = enable_fec_anatop_clock(ENET_25MHz);
+	int ret = enable_fec_anatop_clock(ENET_25MHZ);
 	if (ret)
 		return ret;
 
Signature · 1 line
2.1.3
#1 lore raw

PATCH 1/1 arm: mx6: Change defines ENET_xxMHz to ENET_xxMHZ (no CamelCase)

Hello Stefan,

Am 19.11.2014 16:30, schrieb Stefan Roese:

Quoted 8 lines

As checkpatch complaines about these camel-case defines, lets change them to only use upper-case characters.

Cc Heiko Schocher <[email protected]>
Cc Fabio Estevam <[email protected]>
Cc Jon Nettleton <[email protected]>
Cc Stefano Babic <[email protected]>
Quoted 8 lines
---
  arch/arm/cpu/armv7/mx6/clock.c              | 2 +-
  arch/arm/include/asm/arch-mx6/clock.h       | 8 ++++----
  board/aristainetos/aristainetos.c           | 2 +-
  board/freescale/mx6slevk/mx6slevk.c         | 2 +-
  board/freescale/mx6sxsabresd/mx6sxsabresd.c | 2 +-
  board/solidrun/hummingboard/hummingboard.c  | 2 +-
  6 files changed, 9 insertions(+), 9 deletions(-)

Thanks!

bye,
Heiko

Signature · 2 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany