On Thu, Oct 29, 2015 at 11:15:51AM -0600, Simon Glass wrote: >On 29 October 2015 at 01:57, Peng Fan <peng....@freescale.com> 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> >> --- >> common/board_f.c | 5 +---- >> 1 file changed, 1 insertion(+), 4 deletions(-) > >nit: I think it would be better to use a subject like "common: >Simplify get_clocks() #ifdef".
Thanks, will take your suggestion and send out v2. Regards, Peng. > >Reviewed-by: Simon Glass <s...@chromium.org> -- _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot