Quoting Stephen Rothwell (2018-10-17 14:51:53) > Hi all, > > After merging the clk tree, today's linux-next build (x86_64 allmodconfig) > produced this warning: > > WARNING: modpost: missing MODULE_LICENSE() in drivers/clk/qcom/gcc-sdm660.o > see include/linux/module.h for more information > > Introduced by commit > > f2a76a2955c0 ("clk: qcom: Add Global Clock controller (GCC) driver for > SDM660") > Thanks! I silenced it.
Hi Boris, On Mon, 12 Dec 2016 09:05:21 +0100 Boris Brezillon
wrote: > > May I ask which toolchain you're using, because my gcc failed to detect > this uninitialized variable (and this is not the first time I have this > problem). I use a powerpc-le hosted cross compiler gcc-5.2.0. -- Cheers, Stephen Rothwell
+Eric Hi Stephen, On Mon, 12 Dec 2016 08:39:44 +1100 Stephen Rothwell
wrote: > Hi all, > > After merging the clk tree, today's linux-next build (arm > multi_v7_defconfig) produced this warning: > > drivers/clk/bcm/clk-bcm2835.c: In function 'bcm2835_clock_determine_rate': > drivers/clk/bcm/clk-bcm2835.c:1069:18: warning: 'best_rate' may be used > uninitialized in this function [-Wmaybe-uninitialized] > *prate = curdiv * best_rate; > ^ > drivers/clk/bcm/clk-bcm2835.c:1032:16: note: 'best_rate' was declared here > unsigned long best_rate; > ^ > > Introduced by commit > > 155e8b3b0ee3 ("clk: bcm: Support rate change propagation on bcm2835 clocks") > Thanks for the report (I just sent a patch fixing this warning). May I ask which toolchain you're using, because my gcc failed to detect this uninitialized variable (and this is not the first time I have this problem). Regards, Boris
On 05/06, Stephen Rothwell wrote: Hi all, After merging the clk tree, today's linux-next build (x86_64 allmodconfig) produced this warning: drivers/clk/clk.c:2231:13: warning: 'clk_is_orphan' defined but not used [-Wunused-function] static bool clk_is_orphan(const struct clk *clk) ^ Introduced by commit ece3ffbe1b7b (clk: prevent orphan clocks from being used). CONFIG_OF is not set for this build ... Thanks for the report. kbuild robot also reported this problem the other day. We came up with a way to not have this function at all though so this problem should go away tomorrow. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/