[linux-next:master 6122/12912] drivers/clk/ingenic/tcu.c:326:34: warning: unused variable 'ingenic_tcu_of_match'

[linux-next:master 6122/12912] drivers/clk/ingenic/tcu.c:326:34: warning: unused variable 'ingenic_tcu_of_match'

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 2609 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   b0523c7b1c9d0edcd6c0fe6d2cb558a9ad5c60a8
commit: bbd7ffdbef6888459f301c5889f3b14ada38b913 [6122/12912] clk: Allow the common clk framework to be selectable
config: mips-randconfig-r012-20200526 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 3393cc4cebf9969db94dc424b7a2b6195589c33b)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install mips cross compiling tool for clang build
        # apt-get install binutils-mips-linux-gnu
        git checkout bbd7ffdbef6888459f301c5889f3b14ada38b913
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>

All warnings (new ones prefixed by >>, old ones prefixed by <<):

>> drivers/clk/ingenic/tcu.c:326:34: warning: unused variable 'ingenic_tcu_of_match' [-Wunused-const-variable]
static const struct of_device_id ingenic_tcu_of_match[] __initconst = {
^
1 warning generated.

vim +/ingenic_tcu_of_match +326 drivers/clk/ingenic/tcu.c

dc6a81c3382f74f 周琰杰 (Zhou Yanjie  2020-03-17  325) 
4f89e4b8f1215c1 Paul Cercueil        2019-07-24 @326  static const struct of_device_id ingenic_tcu_of_match[] __initconst = {
4f89e4b8f1215c1 Paul Cercueil        2019-07-24  327  	{ .compatible = "ingenic,jz4740-tcu", .data = &jz4740_soc_info, },
4f89e4b8f1215c1 Paul Cercueil        2019-07-24  328  	{ .compatible = "ingenic,jz4725b-tcu", .data = &jz4725b_soc_info, },
4f89e4b8f1215c1 Paul Cercueil        2019-07-24  329  	{ .compatible = "ingenic,jz4770-tcu", .data = &jz4770_soc_info, },
dc6a81c3382f74f 周琰杰 (Zhou Yanjie  2020-03-17  330) 	{ .compatible = "ingenic,x1000-tcu", .data = &x1000_soc_info, },
4f89e4b8f1215c1 Paul Cercueil        2019-07-24  331  	{ /* sentinel */ }
4f89e4b8f1215c1 Paul Cercueil        2019-07-24  332  };
4f89e4b8f1215c1 Paul Cercueil        2019-07-24  333  

:::::: The code at line 326 was first introduced by commit
:::::: 4f89e4b8f1215c113d87081efc99f28b3fcb6292 clk: ingenic: Add driver for the TCU clocks

:::::: TO: Paul Cercueil <paul@crapouillou.net>
:::::: CC: Paul Burton <paul.burton@mips.com>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 33283 bytes --]

Re: [linux-next:master 6122/12912] drivers/clk/ingenic/tcu.c:326:34: warning: unused variable 'ingenic_tcu_of_match'

[Stephen Boyd]

[-- Attachment #1: Type: text/plain, Size: 1446 bytes --]

Quoting kbuild test robot (2020-05-26 10:54:49)
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head:   b0523c7b1c9d0edcd6c0fe6d2cb558a9ad5c60a8
> commit: bbd7ffdbef6888459f301c5889f3b14ada38b913 [6122/12912] clk: Allow the common clk framework to be selectable
> config: mips-randconfig-r012-20200526 (attached as .config)
> compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 3393cc4cebf9969db94dc424b7a2b6195589c33b)
> reproduce (this is a W=1 build):
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # install mips cross compiling tool for clang build
>         # apt-get install binutils-mips-linux-gnu
>         git checkout bbd7ffdbef6888459f301c5889f3b14ada38b913
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips 
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> All warnings (new ones prefixed by >>, old ones prefixed by <<):
> 
> >> drivers/clk/ingenic/tcu.c:326:34: warning: unused variable 'ingenic_tcu_of_match' [-Wunused-const-variable]
> static const struct of_device_id ingenic_tcu_of_match[] __initconst = {

Looks like we need to add a __maybe_unused marking here. I'll do that
now.

Re: [linux-next:master 6122/12912] drivers/clk/ingenic/tcu.c:326:34: warning: unused variable 'ingenic_tcu_of_match'

[Arnd Bergmann]

[-- Attachment #1: Type: text/plain, Size: 1666 bytes --]

On Fri, May 29, 2020 at 1:32 AM Stephen Boyd <sboyd@kernel.org> wrote:
>
> Quoting kbuild test robot (2020-05-26 10:54:49)
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > head:   b0523c7b1c9d0edcd6c0fe6d2cb558a9ad5c60a8
> > commit: bbd7ffdbef6888459f301c5889f3b14ada38b913 [6122/12912] clk: Allow the common clk framework to be selectable
> > config: mips-randconfig-r012-20200526 (attached as .config)
> > compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 3393cc4cebf9969db94dc424b7a2b6195589c33b)
> > reproduce (this is a W=1 build):
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         # install mips cross compiling tool for clang build
> >         # apt-get install binutils-mips-linux-gnu
> >         git checkout bbd7ffdbef6888459f301c5889f3b14ada38b913
> >         # save the attached .config to linux build tree
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kbuild test robot <lkp@intel.com>
> >
> > All warnings (new ones prefixed by >>, old ones prefixed by <<):
> >
> > >> drivers/clk/ingenic/tcu.c:326:34: warning: unused variable 'ingenic_tcu_of_match' [-Wunused-const-variable]
> > static const struct of_device_id ingenic_tcu_of_match[] __initconst = {
>
> Looks like we need to add a __maybe_unused marking here. I'll do that
> now.

Either that, or mark the driver 'depends on OF', which is probably
needed anyway.

      Arnd