Re: linux-next: build warning after merge of the clk tree

2018-10-18 Thread Stephen Boyd
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.



linux-next: build warning after merge of the clk tree

2018-10-17 Thread Stephen Rothwell
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")

-- 
Cheers,
Stephen Rothwell


pgp3FfFnVjYlC.pgp
Description: OpenPGP digital signature


Re: linux-next: build warning after merge of the clk tree

2016-12-12 Thread Stephen Rothwell
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


Re: linux-next: build warning after merge of the clk tree

2016-12-12 Thread Boris Brezillon
+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


linux-next: build warning after merge of the clk tree

2016-12-11 Thread Stephen Rothwell
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")

-- 
Cheers,
Stephen Rothwell


linux-next: build warning after merge of the clk tree

2015-05-05 Thread Stephen Rothwell
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 ...

-- 
Cheers,
Stephen Rothwells...@canb.auug.org.au


pgp2RFVV5FHul.pgp
Description: OpenPGP digital signature


Re: linux-next: build warning after merge of the clk tree

2015-05-05 Thread Stephen Boyd
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/