* drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match'
@ 2023-07-24 15:20 kernel test robot
2023-07-25 8:03 ` Sergio Paracuellos
0 siblings, 1 reply; 5+ messages in thread
From: kernel test robot @ 2023-07-24 15:20 UTC (permalink / raw)
To: Sergio Paracuellos; +Cc: llvm, oe-kbuild-all, linux-kernel, Thomas Bogendoerfer
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 6eaae198076080886b9e7d57f4ae06fa782f90ef
commit: 6f3b15586eef736831abe6a14f2a6906bc0dc074 clk: ralink: add clock and reset driver for MTMIPS SoCs
date: 5 weeks ago
config: x86_64-buildonly-randconfig-r003-20230723 (https://download.01.org/0day-ci/archive/20230724/202307242310.CdOnd2py-lkp@intel.com/config)
compiler: clang version 15.0.7 (https://github.com/llvm/llvm-project.git 8dfdcc7b7bf66834a761bd8de445840ef68e4d1a)
reproduce: (https://download.01.org/0day-ci/archive/20230724/202307242310.CdOnd2py-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202307242310.CdOnd2py-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/clk/ralink/clk-mtmips.c:309:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
return ret;
^~~
drivers/clk/ralink/clk-mtmips.c:285:9: note: initialize the variable 'ret' to silence this warning
int ret, i;
^
= 0
drivers/clk/ralink/clk-mtmips.c:359:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
return ret;
^~~
drivers/clk/ralink/clk-mtmips.c:335:9: note: initialize the variable 'ret' to silence this warning
int ret, i;
^
= 0
>> drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match' [-Wunused-const-variable]
static const struct of_device_id mtmips_of_match[] = {
^
3 warnings generated.
vim +/mtmips_of_match +821 drivers/clk/ralink/clk-mtmips.c
820
> 821 static const struct of_device_id mtmips_of_match[] = {
822 {
823 .compatible = "ralink,rt2880-sysc",
824 .data = &rt2880_clk_data,
825 },
826 {
827 .compatible = "ralink,rt3050-sysc",
828 .data = &rt305x_clk_data,
829 },
830 {
831 .compatible = "ralink,rt3052-sysc",
832 .data = &rt305x_clk_data,
833 },
834 {
835 .compatible = "ralink,rt3352-sysc",
836 .data = &rt3352_clk_data,
837 },
838 {
839 .compatible = "ralink,rt3883-sysc",
840 .data = &rt3883_clk_data,
841 },
842 {
843 .compatible = "ralink,rt5350-sysc",
844 .data = &rt5350_clk_data,
845 },
846 {
847 .compatible = "ralink,mt7620-sysc",
848 .data = &mt7620_clk_data,
849 },
850 {
851 .compatible = "ralink,mt7628-sysc",
852 .data = &mt76x8_clk_data,
853 },
854 {
855 .compatible = "ralink,mt7688-sysc",
856 .data = &mt76x8_clk_data,
857 },
858 {}
859 };
860
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match'
2023-07-24 15:20 kernel test robot
@ 2023-07-25 8:03 ` Sergio Paracuellos
2023-08-01 16:28 ` Nick Desaulniers
0 siblings, 1 reply; 5+ messages in thread
From: Sergio Paracuellos @ 2023-07-25 8:03 UTC (permalink / raw)
To: kernel test robot; +Cc: llvm, oe-kbuild-all, linux-kernel, Thomas Bogendoerfer
Hi!
On Mon, Jul 24, 2023 at 5:21 PM kernel test robot <lkp@intel.com> wrote:
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 6eaae198076080886b9e7d57f4ae06fa782f90ef
> commit: 6f3b15586eef736831abe6a14f2a6906bc0dc074 clk: ralink: add clock and reset driver for MTMIPS SoCs
> date: 5 weeks ago
> config: x86_64-buildonly-randconfig-r003-20230723 (https://download.01.org/0day-ci/archive/20230724/202307242310.CdOnd2py-lkp@intel.com/config)
> compiler: clang version 15.0.7 (https://github.com/llvm/llvm-project.git 8dfdcc7b7bf66834a761bd8de445840ef68e4d1a)
> reproduce: (https://download.01.org/0day-ci/archive/20230724/202307242310.CdOnd2py-lkp@intel.com/reproduce)
>
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202307242310.CdOnd2py-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
> drivers/clk/ralink/clk-mtmips.c:309:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
> return ret;
> ^~~
> drivers/clk/ralink/clk-mtmips.c:285:9: note: initialize the variable 'ret' to silence this warning
> int ret, i;
> ^
> = 0
> drivers/clk/ralink/clk-mtmips.c:359:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
> return ret;
> ^~~
> drivers/clk/ralink/clk-mtmips.c:335:9: note: initialize the variable 'ret' to silence this warning
> int ret, i;
> ^
> = 0
This two errors were already fixed in:
6e68dae946e3: ("clk: ralink: mtmips: Fix uninitialized use of ret in
mtmips_register_{fixed,factor}_clocks()")
> >> drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match' [-Wunused-const-variable]
> static const struct of_device_id mtmips_of_match[] = {
> ^
> 3 warnings generated.
This one is because CONFIG_OF is disabled so 'of_match_node()' turns
into a dummy macro. What would be the correct approach here?
Add select OF in Kconfig for this driver or silence the warning
declaring 'mtmips_of_match' with __maybe_unused?
Thanks in advance for clarification.
Best regards,
Sergio Paracuellos
>
>
> vim +/mtmips_of_match +821 drivers/clk/ralink/clk-mtmips.c
>
> 820
> > 821 static const struct of_device_id mtmips_of_match[] = {
> 822 {
> 823 .compatible = "ralink,rt2880-sysc",
> 824 .data = &rt2880_clk_data,
> 825 },
> 826 {
> 827 .compatible = "ralink,rt3050-sysc",
> 828 .data = &rt305x_clk_data,
> 829 },
> 830 {
> 831 .compatible = "ralink,rt3052-sysc",
> 832 .data = &rt305x_clk_data,
> 833 },
> 834 {
> 835 .compatible = "ralink,rt3352-sysc",
> 836 .data = &rt3352_clk_data,
> 837 },
> 838 {
> 839 .compatible = "ralink,rt3883-sysc",
> 840 .data = &rt3883_clk_data,
> 841 },
> 842 {
> 843 .compatible = "ralink,rt5350-sysc",
> 844 .data = &rt5350_clk_data,
> 845 },
> 846 {
> 847 .compatible = "ralink,mt7620-sysc",
> 848 .data = &mt7620_clk_data,
> 849 },
> 850 {
> 851 .compatible = "ralink,mt7628-sysc",
> 852 .data = &mt76x8_clk_data,
> 853 },
> 854 {
> 855 .compatible = "ralink,mt7688-sysc",
> 856 .data = &mt76x8_clk_data,
> 857 },
> 858 {}
> 859 };
> 860
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match'
2023-07-25 8:03 ` Sergio Paracuellos
@ 2023-08-01 16:28 ` Nick Desaulniers
0 siblings, 0 replies; 5+ messages in thread
From: Nick Desaulniers @ 2023-08-01 16:28 UTC (permalink / raw)
To: Sergio Paracuellos
Cc: kernel test robot, llvm, oe-kbuild-all, linux-kernel,
Thomas Bogendoerfer
On Tue, Jul 25, 2023 at 1:04 AM Sergio Paracuellos
<sergio.paracuellos@gmail.com> wrote:
>
> Hi!
>
> On Mon, Jul 24, 2023 at 5:21 PM kernel test robot <lkp@intel.com> wrote:
> >
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > head: 6eaae198076080886b9e7d57f4ae06fa782f90ef
> > commit: 6f3b15586eef736831abe6a14f2a6906bc0dc074 clk: ralink: add clock and reset driver for MTMIPS SoCs
> > date: 5 weeks ago
> > config: x86_64-buildonly-randconfig-r003-20230723 (https://download.01.org/0day-ci/archive/20230724/202307242310.CdOnd2py-lkp@intel.com/config)
> > compiler: clang version 15.0.7 (https://github.com/llvm/llvm-project.git 8dfdcc7b7bf66834a761bd8de445840ef68e4d1a)
> > reproduce: (https://download.01.org/0day-ci/archive/20230724/202307242310.CdOnd2py-lkp@intel.com/reproduce)
> >
> > If you fix the issue in a separate patch/commit (i.e. not just a new version of
> > the same patch/commit), kindly add following tags
> > | Reported-by: kernel test robot <lkp@intel.com>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202307242310.CdOnd2py-lkp@intel.com/
> >
> > All warnings (new ones prefixed by >>):
> >
> > drivers/clk/ralink/clk-mtmips.c:309:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
> > return ret;
> > ^~~
> > drivers/clk/ralink/clk-mtmips.c:285:9: note: initialize the variable 'ret' to silence this warning
> > int ret, i;
> > ^
> > = 0
> > drivers/clk/ralink/clk-mtmips.c:359:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
> > return ret;
> > ^~~
> > drivers/clk/ralink/clk-mtmips.c:335:9: note: initialize the variable 'ret' to silence this warning
> > int ret, i;
> > ^
> > = 0
>
> This two errors were already fixed in:
>
> 6e68dae946e3: ("clk: ralink: mtmips: Fix uninitialized use of ret in
> mtmips_register_{fixed,factor}_clocks()")
>
> > >> drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match' [-Wunused-const-variable]
> > static const struct of_device_id mtmips_of_match[] = {
> > ^
> > 3 warnings generated.
>
> This one is because CONFIG_OF is disabled so 'of_match_node()' turns
> into a dummy macro. What would be the correct approach here?
> Add select OF in Kconfig for this driver or silence the warning
> declaring 'mtmips_of_match' with __maybe_unused?
That or perhaps wrapping the definition of `mtmips_of_match` in
`#ifdef CONFIG_OF`?
>
> Thanks in advance for clarification.
>
> Best regards,
> Sergio Paracuellos
>
> >
> >
> > vim +/mtmips_of_match +821 drivers/clk/ralink/clk-mtmips.c
> >
> > 820
> > > 821 static const struct of_device_id mtmips_of_match[] = {
> > 822 {
> > 823 .compatible = "ralink,rt2880-sysc",
> > 824 .data = &rt2880_clk_data,
> > 825 },
> > 826 {
> > 827 .compatible = "ralink,rt3050-sysc",
> > 828 .data = &rt305x_clk_data,
> > 829 },
> > 830 {
> > 831 .compatible = "ralink,rt3052-sysc",
> > 832 .data = &rt305x_clk_data,
> > 833 },
> > 834 {
> > 835 .compatible = "ralink,rt3352-sysc",
> > 836 .data = &rt3352_clk_data,
> > 837 },
> > 838 {
> > 839 .compatible = "ralink,rt3883-sysc",
> > 840 .data = &rt3883_clk_data,
> > 841 },
> > 842 {
> > 843 .compatible = "ralink,rt5350-sysc",
> > 844 .data = &rt5350_clk_data,
> > 845 },
> > 846 {
> > 847 .compatible = "ralink,mt7620-sysc",
> > 848 .data = &mt7620_clk_data,
> > 849 },
> > 850 {
> > 851 .compatible = "ralink,mt7628-sysc",
> > 852 .data = &mt76x8_clk_data,
> > 853 },
> > 854 {
> > 855 .compatible = "ralink,mt7688-sysc",
> > 856 .data = &mt76x8_clk_data,
> > 857 },
> > 858 {}
> > 859 };
> > 860
> >
> > --
> > 0-DAY CI Kernel Test Service
> > https://github.com/intel/lkp-tests/wiki
>
--
Thanks,
~Nick Desaulniers
^ permalink raw reply [flat|nested] 5+ messages in thread
* drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match'
@ 2023-10-14 19:24 kernel test robot
2023-10-15 7:03 ` Sergio Paracuellos
0 siblings, 1 reply; 5+ messages in thread
From: kernel test robot @ 2023-10-14 19:24 UTC (permalink / raw)
To: Sergio Paracuellos; +Cc: llvm, oe-kbuild-all, linux-kernel, Thomas Bogendoerfer
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 70f8c6f8f8800d970b10676cceae42bba51a4899
commit: 6f3b15586eef736831abe6a14f2a6906bc0dc074 clk: ralink: add clock and reset driver for MTMIPS SoCs
date: 4 months ago
config: hexagon-randconfig-r016-20230817 (https://download.01.org/0day-ci/archive/20231015/202310150352.KyKovpbT-lkp@intel.com/config)
compiler: clang version 16.0.4 (https://github.com/llvm/llvm-project.git ae42196bc493ffe877a7e3dff8be32035dea4d07)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231015/202310150352.KyKovpbT-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202310150352.KyKovpbT-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from drivers/clk/ralink/clk-mtmips.c:11:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:13:
In file included from arch/hexagon/include/asm/io.h:334:
include/asm-generic/io.h:547:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __raw_readb(PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:560:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:37:51: note: expanded from macro '__le16_to_cpu'
#define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
^
In file included from drivers/clk/ralink/clk-mtmips.c:11:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:13:
In file included from arch/hexagon/include/asm/io.h:334:
include/asm-generic/io.h:573:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:35:51: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
^
In file included from drivers/clk/ralink/clk-mtmips.c:11:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:13:
In file included from arch/hexagon/include/asm/io.h:334:
include/asm-generic/io.h:584:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writeb(value, PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:594:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:604:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
drivers/clk/ralink/clk-mtmips.c:309:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
return ret;
^~~
drivers/clk/ralink/clk-mtmips.c:285:9: note: initialize the variable 'ret' to silence this warning
int ret, i;
^
= 0
drivers/clk/ralink/clk-mtmips.c:359:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
return ret;
^~~
drivers/clk/ralink/clk-mtmips.c:335:9: note: initialize the variable 'ret' to silence this warning
int ret, i;
^
= 0
>> drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match' [-Wunused-const-variable]
static const struct of_device_id mtmips_of_match[] = {
^
9 warnings generated.
vim +/mtmips_of_match +821 drivers/clk/ralink/clk-mtmips.c
820
> 821 static const struct of_device_id mtmips_of_match[] = {
822 {
823 .compatible = "ralink,rt2880-sysc",
824 .data = &rt2880_clk_data,
825 },
826 {
827 .compatible = "ralink,rt3050-sysc",
828 .data = &rt305x_clk_data,
829 },
830 {
831 .compatible = "ralink,rt3052-sysc",
832 .data = &rt305x_clk_data,
833 },
834 {
835 .compatible = "ralink,rt3352-sysc",
836 .data = &rt3352_clk_data,
837 },
838 {
839 .compatible = "ralink,rt3883-sysc",
840 .data = &rt3883_clk_data,
841 },
842 {
843 .compatible = "ralink,rt5350-sysc",
844 .data = &rt5350_clk_data,
845 },
846 {
847 .compatible = "ralink,mt7620-sysc",
848 .data = &mt7620_clk_data,
849 },
850 {
851 .compatible = "ralink,mt7628-sysc",
852 .data = &mt76x8_clk_data,
853 },
854 {
855 .compatible = "ralink,mt7688-sysc",
856 .data = &mt76x8_clk_data,
857 },
858 {}
859 };
860
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match'
2023-10-14 19:24 drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match' kernel test robot
@ 2023-10-15 7:03 ` Sergio Paracuellos
0 siblings, 0 replies; 5+ messages in thread
From: Sergio Paracuellos @ 2023-10-15 7:03 UTC (permalink / raw)
To: kernel test robot; +Cc: llvm, oe-kbuild-all, linux-kernel, Thomas Bogendoerfer
Hi,
On Sat, Oct 14, 2023 at 9:25 PM kernel test robot <lkp@intel.com> wrote:
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 70f8c6f8f8800d970b10676cceae42bba51a4899
> commit: 6f3b15586eef736831abe6a14f2a6906bc0dc074 clk: ralink: add clock and reset driver for MTMIPS SoCs
> date: 4 months ago
> config: hexagon-randconfig-r016-20230817 (https://download.01.org/0day-ci/archive/20231015/202310150352.KyKovpbT-lkp@intel.com/config)
> compiler: clang version 16.0.4 (https://github.com/llvm/llvm-project.git ae42196bc493ffe877a7e3dff8be32035dea4d07)
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231015/202310150352.KyKovpbT-lkp@intel.com/reproduce)
>
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202310150352.KyKovpbT-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
> In file included from drivers/clk/ralink/clk-mtmips.c:11:
> In file included from include/linux/regmap.h:20:
> In file included from include/linux/iopoll.h:14:
> In file included from include/linux/io.h:13:
> In file included from arch/hexagon/include/asm/io.h:334:
> include/asm-generic/io.h:547:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
> val = __raw_readb(PCI_IOBASE + addr);
> ~~~~~~~~~~ ^
> include/asm-generic/io.h:560:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
> val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
> ~~~~~~~~~~ ^
> include/uapi/linux/byteorder/little_endian.h:37:51: note: expanded from macro '__le16_to_cpu'
> #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
> ^
> In file included from drivers/clk/ralink/clk-mtmips.c:11:
> In file included from include/linux/regmap.h:20:
> In file included from include/linux/iopoll.h:14:
> In file included from include/linux/io.h:13:
> In file included from arch/hexagon/include/asm/io.h:334:
> include/asm-generic/io.h:573:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
> val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
> ~~~~~~~~~~ ^
> include/uapi/linux/byteorder/little_endian.h:35:51: note: expanded from macro '__le32_to_cpu'
> #define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
> ^
> In file included from drivers/clk/ralink/clk-mtmips.c:11:
> In file included from include/linux/regmap.h:20:
> In file included from include/linux/iopoll.h:14:
> In file included from include/linux/io.h:13:
> In file included from arch/hexagon/include/asm/io.h:334:
> include/asm-generic/io.h:584:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
> __raw_writeb(value, PCI_IOBASE + addr);
> ~~~~~~~~~~ ^
> include/asm-generic/io.h:594:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
> __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
> ~~~~~~~~~~ ^
> include/asm-generic/io.h:604:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
> __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
> ~~~~~~~~~~ ^
> drivers/clk/ralink/clk-mtmips.c:309:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
> return ret;
> ^~~
> drivers/clk/ralink/clk-mtmips.c:285:9: note: initialize the variable 'ret' to silence this warning
> int ret, i;
> ^
> = 0
> drivers/clk/ralink/clk-mtmips.c:359:9: warning: variable 'ret' is uninitialized when used here [-Wuninitialized]
> return ret;
> ^~~
> drivers/clk/ralink/clk-mtmips.c:335:9: note: initialize the variable 'ret' to silence this warning
> int ret, i;
> ^
> = 0
> >> drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match' [-Wunused-const-variable]
> static const struct of_device_id mtmips_of_match[] = {
> ^
> 9 warnings generated.
I already sent this patch some time ago and still waiting for its review:
https://lore.kernel.org/lkml/20230827023932.501102-1-sergio.paracuellos@gmail.com/t/
[snip]
Thanks,
Sergio Paracuellos
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-10-15 7:03 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-14 19:24 drivers/clk/ralink/clk-mtmips.c:821:34: warning: unused variable 'mtmips_of_match' kernel test robot
2023-10-15 7:03 ` Sergio Paracuellos
-- strict thread matches above, loose matches on Subject: below --
2023-07-24 15:20 kernel test robot
2023-07-25 8:03 ` Sergio Paracuellos
2023-08-01 16:28 ` Nick Desaulniers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox