Hi Peng, On 30/10/2015 10:30, Peng Fan wrote:
get_clocks is wrapped by CONFIG_FSL_CLK and CONFIG_M68K in seperate piece code. They can be merged into one snippet.Signed-off-by: Peng Fan <peng....@freescale.com> Cc: Tom Rini <tr...@konsulko.com> Cc: Simon Glass <s...@chromium.org> Cc: Bin Meng <bmeng...@gmail.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: "ang...@sysam.it" <ang...@sysam.it> Cc: Daniel Schwierzeck <daniel.schwierz...@gmail.com> Cc: Stephen Warren <swar...@nvidia.com> Cc: "Andreas Bießmann" <andreas.de...@googlemail.com> Reviewed-by: Simon Glass <s...@chromium.org> --- Changes V2: Refine patch subject as Simon suggested. Add Simon's Reviewed-by common/board_f.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/common/board_f.c b/common/board_f.c index 486e828..7bc05b1 100644 --- a/common/board_f.c +++ b/common/board_f.c @@ -806,10 +806,7 @@ static init_fnc_t init_sequence_f[] = { #if defined(CONFIG_BOARD_POSTCLK_INIT) board_postclk_init, #endif -#ifdef CONFIG_FSL_CLK - get_clocks, -#endif -#ifdef CONFIG_M68K +#if defined(CONFIG_FSL_CLK) || defined(CONFIG_M68K) get_clocks, #endif env_init, /* initialize environment */
change is ok for me. Acked-by: Angelo Dureghello <ang...@sysam.it> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
