[linux-stable-rc:linux-4.4.y 9235/9999] arch/powerpc/platforms/83xx/mpc832x_rdb.c:134:13: error: comparison between signed and unsigned integer expressions

[linux-stable-rc:linux-4.4.y 9235/9999] arch/powerpc/platforms/83xx/mpc832x_rdb.c:134:13: error: comparison between signed and unsigned integer expressions

[kernel test robot]

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

Hi Dirk,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.4.y
head:   95a3867e897abd7811196123f81a119a75aba863
commit: ce513359d8507123e63f34b56e67ad558074be22 [9235/9999] scripts/dtc: Remove redundant YYLOC global declaration
config: powerpc-mpc83xx_defconfig (attached as .config)
compiler: powerpc-linux-gcc (GCC) 7.5.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/stable/linux-stable-rc.git/commit/?id=ce513359d8507123e63f34b56e67ad558074be22
        git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
        git fetch --no-tags linux-stable-rc linux-4.4.y
        git checkout ce513359d8507123e63f34b56e67ad558074be22
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-7.5.0 make.cross ARCH=powerpc 

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 include/linux/scatterlist.h:7:0,
                    from include/linux/dmapool.h:14,
                    from include/linux/pci.h:1228,
                    from arch/powerpc/platforms/83xx/mpc832x_rdb.c:17:
   include/linux/mm.h: In function 'is_vmalloc_addr':
   include/linux/mm.h:386:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
     return addr >= VMALLOC_START && addr < VMALLOC_END;
                 ^~
   arch/powerpc/platforms/83xx/mpc832x_rdb.c: In function 'fsl_spi_init':
>> arch/powerpc/platforms/83xx/mpc832x_rdb.c:134:13: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
     if (sysclk == -1) {
                ^~
   arch/powerpc/platforms/83xx/mpc832x_rdb.c:136:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
      if (sysclk == -1)
                 ^~
   cc1: all warnings being treated as errors


vim +134 arch/powerpc/platforms/83xx/mpc832x_rdb.c

e2801806de1c9c Anton Vorontsov 2009-03-31  123  
e2801806de1c9c Anton Vorontsov 2009-03-31  124  static int __init fsl_spi_init(struct spi_board_info *board_infos,
e2801806de1c9c Anton Vorontsov 2009-03-31  125  			       unsigned int num_board_infos,
e2801806de1c9c Anton Vorontsov 2009-03-31  126  			       void (*cs_control)(struct spi_device *spi,
e2801806de1c9c Anton Vorontsov 2009-03-31  127  						  bool on))
e2801806de1c9c Anton Vorontsov 2009-03-31  128  {
e2801806de1c9c Anton Vorontsov 2009-03-31  129  	u32 sysclk = -1;
e2801806de1c9c Anton Vorontsov 2009-03-31  130  	int ret;
e2801806de1c9c Anton Vorontsov 2009-03-31  131  
e2801806de1c9c Anton Vorontsov 2009-03-31  132  	/* SPI controller is either clocked from QE or SoC clock */
e2801806de1c9c Anton Vorontsov 2009-03-31  133  	sysclk = get_brgfreq();
e2801806de1c9c Anton Vorontsov 2009-03-31 @134  	if (sysclk == -1) {
e2801806de1c9c Anton Vorontsov 2009-03-31  135  		sysclk = fsl_get_sys_freq();
e2801806de1c9c Anton Vorontsov 2009-03-31  136  		if (sysclk == -1)
e2801806de1c9c Anton Vorontsov 2009-03-31  137  			return -ENODEV;
e2801806de1c9c Anton Vorontsov 2009-03-31  138  	}
e2801806de1c9c Anton Vorontsov 2009-03-31  139  
e2801806de1c9c Anton Vorontsov 2009-03-31  140  	ret = of_fsl_spi_probe(NULL, "fsl,spi", sysclk, board_infos,
e2801806de1c9c Anton Vorontsov 2009-03-31  141  			       num_board_infos, cs_control);
e2801806de1c9c Anton Vorontsov 2009-03-31  142  	if (!ret)
e2801806de1c9c Anton Vorontsov 2009-03-31  143  		of_fsl_spi_probe("spi", "fsl_spi", sysclk, board_infos,
e2801806de1c9c Anton Vorontsov 2009-03-31  144  				 num_board_infos, cs_control);
e2801806de1c9c Anton Vorontsov 2009-03-31  145  
e2801806de1c9c Anton Vorontsov 2009-03-31  146  	return spi_register_board_info(board_infos, num_board_infos);
e2801806de1c9c Anton Vorontsov 2009-03-31  147  }
e2801806de1c9c Anton Vorontsov 2009-03-31  148  

:::::: The code at line 134 was first introduced by commit
:::::: e2801806de1c9c1d03b7d1bfcb2e01dd4d389e80 powerpc/fsl_soc: isolate legacy fsl_spi support to mpc832x_rdb boards

:::::: TO: Anton Vorontsov <avorontsov@ru.mvista.com>
:::::: CC: Linus Torvalds <torvalds@linux-foundation.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: 16131 bytes --]