All of lore.kernel.org
 help / color / mirror / Atom feed
* [linux-next:master 6546/14055] arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared
@ 2021-06-29 13:10 ` kernel test robot
  0 siblings, 0 replies; 6+ messages in thread
From: kernel test robot @ 2021-06-29 13:10 UTC (permalink / raw)
  To: Sergio Paracuellos
  Cc: kbuild-all, Linux Memory Management List, Linus Walleij

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   73748627df83aab934c81332ca83a44ab8c7b3e3
commit: 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384 [6546/14055] pinctrl: ralink: move RT305X SoC pinmux config into a new 'pinctrl-rt305x.c' file
config: mips-randconfig-c003-20210629 (attached as .config)
compiler: mipsel-linux-gcc (GCC) 9.3.0
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
        # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
        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 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
        # save the attached .config to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross O=build_dir ARCH=mips SHELL=/bin/bash

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

All errors (new ones prefixed by >>):

   In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
>> arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared (first use in this function)
      18 |  return ralink_soc == RT305X_SOC_RT3050;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:18:23: note: each undeclared identifier is reported only once for each function it appears in
>> arch/mips/include/asm/mach-ralink/rt305x.h:18:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      18 |  return ralink_soc == RT305X_SOC_RT3050;
         |  ^~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
>> arch/mips/include/asm/mach-ralink/rt305x.h:23:23: error: 'RT305X_SOC_RT3052' undeclared (first use in this function)
      23 |  return ralink_soc == RT305X_SOC_RT3052;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:23:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      23 |  return ralink_soc == RT305X_SOC_RT3052;
         |  ^~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
>> arch/mips/include/asm/mach-ralink/rt305x.h:33:23: error: 'RT305X_SOC_RT3350' undeclared (first use in this function)
      33 |  return ralink_soc == RT305X_SOC_RT3350;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:33:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      33 |  return ralink_soc == RT305X_SOC_RT3350;
         |  ^~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
>> arch/mips/include/asm/mach-ralink/rt305x.h:38:23: error: 'RT305X_SOC_RT3352' undeclared (first use in this function)
      38 |  return ralink_soc == RT305X_SOC_RT3352;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:38:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      38 |  return ralink_soc == RT305X_SOC_RT3352;
         |  ^~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt5350':
>> arch/mips/include/asm/mach-ralink/rt305x.h:43:23: error: 'RT305X_SOC_RT5350' undeclared (first use in this function)
      43 |  return ralink_soc == RT305X_SOC_RT5350;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:43:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      43 |  return ralink_soc == RT305X_SOC_RT5350;
         |  ^~~~~~
   In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
   arch/mips/include/asm/mach-ralink/rt305x.h:44:1: error: control reaches end of non-void function [-Werror=return-type]
      44 | }
         | ^
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
   arch/mips/include/asm/mach-ralink/rt305x.h:19:1: error: control reaches end of non-void function [-Werror=return-type]
      19 | }
         | ^
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
   arch/mips/include/asm/mach-ralink/rt305x.h:24:1: error: control reaches end of non-void function [-Werror=return-type]
      24 | }
         | ^
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
   arch/mips/include/asm/mach-ralink/rt305x.h:34:1: error: control reaches end of non-void function [-Werror=return-type]
      34 | }
         | ^
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
   arch/mips/include/asm/mach-ralink/rt305x.h:39:1: error: control reaches end of non-void function [-Werror=return-type]
      39 | }
         | ^
   cc1: some warnings being treated as errors


vim +/RT305X_SOC_RT3050 +18 arch/mips/include/asm/mach-ralink/rt305x.h

2809b31770d7fd John Crispin 2013-01-20  15  
2809b31770d7fd John Crispin 2013-01-20  16  static inline int soc_is_rt3050(void)
2809b31770d7fd John Crispin 2013-01-20  17  {
418d29c8706170 John Crispin 2015-11-04 @18  	return ralink_soc == RT305X_SOC_RT3050;
2809b31770d7fd John Crispin 2013-01-20  19  }
2809b31770d7fd John Crispin 2013-01-20  20  
2809b31770d7fd John Crispin 2013-01-20  21  static inline int soc_is_rt3052(void)
2809b31770d7fd John Crispin 2013-01-20  22  {
418d29c8706170 John Crispin 2015-11-04 @23  	return ralink_soc == RT305X_SOC_RT3052;
2809b31770d7fd John Crispin 2013-01-20  24  }
2809b31770d7fd John Crispin 2013-01-20  25  
2809b31770d7fd John Crispin 2013-01-20  26  static inline int soc_is_rt305x(void)
2809b31770d7fd John Crispin 2013-01-20  27  {
2809b31770d7fd John Crispin 2013-01-20  28  	return soc_is_rt3050() || soc_is_rt3052();
2809b31770d7fd John Crispin 2013-01-20  29  }
2809b31770d7fd John Crispin 2013-01-20  30  
2809b31770d7fd John Crispin 2013-01-20  31  static inline int soc_is_rt3350(void)
2809b31770d7fd John Crispin 2013-01-20  32  {
418d29c8706170 John Crispin 2015-11-04 @33  	return ralink_soc == RT305X_SOC_RT3350;
2809b31770d7fd John Crispin 2013-01-20  34  }
2809b31770d7fd John Crispin 2013-01-20  35  
2809b31770d7fd John Crispin 2013-01-20  36  static inline int soc_is_rt3352(void)
2809b31770d7fd John Crispin 2013-01-20  37  {
418d29c8706170 John Crispin 2015-11-04 @38  	return ralink_soc == RT305X_SOC_RT3352;
2809b31770d7fd John Crispin 2013-01-20  39  }
2809b31770d7fd John Crispin 2013-01-20  40  
2809b31770d7fd John Crispin 2013-01-20  41  static inline int soc_is_rt5350(void)
2809b31770d7fd John Crispin 2013-01-20  42  {
418d29c8706170 John Crispin 2015-11-04 @43  	return ralink_soc == RT305X_SOC_RT5350;
2809b31770d7fd John Crispin 2013-01-20  44  }
2809b31770d7fd John Crispin 2013-01-20  45  

:::::: The code at line 18 was first introduced by commit
:::::: 418d29c8706170c6801998e48341181b274a47a8 MIPS: ralink: Unify SoC id handling

:::::: TO: John Crispin <blogic@openwrt.org>
:::::: CC: Ralf Baechle <ralf@linux-mips.org>

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

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [linux-next:master 6546/14055] arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared
@ 2021-06-29 13:10 ` kernel test robot
  0 siblings, 0 replies; 6+ messages in thread
From: kernel test robot @ 2021-06-29 13:10 UTC (permalink / raw)
  To: kbuild-all

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   73748627df83aab934c81332ca83a44ab8c7b3e3
commit: 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384 [6546/14055] pinctrl: ralink: move RT305X SoC pinmux config into a new 'pinctrl-rt305x.c' file
config: mips-randconfig-c003-20210629 (attached as .config)
compiler: mipsel-linux-gcc (GCC) 9.3.0
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
        # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
        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 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
        # save the attached .config to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross O=build_dir ARCH=mips SHELL=/bin/bash

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

All errors (new ones prefixed by >>):

   In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
>> arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared (first use in this function)
      18 |  return ralink_soc == RT305X_SOC_RT3050;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:18:23: note: each undeclared identifier is reported only once for each function it appears in
>> arch/mips/include/asm/mach-ralink/rt305x.h:18:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      18 |  return ralink_soc == RT305X_SOC_RT3050;
         |  ^~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
>> arch/mips/include/asm/mach-ralink/rt305x.h:23:23: error: 'RT305X_SOC_RT3052' undeclared (first use in this function)
      23 |  return ralink_soc == RT305X_SOC_RT3052;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:23:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      23 |  return ralink_soc == RT305X_SOC_RT3052;
         |  ^~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
>> arch/mips/include/asm/mach-ralink/rt305x.h:33:23: error: 'RT305X_SOC_RT3350' undeclared (first use in this function)
      33 |  return ralink_soc == RT305X_SOC_RT3350;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:33:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      33 |  return ralink_soc == RT305X_SOC_RT3350;
         |  ^~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
>> arch/mips/include/asm/mach-ralink/rt305x.h:38:23: error: 'RT305X_SOC_RT3352' undeclared (first use in this function)
      38 |  return ralink_soc == RT305X_SOC_RT3352;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:38:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      38 |  return ralink_soc == RT305X_SOC_RT3352;
         |  ^~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt5350':
>> arch/mips/include/asm/mach-ralink/rt305x.h:43:23: error: 'RT305X_SOC_RT5350' undeclared (first use in this function)
      43 |  return ralink_soc == RT305X_SOC_RT5350;
         |                       ^~~~~~~~~~~~~~~~~
   arch/mips/include/asm/mach-ralink/rt305x.h:43:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
      43 |  return ralink_soc == RT305X_SOC_RT5350;
         |  ^~~~~~
   In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
   arch/mips/include/asm/mach-ralink/rt305x.h:44:1: error: control reaches end of non-void function [-Werror=return-type]
      44 | }
         | ^
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
   arch/mips/include/asm/mach-ralink/rt305x.h:19:1: error: control reaches end of non-void function [-Werror=return-type]
      19 | }
         | ^
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
   arch/mips/include/asm/mach-ralink/rt305x.h:24:1: error: control reaches end of non-void function [-Werror=return-type]
      24 | }
         | ^
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
   arch/mips/include/asm/mach-ralink/rt305x.h:34:1: error: control reaches end of non-void function [-Werror=return-type]
      34 | }
         | ^
   arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
   arch/mips/include/asm/mach-ralink/rt305x.h:39:1: error: control reaches end of non-void function [-Werror=return-type]
      39 | }
         | ^
   cc1: some warnings being treated as errors


vim +/RT305X_SOC_RT3050 +18 arch/mips/include/asm/mach-ralink/rt305x.h

2809b31770d7fd John Crispin 2013-01-20  15  
2809b31770d7fd John Crispin 2013-01-20  16  static inline int soc_is_rt3050(void)
2809b31770d7fd John Crispin 2013-01-20  17  {
418d29c8706170 John Crispin 2015-11-04 @18  	return ralink_soc == RT305X_SOC_RT3050;
2809b31770d7fd John Crispin 2013-01-20  19  }
2809b31770d7fd John Crispin 2013-01-20  20  
2809b31770d7fd John Crispin 2013-01-20  21  static inline int soc_is_rt3052(void)
2809b31770d7fd John Crispin 2013-01-20  22  {
418d29c8706170 John Crispin 2015-11-04 @23  	return ralink_soc == RT305X_SOC_RT3052;
2809b31770d7fd John Crispin 2013-01-20  24  }
2809b31770d7fd John Crispin 2013-01-20  25  
2809b31770d7fd John Crispin 2013-01-20  26  static inline int soc_is_rt305x(void)
2809b31770d7fd John Crispin 2013-01-20  27  {
2809b31770d7fd John Crispin 2013-01-20  28  	return soc_is_rt3050() || soc_is_rt3052();
2809b31770d7fd John Crispin 2013-01-20  29  }
2809b31770d7fd John Crispin 2013-01-20  30  
2809b31770d7fd John Crispin 2013-01-20  31  static inline int soc_is_rt3350(void)
2809b31770d7fd John Crispin 2013-01-20  32  {
418d29c8706170 John Crispin 2015-11-04 @33  	return ralink_soc == RT305X_SOC_RT3350;
2809b31770d7fd John Crispin 2013-01-20  34  }
2809b31770d7fd John Crispin 2013-01-20  35  
2809b31770d7fd John Crispin 2013-01-20  36  static inline int soc_is_rt3352(void)
2809b31770d7fd John Crispin 2013-01-20  37  {
418d29c8706170 John Crispin 2015-11-04 @38  	return ralink_soc == RT305X_SOC_RT3352;
2809b31770d7fd John Crispin 2013-01-20  39  }
2809b31770d7fd John Crispin 2013-01-20  40  
2809b31770d7fd John Crispin 2013-01-20  41  static inline int soc_is_rt5350(void)
2809b31770d7fd John Crispin 2013-01-20  42  {
418d29c8706170 John Crispin 2015-11-04 @43  	return ralink_soc == RT305X_SOC_RT5350;
2809b31770d7fd John Crispin 2013-01-20  44  }
2809b31770d7fd John Crispin 2013-01-20  45  

:::::: The code at line 18 was first introduced by commit
:::::: 418d29c8706170c6801998e48341181b274a47a8 MIPS: ralink: Unify SoC id handling

:::::: TO: John Crispin <blogic@openwrt.org>
:::::: CC: Ralf Baechle <ralf@linux-mips.org>

---
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: 26033 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [linux-next:master 6546/14055] arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared
  2021-06-29 13:10 ` kernel test robot
@ 2021-06-29 14:16   ` Sergio Paracuellos
  -1 siblings, 0 replies; 6+ messages in thread
From: Sergio Paracuellos @ 2021-06-29 14:16 UTC (permalink / raw)
  To: kernel test robot; +Cc: kbuild-all, Linux Memory Management List, Linus Walleij

Hi,

On Tue, Jun 29, 2021 at 3:10 PM kernel test robot <lkp@intel.com> wrote:
>
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head:   73748627df83aab934c81332ca83a44ab8c7b3e3
> commit: 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384 [6546/14055] pinctrl: ralink: move RT305X SoC pinmux config into a new 'pinctrl-rt305x.c' file
> config: mips-randconfig-c003-20210629 (attached as .config)
> compiler: mipsel-linux-gcc (GCC) 9.3.0
> 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
>         # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
>         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 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
>         # save the attached .config to linux build tree
>         mkdir build_dir
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross O=build_dir ARCH=mips SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
>    In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared (first use in this function)
>       18 |  return ralink_soc == RT305X_SOC_RT3050;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:18:23: note: each undeclared identifier is reported only once for each function it appears in
> >> arch/mips/include/asm/mach-ralink/rt305x.h:18:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       18 |  return ralink_soc == RT305X_SOC_RT3050;
>          |  ^~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:23:23: error: 'RT305X_SOC_RT3052' undeclared (first use in this function)
>       23 |  return ralink_soc == RT305X_SOC_RT3052;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:23:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       23 |  return ralink_soc == RT305X_SOC_RT3052;
>          |  ^~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:33:23: error: 'RT305X_SOC_RT3350' undeclared (first use in this function)
>       33 |  return ralink_soc == RT305X_SOC_RT3350;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:33:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       33 |  return ralink_soc == RT305X_SOC_RT3350;
>          |  ^~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:38:23: error: 'RT305X_SOC_RT3352' undeclared (first use in this function)
>       38 |  return ralink_soc == RT305X_SOC_RT3352;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:38:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       38 |  return ralink_soc == RT305X_SOC_RT3352;
>          |  ^~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt5350':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:43:23: error: 'RT305X_SOC_RT5350' undeclared (first use in this function)
>       43 |  return ralink_soc == RT305X_SOC_RT5350;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:43:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       43 |  return ralink_soc == RT305X_SOC_RT5350;
>          |  ^~~~~~
>    In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
>    arch/mips/include/asm/mach-ralink/rt305x.h:44:1: error: control reaches end of non-void function [-Werror=return-type]
>       44 | }
>          | ^
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
>    arch/mips/include/asm/mach-ralink/rt305x.h:19:1: error: control reaches end of non-void function [-Werror=return-type]
>       19 | }
>          | ^
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
>    arch/mips/include/asm/mach-ralink/rt305x.h:24:1: error: control reaches end of non-void function [-Werror=return-type]
>       24 | }
>          | ^
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
>    arch/mips/include/asm/mach-ralink/rt305x.h:34:1: error: control reaches end of non-void function [-Werror=return-type]
>       34 | }
>          | ^
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
>    arch/mips/include/asm/mach-ralink/rt305x.h:39:1: error: control reaches end of non-void function [-Werror=return-type]
>       39 | }
>          | ^
>    cc1: some warnings being treated as errors

There is a missing include after creating the file
'drivers/pinctrl/ralink/pinctrl-rt305x.c': #include
<asm/mach-ralink/ralink_regs.h>

Previous to this move, 'arch/mips/ralink/rt305x.c' was including
'<asm/mach-ralink/rt305x.h>' and before this include in
the same C file '<asm/mach-ralink/ralink_regs.h>' is being included. I
don't know why this is being including in C file and not
in the header itself. So to be coherent we should add the include
<asm/mach-ralink/ralink_regs.h> in the new
'drivers/pinctrl/ralink/pinctrl-rt305x.c'
as follows:

diff --git a/drivers/pinctrl/ralink/pinctrl-rt305x.c
b/drivers/pinctrl/ralink/pinctrl-rt305x.c
index 699fe18e7000..5d8fa156c003 100644
--- a/drivers/pinctrl/ralink/pinctrl-rt305x.c
+++ b/drivers/pinctrl/ralink/pinctrl-rt305x.c
@@ -1,5 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0-only

+#include <asm/mach-ralink/ralink_regs.h>
 #include <asm/mach-ralink/rt305x.h>
 #include <linux/module.h>
 #include <linux/platform_device.h>

This should fix the problem. Maybe there is a better solution...
Linus, what do you think?

Best regards,
    Sergio Paracuellos

>
>
> vim +/RT305X_SOC_RT3050 +18 arch/mips/include/asm/mach-ralink/rt305x.h
>
> 2809b31770d7fd John Crispin 2013-01-20  15
> 2809b31770d7fd John Crispin 2013-01-20  16  static inline int soc_is_rt3050(void)
> 2809b31770d7fd John Crispin 2013-01-20  17  {
> 418d29c8706170 John Crispin 2015-11-04 @18      return ralink_soc == RT305X_SOC_RT3050;
> 2809b31770d7fd John Crispin 2013-01-20  19  }
> 2809b31770d7fd John Crispin 2013-01-20  20
> 2809b31770d7fd John Crispin 2013-01-20  21  static inline int soc_is_rt3052(void)
> 2809b31770d7fd John Crispin 2013-01-20  22  {
> 418d29c8706170 John Crispin 2015-11-04 @23      return ralink_soc == RT305X_SOC_RT3052;
> 2809b31770d7fd John Crispin 2013-01-20  24  }
> 2809b31770d7fd John Crispin 2013-01-20  25
> 2809b31770d7fd John Crispin 2013-01-20  26  static inline int soc_is_rt305x(void)
> 2809b31770d7fd John Crispin 2013-01-20  27  {
> 2809b31770d7fd John Crispin 2013-01-20  28      return soc_is_rt3050() || soc_is_rt3052();
> 2809b31770d7fd John Crispin 2013-01-20  29  }
> 2809b31770d7fd John Crispin 2013-01-20  30
> 2809b31770d7fd John Crispin 2013-01-20  31  static inline int soc_is_rt3350(void)
> 2809b31770d7fd John Crispin 2013-01-20  32  {
> 418d29c8706170 John Crispin 2015-11-04 @33      return ralink_soc == RT305X_SOC_RT3350;
> 2809b31770d7fd John Crispin 2013-01-20  34  }
> 2809b31770d7fd John Crispin 2013-01-20  35
> 2809b31770d7fd John Crispin 2013-01-20  36  static inline int soc_is_rt3352(void)
> 2809b31770d7fd John Crispin 2013-01-20  37  {
> 418d29c8706170 John Crispin 2015-11-04 @38      return ralink_soc == RT305X_SOC_RT3352;
> 2809b31770d7fd John Crispin 2013-01-20  39  }
> 2809b31770d7fd John Crispin 2013-01-20  40
> 2809b31770d7fd John Crispin 2013-01-20  41  static inline int soc_is_rt5350(void)
> 2809b31770d7fd John Crispin 2013-01-20  42  {
> 418d29c8706170 John Crispin 2015-11-04 @43      return ralink_soc == RT305X_SOC_RT5350;
> 2809b31770d7fd John Crispin 2013-01-20  44  }
> 2809b31770d7fd John Crispin 2013-01-20  45
>
> :::::: The code at line 18 was first introduced by commit
> :::::: 418d29c8706170c6801998e48341181b274a47a8 MIPS: ralink: Unify SoC id handling
>
> :::::: TO: John Crispin <blogic@openwrt.org>
> :::::: CC: Ralf Baechle <ralf@linux-mips.org>
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [linux-next:master 6546/14055] arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared
@ 2021-06-29 14:16   ` Sergio Paracuellos
  0 siblings, 0 replies; 6+ messages in thread
From: Sergio Paracuellos @ 2021-06-29 14:16 UTC (permalink / raw)
  To: kbuild-all

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

Hi,

On Tue, Jun 29, 2021 at 3:10 PM kernel test robot <lkp@intel.com> wrote:
>
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head:   73748627df83aab934c81332ca83a44ab8c7b3e3
> commit: 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384 [6546/14055] pinctrl: ralink: move RT305X SoC pinmux config into a new 'pinctrl-rt305x.c' file
> config: mips-randconfig-c003-20210629 (attached as .config)
> compiler: mipsel-linux-gcc (GCC) 9.3.0
> 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
>         # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
>         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 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
>         # save the attached .config to linux build tree
>         mkdir build_dir
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross O=build_dir ARCH=mips SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
>    In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared (first use in this function)
>       18 |  return ralink_soc == RT305X_SOC_RT3050;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:18:23: note: each undeclared identifier is reported only once for each function it appears in
> >> arch/mips/include/asm/mach-ralink/rt305x.h:18:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       18 |  return ralink_soc == RT305X_SOC_RT3050;
>          |  ^~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:23:23: error: 'RT305X_SOC_RT3052' undeclared (first use in this function)
>       23 |  return ralink_soc == RT305X_SOC_RT3052;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:23:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       23 |  return ralink_soc == RT305X_SOC_RT3052;
>          |  ^~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:33:23: error: 'RT305X_SOC_RT3350' undeclared (first use in this function)
>       33 |  return ralink_soc == RT305X_SOC_RT3350;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:33:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       33 |  return ralink_soc == RT305X_SOC_RT3350;
>          |  ^~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:38:23: error: 'RT305X_SOC_RT3352' undeclared (first use in this function)
>       38 |  return ralink_soc == RT305X_SOC_RT3352;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:38:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       38 |  return ralink_soc == RT305X_SOC_RT3352;
>          |  ^~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt5350':
> >> arch/mips/include/asm/mach-ralink/rt305x.h:43:23: error: 'RT305X_SOC_RT5350' undeclared (first use in this function)
>       43 |  return ralink_soc == RT305X_SOC_RT5350;
>          |                       ^~~~~~~~~~~~~~~~~
>    arch/mips/include/asm/mach-ralink/rt305x.h:43:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
>       43 |  return ralink_soc == RT305X_SOC_RT5350;
>          |  ^~~~~~
>    In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
>    arch/mips/include/asm/mach-ralink/rt305x.h:44:1: error: control reaches end of non-void function [-Werror=return-type]
>       44 | }
>          | ^
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
>    arch/mips/include/asm/mach-ralink/rt305x.h:19:1: error: control reaches end of non-void function [-Werror=return-type]
>       19 | }
>          | ^
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
>    arch/mips/include/asm/mach-ralink/rt305x.h:24:1: error: control reaches end of non-void function [-Werror=return-type]
>       24 | }
>          | ^
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
>    arch/mips/include/asm/mach-ralink/rt305x.h:34:1: error: control reaches end of non-void function [-Werror=return-type]
>       34 | }
>          | ^
>    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
>    arch/mips/include/asm/mach-ralink/rt305x.h:39:1: error: control reaches end of non-void function [-Werror=return-type]
>       39 | }
>          | ^
>    cc1: some warnings being treated as errors

There is a missing include after creating the file
'drivers/pinctrl/ralink/pinctrl-rt305x.c': #include
<asm/mach-ralink/ralink_regs.h>

Previous to this move, 'arch/mips/ralink/rt305x.c' was including
'<asm/mach-ralink/rt305x.h>' and before this include in
the same C file '<asm/mach-ralink/ralink_regs.h>' is being included. I
don't know why this is being including in C file and not
in the header itself. So to be coherent we should add the include
<asm/mach-ralink/ralink_regs.h> in the new
'drivers/pinctrl/ralink/pinctrl-rt305x.c'
as follows:

diff --git a/drivers/pinctrl/ralink/pinctrl-rt305x.c
b/drivers/pinctrl/ralink/pinctrl-rt305x.c
index 699fe18e7000..5d8fa156c003 100644
--- a/drivers/pinctrl/ralink/pinctrl-rt305x.c
+++ b/drivers/pinctrl/ralink/pinctrl-rt305x.c
@@ -1,5 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0-only

+#include <asm/mach-ralink/ralink_regs.h>
 #include <asm/mach-ralink/rt305x.h>
 #include <linux/module.h>
 #include <linux/platform_device.h>

This should fix the problem. Maybe there is a better solution...
Linus, what do you think?

Best regards,
    Sergio Paracuellos

>
>
> vim +/RT305X_SOC_RT3050 +18 arch/mips/include/asm/mach-ralink/rt305x.h
>
> 2809b31770d7fd John Crispin 2013-01-20  15
> 2809b31770d7fd John Crispin 2013-01-20  16  static inline int soc_is_rt3050(void)
> 2809b31770d7fd John Crispin 2013-01-20  17  {
> 418d29c8706170 John Crispin 2015-11-04 @18      return ralink_soc == RT305X_SOC_RT3050;
> 2809b31770d7fd John Crispin 2013-01-20  19  }
> 2809b31770d7fd John Crispin 2013-01-20  20
> 2809b31770d7fd John Crispin 2013-01-20  21  static inline int soc_is_rt3052(void)
> 2809b31770d7fd John Crispin 2013-01-20  22  {
> 418d29c8706170 John Crispin 2015-11-04 @23      return ralink_soc == RT305X_SOC_RT3052;
> 2809b31770d7fd John Crispin 2013-01-20  24  }
> 2809b31770d7fd John Crispin 2013-01-20  25
> 2809b31770d7fd John Crispin 2013-01-20  26  static inline int soc_is_rt305x(void)
> 2809b31770d7fd John Crispin 2013-01-20  27  {
> 2809b31770d7fd John Crispin 2013-01-20  28      return soc_is_rt3050() || soc_is_rt3052();
> 2809b31770d7fd John Crispin 2013-01-20  29  }
> 2809b31770d7fd John Crispin 2013-01-20  30
> 2809b31770d7fd John Crispin 2013-01-20  31  static inline int soc_is_rt3350(void)
> 2809b31770d7fd John Crispin 2013-01-20  32  {
> 418d29c8706170 John Crispin 2015-11-04 @33      return ralink_soc == RT305X_SOC_RT3350;
> 2809b31770d7fd John Crispin 2013-01-20  34  }
> 2809b31770d7fd John Crispin 2013-01-20  35
> 2809b31770d7fd John Crispin 2013-01-20  36  static inline int soc_is_rt3352(void)
> 2809b31770d7fd John Crispin 2013-01-20  37  {
> 418d29c8706170 John Crispin 2015-11-04 @38      return ralink_soc == RT305X_SOC_RT3352;
> 2809b31770d7fd John Crispin 2013-01-20  39  }
> 2809b31770d7fd John Crispin 2013-01-20  40
> 2809b31770d7fd John Crispin 2013-01-20  41  static inline int soc_is_rt5350(void)
> 2809b31770d7fd John Crispin 2013-01-20  42  {
> 418d29c8706170 John Crispin 2015-11-04 @43      return ralink_soc == RT305X_SOC_RT5350;
> 2809b31770d7fd John Crispin 2013-01-20  44  }
> 2809b31770d7fd John Crispin 2013-01-20  45
>
> :::::: The code at line 18 was first introduced by commit
> :::::: 418d29c8706170c6801998e48341181b274a47a8 MIPS: ralink: Unify SoC id handling
>
> :::::: TO: John Crispin <blogic@openwrt.org>
> :::::: CC: Ralf Baechle <ralf@linux-mips.org>
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [linux-next:master 6546/14055] arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared
  2021-06-29 14:16   ` Sergio Paracuellos
@ 2021-06-29 14:57     ` Sergio Paracuellos
  -1 siblings, 0 replies; 6+ messages in thread
From: Sergio Paracuellos @ 2021-06-29 14:57 UTC (permalink / raw)
  To: kernel test robot; +Cc: kbuild-all, Linux Memory Management List, Linus Walleij

On Tue, Jun 29, 2021 at 4:16 PM Sergio Paracuellos
<sergio.paracuellos@gmail.com> wrote:
>
> Hi,
>
> On Tue, Jun 29, 2021 at 3:10 PM kernel test robot <lkp@intel.com> wrote:
> >
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > head:   73748627df83aab934c81332ca83a44ab8c7b3e3
> > commit: 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384 [6546/14055] pinctrl: ralink: move RT305X SoC pinmux config into a new 'pinctrl-rt305x.c' file
> > config: mips-randconfig-c003-20210629 (attached as .config)
> > compiler: mipsel-linux-gcc (GCC) 9.3.0
> > 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
> >         # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
> >         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 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
> >         # save the attached .config to linux build tree
> >         mkdir build_dir
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross O=build_dir ARCH=mips SHELL=/bin/bash
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@intel.com>
> >
> > All errors (new ones prefixed by >>):
> >
> >    In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared (first use in this function)
> >       18 |  return ralink_soc == RT305X_SOC_RT3050;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:18:23: note: each undeclared identifier is reported only once for each function it appears in
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:18:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       18 |  return ralink_soc == RT305X_SOC_RT3050;
> >          |  ^~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:23:23: error: 'RT305X_SOC_RT3052' undeclared (first use in this function)
> >       23 |  return ralink_soc == RT305X_SOC_RT3052;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:23:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       23 |  return ralink_soc == RT305X_SOC_RT3052;
> >          |  ^~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:33:23: error: 'RT305X_SOC_RT3350' undeclared (first use in this function)
> >       33 |  return ralink_soc == RT305X_SOC_RT3350;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:33:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       33 |  return ralink_soc == RT305X_SOC_RT3350;
> >          |  ^~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:38:23: error: 'RT305X_SOC_RT3352' undeclared (first use in this function)
> >       38 |  return ralink_soc == RT305X_SOC_RT3352;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:38:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       38 |  return ralink_soc == RT305X_SOC_RT3352;
> >          |  ^~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt5350':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:43:23: error: 'RT305X_SOC_RT5350' undeclared (first use in this function)
> >       43 |  return ralink_soc == RT305X_SOC_RT5350;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:43:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       43 |  return ralink_soc == RT305X_SOC_RT5350;
> >          |  ^~~~~~
> >    In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
> >    arch/mips/include/asm/mach-ralink/rt305x.h:44:1: error: control reaches end of non-void function [-Werror=return-type]
> >       44 | }
> >          | ^
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
> >    arch/mips/include/asm/mach-ralink/rt305x.h:19:1: error: control reaches end of non-void function [-Werror=return-type]
> >       19 | }
> >          | ^
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
> >    arch/mips/include/asm/mach-ralink/rt305x.h:24:1: error: control reaches end of non-void function [-Werror=return-type]
> >       24 | }
> >          | ^
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
> >    arch/mips/include/asm/mach-ralink/rt305x.h:34:1: error: control reaches end of non-void function [-Werror=return-type]
> >       34 | }
> >          | ^
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
> >    arch/mips/include/asm/mach-ralink/rt305x.h:39:1: error: control reaches end of non-void function [-Werror=return-type]
> >       39 | }
> >          | ^
> >    cc1: some warnings being treated as errors
>
> There is a missing include after creating the file
> 'drivers/pinctrl/ralink/pinctrl-rt305x.c': #include
> <asm/mach-ralink/ralink_regs.h>
>
> Previous to this move, 'arch/mips/ralink/rt305x.c' was including
> '<asm/mach-ralink/rt305x.h>' and before this include in
> the same C file '<asm/mach-ralink/ralink_regs.h>' is being included. I
> don't know why this is being including in C file and not
> in the header itself. So to be coherent we should add the include
> <asm/mach-ralink/ralink_regs.h> in the new
> 'drivers/pinctrl/ralink/pinctrl-rt305x.c'
> as follows:
>
> diff --git a/drivers/pinctrl/ralink/pinctrl-rt305x.c
> b/drivers/pinctrl/ralink/pinctrl-rt305x.c
> index 699fe18e7000..5d8fa156c003 100644
> --- a/drivers/pinctrl/ralink/pinctrl-rt305x.c
> +++ b/drivers/pinctrl/ralink/pinctrl-rt305x.c
> @@ -1,5 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0-only
>
> +#include <asm/mach-ralink/ralink_regs.h>
>  #include <asm/mach-ralink/rt305x.h>
>  #include <linux/module.h>
>  #include <linux/platform_device.h>
>
> This should fix the problem. Maybe there is a better solution...
> Linus, what do you think?

I have already sent this patch: https://lkml.org/lkml/2021/6/29/414

Linus, let me know if this is ok for you.

Thanks,
    Sergio Paracuellos

>
> Best regards,
>     Sergio Paracuellos
>
> >
> >
> > vim +/RT305X_SOC_RT3050 +18 arch/mips/include/asm/mach-ralink/rt305x.h
> >
> > 2809b31770d7fd John Crispin 2013-01-20  15
> > 2809b31770d7fd John Crispin 2013-01-20  16  static inline int soc_is_rt3050(void)
> > 2809b31770d7fd John Crispin 2013-01-20  17  {
> > 418d29c8706170 John Crispin 2015-11-04 @18      return ralink_soc == RT305X_SOC_RT3050;
> > 2809b31770d7fd John Crispin 2013-01-20  19  }
> > 2809b31770d7fd John Crispin 2013-01-20  20
> > 2809b31770d7fd John Crispin 2013-01-20  21  static inline int soc_is_rt3052(void)
> > 2809b31770d7fd John Crispin 2013-01-20  22  {
> > 418d29c8706170 John Crispin 2015-11-04 @23      return ralink_soc == RT305X_SOC_RT3052;
> > 2809b31770d7fd John Crispin 2013-01-20  24  }
> > 2809b31770d7fd John Crispin 2013-01-20  25
> > 2809b31770d7fd John Crispin 2013-01-20  26  static inline int soc_is_rt305x(void)
> > 2809b31770d7fd John Crispin 2013-01-20  27  {
> > 2809b31770d7fd John Crispin 2013-01-20  28      return soc_is_rt3050() || soc_is_rt3052();
> > 2809b31770d7fd John Crispin 2013-01-20  29  }
> > 2809b31770d7fd John Crispin 2013-01-20  30
> > 2809b31770d7fd John Crispin 2013-01-20  31  static inline int soc_is_rt3350(void)
> > 2809b31770d7fd John Crispin 2013-01-20  32  {
> > 418d29c8706170 John Crispin 2015-11-04 @33      return ralink_soc == RT305X_SOC_RT3350;
> > 2809b31770d7fd John Crispin 2013-01-20  34  }
> > 2809b31770d7fd John Crispin 2013-01-20  35
> > 2809b31770d7fd John Crispin 2013-01-20  36  static inline int soc_is_rt3352(void)
> > 2809b31770d7fd John Crispin 2013-01-20  37  {
> > 418d29c8706170 John Crispin 2015-11-04 @38      return ralink_soc == RT305X_SOC_RT3352;
> > 2809b31770d7fd John Crispin 2013-01-20  39  }
> > 2809b31770d7fd John Crispin 2013-01-20  40
> > 2809b31770d7fd John Crispin 2013-01-20  41  static inline int soc_is_rt5350(void)
> > 2809b31770d7fd John Crispin 2013-01-20  42  {
> > 418d29c8706170 John Crispin 2015-11-04 @43      return ralink_soc == RT305X_SOC_RT5350;
> > 2809b31770d7fd John Crispin 2013-01-20  44  }
> > 2809b31770d7fd John Crispin 2013-01-20  45
> >
> > :::::: The code at line 18 was first introduced by commit
> > :::::: 418d29c8706170c6801998e48341181b274a47a8 MIPS: ralink: Unify SoC id handling
> >
> > :::::: TO: John Crispin <blogic@openwrt.org>
> > :::::: CC: Ralf Baechle <ralf@linux-mips.org>
> >
> > ---
> > 0-DAY CI Kernel Test Service, Intel Corporation
> > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [linux-next:master 6546/14055] arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared
@ 2021-06-29 14:57     ` Sergio Paracuellos
  0 siblings, 0 replies; 6+ messages in thread
From: Sergio Paracuellos @ 2021-06-29 14:57 UTC (permalink / raw)
  To: kbuild-all

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

On Tue, Jun 29, 2021 at 4:16 PM Sergio Paracuellos
<sergio.paracuellos@gmail.com> wrote:
>
> Hi,
>
> On Tue, Jun 29, 2021 at 3:10 PM kernel test robot <lkp@intel.com> wrote:
> >
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > head:   73748627df83aab934c81332ca83a44ab8c7b3e3
> > commit: 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384 [6546/14055] pinctrl: ralink: move RT305X SoC pinmux config into a new 'pinctrl-rt305x.c' file
> > config: mips-randconfig-c003-20210629 (attached as .config)
> > compiler: mipsel-linux-gcc (GCC) 9.3.0
> > 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
> >         # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
> >         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 3a1b0ca5a83b69f95ab9641ae0e77526a6fee384
> >         # save the attached .config to linux build tree
> >         mkdir build_dir
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross O=build_dir ARCH=mips SHELL=/bin/bash
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@intel.com>
> >
> > All errors (new ones prefixed by >>):
> >
> >    In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared (first use in this function)
> >       18 |  return ralink_soc == RT305X_SOC_RT3050;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:18:23: note: each undeclared identifier is reported only once for each function it appears in
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:18:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       18 |  return ralink_soc == RT305X_SOC_RT3050;
> >          |  ^~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:23:23: error: 'RT305X_SOC_RT3052' undeclared (first use in this function)
> >       23 |  return ralink_soc == RT305X_SOC_RT3052;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:23:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       23 |  return ralink_soc == RT305X_SOC_RT3052;
> >          |  ^~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:33:23: error: 'RT305X_SOC_RT3350' undeclared (first use in this function)
> >       33 |  return ralink_soc == RT305X_SOC_RT3350;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:33:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       33 |  return ralink_soc == RT305X_SOC_RT3350;
> >          |  ^~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:38:23: error: 'RT305X_SOC_RT3352' undeclared (first use in this function)
> >       38 |  return ralink_soc == RT305X_SOC_RT3352;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:38:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       38 |  return ralink_soc == RT305X_SOC_RT3352;
> >          |  ^~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt5350':
> > >> arch/mips/include/asm/mach-ralink/rt305x.h:43:23: error: 'RT305X_SOC_RT5350' undeclared (first use in this function)
> >       43 |  return ralink_soc == RT305X_SOC_RT5350;
> >          |                       ^~~~~~~~~~~~~~~~~
> >    arch/mips/include/asm/mach-ralink/rt305x.h:43:2: error: 'ralink_soc' has an incomplete type 'enum ralink_soc_type'
> >       43 |  return ralink_soc == RT305X_SOC_RT5350;
> >          |  ^~~~~~
> >    In file included from drivers/pinctrl/ralink/pinctrl-rt305x.c:3:
> >    arch/mips/include/asm/mach-ralink/rt305x.h:44:1: error: control reaches end of non-void function [-Werror=return-type]
> >       44 | }
> >          | ^
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3050':
> >    arch/mips/include/asm/mach-ralink/rt305x.h:19:1: error: control reaches end of non-void function [-Werror=return-type]
> >       19 | }
> >          | ^
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3052':
> >    arch/mips/include/asm/mach-ralink/rt305x.h:24:1: error: control reaches end of non-void function [-Werror=return-type]
> >       24 | }
> >          | ^
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3350':
> >    arch/mips/include/asm/mach-ralink/rt305x.h:34:1: error: control reaches end of non-void function [-Werror=return-type]
> >       34 | }
> >          | ^
> >    arch/mips/include/asm/mach-ralink/rt305x.h: In function 'soc_is_rt3352':
> >    arch/mips/include/asm/mach-ralink/rt305x.h:39:1: error: control reaches end of non-void function [-Werror=return-type]
> >       39 | }
> >          | ^
> >    cc1: some warnings being treated as errors
>
> There is a missing include after creating the file
> 'drivers/pinctrl/ralink/pinctrl-rt305x.c': #include
> <asm/mach-ralink/ralink_regs.h>
>
> Previous to this move, 'arch/mips/ralink/rt305x.c' was including
> '<asm/mach-ralink/rt305x.h>' and before this include in
> the same C file '<asm/mach-ralink/ralink_regs.h>' is being included. I
> don't know why this is being including in C file and not
> in the header itself. So to be coherent we should add the include
> <asm/mach-ralink/ralink_regs.h> in the new
> 'drivers/pinctrl/ralink/pinctrl-rt305x.c'
> as follows:
>
> diff --git a/drivers/pinctrl/ralink/pinctrl-rt305x.c
> b/drivers/pinctrl/ralink/pinctrl-rt305x.c
> index 699fe18e7000..5d8fa156c003 100644
> --- a/drivers/pinctrl/ralink/pinctrl-rt305x.c
> +++ b/drivers/pinctrl/ralink/pinctrl-rt305x.c
> @@ -1,5 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0-only
>
> +#include <asm/mach-ralink/ralink_regs.h>
>  #include <asm/mach-ralink/rt305x.h>
>  #include <linux/module.h>
>  #include <linux/platform_device.h>
>
> This should fix the problem. Maybe there is a better solution...
> Linus, what do you think?

I have already sent this patch: https://lkml.org/lkml/2021/6/29/414

Linus, let me know if this is ok for you.

Thanks,
    Sergio Paracuellos

>
> Best regards,
>     Sergio Paracuellos
>
> >
> >
> > vim +/RT305X_SOC_RT3050 +18 arch/mips/include/asm/mach-ralink/rt305x.h
> >
> > 2809b31770d7fd John Crispin 2013-01-20  15
> > 2809b31770d7fd John Crispin 2013-01-20  16  static inline int soc_is_rt3050(void)
> > 2809b31770d7fd John Crispin 2013-01-20  17  {
> > 418d29c8706170 John Crispin 2015-11-04 @18      return ralink_soc == RT305X_SOC_RT3050;
> > 2809b31770d7fd John Crispin 2013-01-20  19  }
> > 2809b31770d7fd John Crispin 2013-01-20  20
> > 2809b31770d7fd John Crispin 2013-01-20  21  static inline int soc_is_rt3052(void)
> > 2809b31770d7fd John Crispin 2013-01-20  22  {
> > 418d29c8706170 John Crispin 2015-11-04 @23      return ralink_soc == RT305X_SOC_RT3052;
> > 2809b31770d7fd John Crispin 2013-01-20  24  }
> > 2809b31770d7fd John Crispin 2013-01-20  25
> > 2809b31770d7fd John Crispin 2013-01-20  26  static inline int soc_is_rt305x(void)
> > 2809b31770d7fd John Crispin 2013-01-20  27  {
> > 2809b31770d7fd John Crispin 2013-01-20  28      return soc_is_rt3050() || soc_is_rt3052();
> > 2809b31770d7fd John Crispin 2013-01-20  29  }
> > 2809b31770d7fd John Crispin 2013-01-20  30
> > 2809b31770d7fd John Crispin 2013-01-20  31  static inline int soc_is_rt3350(void)
> > 2809b31770d7fd John Crispin 2013-01-20  32  {
> > 418d29c8706170 John Crispin 2015-11-04 @33      return ralink_soc == RT305X_SOC_RT3350;
> > 2809b31770d7fd John Crispin 2013-01-20  34  }
> > 2809b31770d7fd John Crispin 2013-01-20  35
> > 2809b31770d7fd John Crispin 2013-01-20  36  static inline int soc_is_rt3352(void)
> > 2809b31770d7fd John Crispin 2013-01-20  37  {
> > 418d29c8706170 John Crispin 2015-11-04 @38      return ralink_soc == RT305X_SOC_RT3352;
> > 2809b31770d7fd John Crispin 2013-01-20  39  }
> > 2809b31770d7fd John Crispin 2013-01-20  40
> > 2809b31770d7fd John Crispin 2013-01-20  41  static inline int soc_is_rt5350(void)
> > 2809b31770d7fd John Crispin 2013-01-20  42  {
> > 418d29c8706170 John Crispin 2015-11-04 @43      return ralink_soc == RT305X_SOC_RT5350;
> > 2809b31770d7fd John Crispin 2013-01-20  44  }
> > 2809b31770d7fd John Crispin 2013-01-20  45
> >
> > :::::: The code at line 18 was first introduced by commit
> > :::::: 418d29c8706170c6801998e48341181b274a47a8 MIPS: ralink: Unify SoC id handling
> >
> > :::::: TO: John Crispin <blogic@openwrt.org>
> > :::::: CC: Ralf Baechle <ralf@linux-mips.org>
> >
> > ---
> > 0-DAY CI Kernel Test Service, Intel Corporation
> > https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2021-06-29 14:57 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-29 13:10 [linux-next:master 6546/14055] arch/mips/include/asm/mach-ralink/rt305x.h:18:23: error: 'RT305X_SOC_RT3050' undeclared kernel test robot
2021-06-29 13:10 ` kernel test robot
2021-06-29 14:16 ` Sergio Paracuellos
2021-06-29 14:16   ` Sergio Paracuellos
2021-06-29 14:57   ` Sergio Paracuellos
2021-06-29 14:57     ` Sergio Paracuellos

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.