Message ID | 1491057777-1640-5-git-send-email-eric.gao@rock-chips.com |
---|---|
State | Changes Requested |
Delegated to: | Simon Glass |
Headers | show |
Dear Heiko, Thanks for your review. I will add commit for all patches. The reason I add this declaration is that spl for rk3399 is still not ready, I used ourself rkloader, which can not initial clock suitable for video module. Anyway, I will drop this signal patch and wait for spl ready. Thank you. On 2017年04月02日 19:52, Heiko Stuebner wrote: > 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 > > >
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 + #if CONFIG_IS_ENABLED(OF_PLATDATA) struct rk3399_clk_plat { struct dtd_rockchip_rk3399_cru dtd;