Re: [linux-next:master 3824/6048] drivers/net/dsa/xrs700x/xrs700x_i2c.c:127:34: warning: unused variable 'xrs700x_i2c_dt_ids'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 9, 2021 at 10:33 AM Souptick Joarder <jrdr.linux@xxxxxxxxx> wrote:
>
> On Sat, Feb 6, 2021 at 11:32 PM kernel test robot <lkp@xxxxxxxxx> wrote:
> >
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > head:   59fa6a163ffabc1bf25c5e0e33899e268a96d3cc
> > commit: ee00b24f32eb822f55190efd1078fe572e931d5c [3824/6048] net: dsa: add Arrow SpeedChips XRS700x driver
> > config: x86_64-randconfig-a011-20210207 (attached as .config)
> > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project c9439ca36342fb6013187d0a69aef92736951476)
> > 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 x86_64 cross compiling tool for clang build
> >         # apt-get install binutils-x86-64-linux-gnu
> >         # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=ee00b24f32eb822f55190efd1078fe572e931d5c
> >         git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> >         git fetch --no-tags linux-next master
> >         git checkout ee00b24f32eb822f55190efd1078fe572e931d5c
> >         # save the attached .config to linux build tree
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@xxxxxxxxx>
> >
> > All warnings (new ones prefixed by >>):
> >
> > >> drivers/net/dsa/xrs700x/xrs700x_i2c.c:127:34: warning: unused variable 'xrs700x_i2c_dt_ids' [-Wunused-const-variable]
> >    static const struct of_device_id xrs700x_i2c_dt_ids[] = {
> >                                     ^
> >    1 warning generated.
>
> I think, this can defined under #ifdef CONFIG_OF.

Was this robot reporting this warning for other DSA drivers? A lot of
them appear to use of_device_id in the same manner (unless I'm missing
something). Do we need to do something DSA wide to deal with this?

>
> >
> >
> > vim +/xrs700x_i2c_dt_ids +127 drivers/net/dsa/xrs700x/xrs700x_i2c.c
> >
> >    126
> >  > 127  static const struct of_device_id xrs700x_i2c_dt_ids[] = {
> >    128          { .compatible = "arrow,xrs7003e", .data = &xrs7003e_info },
> >    129          { .compatible = "arrow,xrs7003f", .data = &xrs7003f_info },
> >    130          { .compatible = "arrow,xrs7004e", .data = &xrs7004e_info },
> >    131          { .compatible = "arrow,xrs7004f", .data = &xrs7004f_info },
> >    132          {},
> >    133  };
> >    134  MODULE_DEVICE_TABLE(of, xrs700x_i2c_dt_ids);
> >    135
> >
> > ---
> > 0-DAY CI Kernel Test Service, Intel Corporation
> > https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

-George




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux