On Thu, Oct 29, 2015 at 11:15:51AM -0600, Simon Glass wrote: >On 29 October 2015 at 01:57, Peng Fan <[email protected]> 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 <[email protected]> >> Cc: Tom Rini <[email protected]> >> Cc: Simon Glass <[email protected]> >> Cc: Bin Meng <[email protected]> >> Cc: Alexey Brodkin <[email protected]> >> Cc: "[email protected]" <[email protected]> >> Cc: Daniel Schwierzeck <[email protected]> >> Cc: Stephen Warren <[email protected]> >> Cc: "Andreas Bießmann" <[email protected]> >> --- >> 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 <[email protected]> -- _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
