From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] arm: mx6: Change defines ENET_xxMHz to ENET_xxMHZ (no CamelCase)
Date: Mon, 01 Dec 2014 10:38:15 +0100 [thread overview]
Message-ID: <547C3707.8020104@denx.de> (raw)
In-Reply-To: <1417092403-26965-1-git-send-email-sr@denx.de>
On 27/11/2014 13:46, Stefan Roese wrote:
> As checkpatch complaines about these camel-case defines, lets change
> them to only use upper-case characters.
>
> Signed-off-by: Stefan Roese <sr@denx.de>
> Acked-by: Heiko Schocher <hs@denx.de>
> Cc: Fabio Estevam <fabio.estevam@freescale.com>
> Cc: Jon Nettleton <jon.nettleton@gmail.com>
> Cc: Stefano Babic <sbabic@denx.de>
> ---
> v2:
> - Rebased on current master
>
> 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(-)
>
> diff --git a/arch/arm/cpu/armv7/mx6/clock.c b/arch/arm/cpu/armv7/mx6/clock.c
> index ab7ac3d..93a02ad 100644
> --- a/arch/arm/cpu/armv7/mx6/clock.c
> +++ b/arch/arm/cpu/armv7/mx6/clock.c
> @@ -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);
> diff --git a/arch/arm/include/asm/arch-mx6/clock.h b/arch/arm/include/asm/arch-mx6/clock.h
> index 323805c..226a4cd 100644
> --- a/arch/arm/include/asm/arch-mx6/clock.h
> +++ b/arch/arm/include/asm/arch-mx6/clock.h
> @@ -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);
> diff --git a/board/aristainetos/aristainetos.c b/board/aristainetos/aristainetos.c
> index 06922c0..67ac260 100644
> --- a/board/aristainetos/aristainetos.c
> +++ b/board/aristainetos/aristainetos.c
> @@ -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;
>
> diff --git a/board/freescale/mx6slevk/mx6slevk.c b/board/freescale/mx6slevk/mx6slevk.c
> index 8111edf..cac6d73 100644
> --- a/board/freescale/mx6slevk/mx6slevk.c
> +++ b/board/freescale/mx6slevk/mx6slevk.c
> @@ -234,7 +234,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);
>
> - return enable_fec_anatop_clock(ENET_50MHz);
> + return enable_fec_anatop_clock(ENET_50MHZ);
> }
> #endif
>
> diff --git a/board/freescale/mx6sxsabresd/mx6sxsabresd.c b/board/freescale/mx6sxsabresd/mx6sxsabresd.c
> index 7aee074..8b959b9 100644
> --- a/board/freescale/mx6sxsabresd/mx6sxsabresd.c
> +++ b/board/freescale/mx6sxsabresd/mx6sxsabresd.c
> @@ -168,7 +168,7 @@ static int setup_fec(void)
> reg |= BM_ANADIG_PLL_ENET_REF_25M_ENABLE;
> writel(reg, &anatop->pll_enet);
>
> - return enable_fec_anatop_clock(ENET_125MHz);
> + return enable_fec_anatop_clock(ENET_125MHZ);
> }
>
> int board_eth_init(bd_t *bis)
> diff --git a/board/solidrun/hummingboard/hummingboard.c b/board/solidrun/hummingboard/hummingboard.c
> index 6d204b3..52c384b 100644
> --- a/board/solidrun/hummingboard/hummingboard.c
> +++ b/board/solidrun/hummingboard/hummingboard.c
> @@ -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;
>
>
Applied to u-boot-imx, thanks !
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
next prev parent reply other threads:[~2014-12-01 9:38 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-27 12:46 [U-Boot] [PATCH v2] arm: mx6: Change defines ENET_xxMHz to ENET_xxMHZ (no CamelCase) Stefan Roese
2014-11-27 12:53 ` Fabio Estevam
2014-12-01 9:38 ` Stefano Babic [this message]
2014-12-15 22:34 ` Pavel Machek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=547C3707.8020104@denx.de \
--to=sbabic@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.