[U-Boot] [PATCH v2] MPC837x: set i2c1_clk
Kim Phillips
kim.phillips at freescale.com
Mon May 2 18:31:36 CEST 2011
On Thu, 14 Apr 2011 14:57:40 +0200
Andre Schwarz <andre.schwarz at matrix-vision.de> wrote:
> Running on mpc837x without CONFIG_FSL_ESDHC leads to
> i2c1_clk not being set at all. It is bound to clock
> of encryption module. fix this.
>
>
> Signed-off-by: Andre Schwarz <andre.schwarz at matrix-vision.de>
> ---
> Changes for v2:
> - fix commit message mangling in subject line.
thanks for listing change matter, subsequent versions of patches
should inherit prior versions "In-reply-to:" and "References:" headers.
> arch/powerpc/cpu/mpc83xx/speed.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/arch/powerpc/cpu/mpc83xx/speed.c b/arch/powerpc/cpu/mpc83xx/speed.c
> index 5e616dd..1eca1a8 100644
> --- a/arch/powerpc/cpu/mpc83xx/speed.c
> +++ b/arch/powerpc/cpu/mpc83xx/speed.c
> @@ -327,6 +327,8 @@ int get_clocks(void)
> i2c1_clk = enc_clk;
> #elif defined(CONFIG_FSL_ESDHC)
> i2c1_clk = sdhc_clk;
> +#elif defined(CONFIG_MPC837x)
> + i2c1_clk = enc_clk;
> #endif
can you consolidate this and the other i2c1_clk-to-enc_clk assignments
in one #elif defined(...) || defined(...), etc.
not sure if the FSL_ESDHC ifdef would need to be moved above that.
btw, subjects in this patchseries should be of the form [PATCH x/y].
Thanks,
Kim
More information about the U-Boot
mailing list