[U-Boot] [PATCH v2 04/10] arm: vf610: add anadig pll5 definitions

Albert ARIBAUD
2013-10-17T12:38:41Z

Hi Marcel,

On Mon, 30 Sep 2013 13:26:09 +0200, Marcel Ziswiler
<marcel@ziswiler.com> wrote:

> Add ANADIG PLL5 control definitions required for Ethernet RMII clock
> configuration.
>
> Signed-off-by: Marcel Ziswiler <marcel@ziswiler.com>
> ---
> arch/arm/include/asm/arch-vf610/crm_regs.h | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm/include/asm/arch-vf610/crm_regs.h b/arch/arm/include/asm/arch-vf610/crm_regs.h
> index 04cc5bc..e17c7d1 100644
> --- a/arch/arm/include/asm/arch-vf610/crm_regs.h
> +++ b/arch/arm/include/asm/arch-vf610/crm_regs.h
> @@ -187,6 +187,10 @@ struct anadig_reg {
> #define CCM_CCGR9_FEC0_CTRL_MASK 0x3
> #define CCM_CCGR9_FEC1_CTRL_MASK (0x3 << 2)
>
> +#define ANADIG_PLL5_CTRL_BYPASS (1 << 16)
> +#define ANADIG_PLL5_CTRL_ENABLE (1 << 13)
> +#define ANADIG_PLL5_CTRL_POWERDOWN (1 << 12)
> +#define ANADIG_PLL5_CTRL_DIV_SELECT 1
> #define ANADIG_PLL2_CTRL_ENABLE (1 << 13)
> #define ANADIG_PLL2_CTRL_POWERDOWN (1 << 12)
> #define ANADIG_PLL2_CTRL_DIV_SELECT 1

This and 3/10 could be merged as they touch the same file and do the
same thing, adding definitions.

Ditto for 06, 07 and 08/10 which all touch the same file.

I'd even say all addition patches (3/10 to 8/10) could be grouped
together in one logical change, like "add all definitions needed for
vf610".

Amicalement,
--
Albert.