> -----Original Message-----
> From: Zhiqiang Hou [mailto:zhiqiang....@nxp.com]
> Sent: Monday, September 12, 2016 9:39 AM
> To: u-boot@lists.denx.de; albert.u.b...@aribaud.net; york sun
> <york....@nxp.com>; Vincent Hu <mingkai...@nxp.com>; Prabhakar
> Kushwaha <prabhakar.kushw...@nxp.com>; Calvin Johnson
> <calvin.john...@nxp.com>
> Cc: Z.Q. Hou <zhiqiang....@nxp.com>
> Subject: [PATCH 2/2] armv8/fsl-lsch3: consolidate the clock system
> initialization
>
> From: Hou Zhiqiang <zhiqiang....@nxp.com>
>
> This patch map the sys_info->freq_systembus to Platform PLL, and
> implement the IPs' clock function individually.
>
> Signed-off-by: Hou Zhiqiang <zhiqiang....@nxp.com>
> ---
> .../arm/cpu/armv8/fsl-layerscape/fsl_lsch3_speed.c | 26 +++++++++++++++++--
> ---
> .../include/asm/arch-fsl-layerscape/immap_lsch3.h | 1 +
> include/configs/ls2080a_common.h | 2 +-
> 3 files changed, 22 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/cpu/armv8/fsl-layerscape/fsl_lsch3_speed.c
> b/arch/arm/cpu/armv8/fsl-layerscape/fsl_lsch3_speed.c
> index a9b12a4..daad80a 100644
> --- a/arch/arm/cpu/armv8/fsl-layerscape/fsl_lsch3_speed.c
> +++ b/arch/arm/cpu/armv8/fsl-layerscape/fsl_lsch3_speed.c
> @@ -88,11 +88,10 @@ void get_sys_info(struct sys_info *sys_info)
> #endif
> #endif
>
> + /* The freq_systembus is used to record frequency of platform PLL */
> sys_info->freq_systembus *= (gur_in32(&gur->rcwsr[0]) >>
> FSL_CHASSIS3_RCWSR0_SYS_PLL_RAT_SHIFT) &
> FSL_CHASSIS3_RCWSR0_SYS_PLL_RAT_MASK;
> - /* Platform clock is half of platform PLL */
> - sys_info->freq_systembus /= 2;
This is required only for ls2080A and ls2088A otherwise u-boot will be printing
dicken speed in boot log.
> sys_info->freq_ddrbus *= (gur_in32(&gur->rcwsr[0]) >>
> FSL_CHASSIS3_RCWSR0_MEM_PLL_RAT_SHIFT) &
> FSL_CHASSIS3_RCWSR0_MEM_PLL_RAT_MASK;
> @@ -132,7 +131,7 @@ void get_sys_info(struct sys_info *sys_info)
> ccr = ifc_in32(&ifc_regs.gregs->ifc_ccr);
> ccr = ((ccr & IFC_CCR_CLK_DIV_MASK) >> IFC_CCR_CLK_DIV_SHIFT) + 1;
>
> - sys_info->freq_localbus = sys_info->freq_systembus / ccr;
> + sys_info->freq_localbus = sys_info->freq_systembus / 2 / ccr;
> #endif
> }
>
> @@ -159,7 +158,7 @@ int get_clocks(void)
>
> /********************************************
> * get_bus_freq
> - * return system bus freq in Hz
> + * return platform PLL freq in Hz
> *********************************************/
> ulong get_bus_freq(ulong dummy)
> {
> @@ -190,13 +189,28 @@ ulong get_ddr_freq(ulong ctrl_num)
> return gd->mem_clk;
> }
>
> +int get_i2c_freq(ulong dummy)
> +{
> + return get_bus_freq(0) / 4;
> +}
> +
> +int get_dspi_freq(ulong dummy)
> +{
> + return get_bus_freq(0) / 4;
> +}
> +
> +int get_serial_clock(void)
> +{
> + return get_bus_freq(0) / 4;
> +}
> +
get_bus_freq(0) / 4 --> May not be true for LS1088A. So Avoid this for above
code.
--prabhakar
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
