Hi Eric,
Am Samstag, 1. April 2017, 22:42:54 CEST schrieb eric.gao@rock-chips.com:
From: "eric.gao" eric.gao@rock-chips.com
Signed-off-by: eric.gao eric.gao@rock-chips.com
drivers/clk/rockchip/clk_rk3399.c | 4 ++++ 1 file changed, 4 insertions(+)
diff --git a/drivers/clk/rockchip/clk_rk3399.c b/drivers/clk/rockchip/clk_rk3399.c index 922ce7e..cfcd1e4 100644 --- a/drivers/clk/rockchip/clk_rk3399.c +++ b/drivers/clk/rockchip/clk_rk3399.c @@ -20,6 +20,10 @@
DECLARE_GLOBAL_DATA_PTR;
+#ifndef CONFIG_SPL_BUILD +#define CONFIG_SPL_BUILD +#endif
That looks very strange. In all cases your commit message should explain why a change is necessary, i.e. what you want to achieve.
In this special case, also rkclk_init runs from SPL context and inits the clocks already, the full u-boot then finds the clocks already set up as necessary and should have no need to for rkclk_init at all itself.
Heiko