From mboxrd@z Thu Jan  1 00:00:00 1970
From: York Sun <yorksun@freescale.com>
Date: Wed, 21 Oct 2015 09:52:23 -0700
Subject: [U-Boot] [Patch V6 02/18] common/board_f.c: modify the macro to
 use get_clocks() more common
In-Reply-To: <1445255949-25021-3-git-send-email-b52263@rhuath.am.freescale.net>
References: <1445255949-25021-1-git-send-email-b52263@rhuath.am.freescale.net>
 <1445255949-25021-2-git-send-email-b52263@rhuath.am.freescale.net>
 <1445255949-25021-3-git-send-email-b52263@rhuath.am.freescale.net>
Message-ID: <5627C2C7.5090901@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de



On 10/19/2015 04:58 AM, Gong Qianyu wrote:
> From: Gong Qianyu <Qianyu.Gong@freescale.com>
> 
> get_clocks() should not be limited by ESDHC.
> 
> Signed-off-by: Gong Qianyu <Qianyu.Gong@freescale.com>
> ---
> V6:
>  - No change.
> V5:
>  - No change.
> V4:
>  - No change.
> V3:
>  - Removed defines in PPC configs that have no need to use.
> V2:
>  - No change.
> 
>  common/board_f.c                  | 2 +-
>  include/configs/colibri_vf.h      | 1 +
>  include/configs/ls1021aqds.h      | 1 +
>  include/configs/ls1021atwr.h      | 1 +
>  include/configs/ls2085aqds.h      | 1 +
>  include/configs/ls2085ardb.h      | 1 +
>  include/configs/m53evk.h          | 1 +
>  include/configs/mx25pdk.h         | 1 +
>  include/configs/mx35pdk.h         | 1 +
>  include/configs/mx51evk.h         | 1 +
>  include/configs/mx53ard.h         | 1 +
>  include/configs/mx53evk.h         | 1 +
>  include/configs/mx53loco.h        | 1 +
>  include/configs/mx53smd.h         | 1 +
>  include/configs/mx6_common.h      | 1 +
>  include/configs/mx7_common.h      | 1 +
>  include/configs/usbarmory.h       | 1 +
>  include/configs/vf610twr.h        | 1 +
>  include/configs/woodburn_common.h | 1 +
>  19 files changed, 19 insertions(+), 1 deletion(-)
> 
> diff --git a/common/board_f.c b/common/board_f.c
> index 613332e..1bb84b3 100644
> --- a/common/board_f.c
> +++ b/common/board_f.c
> @@ -806,7 +806,7 @@ static init_fnc_t init_sequence_f[] = {
>  #if defined(CONFIG_BOARD_POSTCLK_INIT)
>  	board_postclk_init,
>  #endif
> -#ifdef CONFIG_FSL_ESDHC
> +#ifdef CONFIG_FSL_CLK

Qianyu,

I feel it is more like a hardware requirement to have clocks calculated for the
rest to operate. Maybe CONFIG_SYS_FSL_CLK is slightly better fit? Anyway please
add it to README to document the usage.

York