On 25 March 2017 at 21:48, Simon Glass <s...@chromium.org> wrote: > On 24 March 2017 at 12:24, Philipp Tomsich > <philipp.toms...@theobroma-systems.com> wrote: >> Due to differences in the code paths for SPL and non-SPL, some static >> constant structures remain unused in each build variant. This raises >> warnings with recent GCC versions (we currently use GCC-6.3). >> >> The warnings addressed in this commit (by matching #if conditions for >> the variable definition with their uses) are: >> >> * for the SPL build: >> drivers/clk/rockchip/clk_rk3399.c:53:29: warning: 'cpll_init_cfg' >> defined but not used [-Wunused-const-variable=] >> static const struct pll_div cpll_init_cfg = PLL_DIVISORS(CPLL_HZ, 1, 2, >> 2); >> ^~~~~~~~~~~~~ >> drivers/clk/rockchip/clk_rk3399.c:52:29: warning: 'gpll_init_cfg' >> defined but not used [-Wunused-const-variable=] >> static const struct pll_div gpll_init_cfg = PLL_DIVISORS(GPLL_HZ, 2, 2, >> 1); >> ^~~~~~~~~~~~~ >> * for the non-SPL build: >> drivers/clk/rockchip/clk_rk3399.c:54:29: warning: 'ppll_init_cfg' >> defined but not used [-Wunused-const-variable=] >> static const struct pll_div ppll_init_cfg = PLL_DIVISORS(PPLL_HZ, 2, 2, >> 1); >> ^~~~~~~~~~~~~ >> >> Signed-off-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com> >> --- >> >> drivers/clk/rockchip/clk_rk3399.c | 5 +++++ >> 1 file changed, 5 insertions(+) > > Acked-by: Simon Glass <s...@chromium.org>
Applied to u-boot-rockchip, thanks! _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot