Hi Eric, On 31/08/2017 17:34, Eric Nelson wrote: > Update these macros to use all upper-case to avoid checkpatch > warnings: > > ENET_25MHz, > ENET_50MHz, > ENET_125MHz, > > Signed-off-by: Eric Nelson <e...@nelint.com> > --- > V2 fixes a couple of references in mx7/clock.c > arch/arm/include/asm/arch-mx7/clock.h | 6 +++--- > arch/arm/mach-imx/mx7/clock.c | 6 +++--- > board/freescale/mx7dsabresd/mx7dsabresd.c | 2 +- > board/technexion/pico-imx7d/pico-imx7d.c | 2 +- > board/toradex/colibri_imx7/colibri_imx7.c | 2 +- > 5 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/arch/arm/include/asm/arch-mx7/clock.h > b/arch/arm/include/asm/arch-mx7/clock.h > index 688d236..3b115ad 100644 > --- a/arch/arm/include/asm/arch-mx7/clock.h > +++ b/arch/arm/include/asm/arch-mx7/clock.h > @@ -318,9 +318,9 @@ struct clk_root_map { > }; > > enum enet_freq { > - ENET_25MHz, > - ENET_50MHz, > - ENET_125MHz, > + ENET_25MHZ, > + ENET_50MHZ, > + ENET_125MHZ, > }; > > u32 get_root_clk(enum clk_root_index clock_id); > diff --git a/arch/arm/mach-imx/mx7/clock.c b/arch/arm/mach-imx/mx7/clock.c > index 2cfde46..8150faa 100644 > --- a/arch/arm/mach-imx/mx7/clock.c > +++ b/arch/arm/mach-imx/mx7/clock.c > @@ -966,15 +966,15 @@ int set_clk_enet(enum enet_freq type) > clock_enable(CCGR_ENET2, 0); > > switch (type) { > - case ENET_125MHz: > + case ENET_125MHZ: > enet1_ref = ENET1_REF_CLK_ROOT_FROM_PLL_ENET_MAIN_125M_CLK; > enet2_ref = ENET2_REF_CLK_ROOT_FROM_PLL_ENET_MAIN_125M_CLK; > break; > - case ENET_50MHz: > + case ENET_50MHZ: > enet1_ref = ENET1_REF_CLK_ROOT_FROM_PLL_ENET_MAIN_50M_CLK; > enet2_ref = ENET2_REF_CLK_ROOT_FROM_PLL_ENET_MAIN_50M_CLK; > break; > - case ENET_25MHz: > + case ENET_25MHZ: > enet1_ref = ENET1_REF_CLK_ROOT_FROM_PLL_ENET_MAIN_25M_CLK; > enet2_ref = ENET2_REF_CLK_ROOT_FROM_PLL_ENET_MAIN_25M_CLK; > break; > diff --git a/board/freescale/mx7dsabresd/mx7dsabresd.c > b/board/freescale/mx7dsabresd/mx7dsabresd.c > index a681ece..5819b18 100644 > --- a/board/freescale/mx7dsabresd/mx7dsabresd.c > +++ b/board/freescale/mx7dsabresd/mx7dsabresd.c > @@ -260,7 +260,7 @@ static int setup_fec(void) > (IOMUXC_GPR_GPR1_GPR_ENET1_TX_CLK_SEL_MASK | > IOMUXC_GPR_GPR1_GPR_ENET1_CLK_DIR_MASK), 0); > > - return set_clk_enet(ENET_125MHz); > + return set_clk_enet(ENET_125MHZ); > } > > > diff --git a/board/technexion/pico-imx7d/pico-imx7d.c > b/board/technexion/pico-imx7d/pico-imx7d.c > index b4c9be7..67bab51 100644 > --- a/board/technexion/pico-imx7d/pico-imx7d.c > +++ b/board/technexion/pico-imx7d/pico-imx7d.c > @@ -182,7 +182,7 @@ static int setup_fec(void) > (IOMUXC_GPR_GPR1_GPR_ENET1_TX_CLK_SEL_MASK | > IOMUXC_GPR_GPR1_GPR_ENET1_CLK_DIR_MASK), 0); > > - return set_clk_enet(ENET_125MHz); > + return set_clk_enet(ENET_125MHZ); > } > > int board_phy_config(struct phy_device *phydev) > diff --git a/board/toradex/colibri_imx7/colibri_imx7.c > b/board/toradex/colibri_imx7/colibri_imx7.c > index 5cb14b4..13b2b57 100644 > --- a/board/toradex/colibri_imx7/colibri_imx7.c > +++ b/board/toradex/colibri_imx7/colibri_imx7.c > @@ -280,7 +280,7 @@ static int setup_fec(void) > IOMUXC_GPR_GPR1_GPR_ENET1_TX_CLK_SEL_MASK); > #endif > > - return set_clk_enet(ENET_50MHz); > + return set_clk_enet(ENET_50MHZ); > } > > int board_phy_config(struct phy_device *phydev) >
Reviewed-by: Stefano Babic <sba...@denx.de> Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot