[chrome-os:chromeos-4.4 13/14] drivers/media/cec/i2c/ch7322.c:197:16: warning: comparison between signed and unsigned integer expressions

[chrome-os:chromeos-4.4 13/14] drivers/media/cec/i2c/ch7322.c:197:16: warning: comparison between signed and unsigned integer expressions

[kernel test robot]

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

tree:   https://chromium.googlesource.com/chromiumos/third_party/kernel chromeos-4.4
head:   7254c60ea46ececa5041496f7907d0fcc582c1ae
commit: 7c7d3263ef8b22d714decdbbaf463c66ce09c731 [13/14] BACKPORT: FROMGIT: media: cec: i2c: ch7322: Add ch7322 CEC controller driver
config: alpha-allyesconfig (attached as .config)
compiler: alpha-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
        git checkout 7c7d3263ef8b22d714decdbbaf463c66ce09c731
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-7.5.0 make.cross ARCH=alpha 

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

All warnings (new ones prefixed by >>):

   In file included from arch/alpha/include/asm/bug.h:22:0,
                    from include/linux/bug.h:4,
                    from include/linux/thread_info.h:11,
                    from arch/alpha/include/asm/current.h:4,
                    from include/linux/mutex.h:13,
                    from include/linux/kernfs.h:13,
                    from include/linux/sysfs.h:15,
                    from include/linux/kobject.h:21,
                    from include/linux/dmi.h:5,
                    from drivers/media/cec/i2c/ch7322.c:17:
   include/linux/cpumask.h: In function 'cpumask_check':
   include/linux/cpumask.h:117:19: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     WARN_ON_ONCE(cpu >= nr_cpumask_bits);
                      ^
   include/asm-generic/bug.h:123:27: note: in definition of macro 'WARN_ON_ONCE'
     int __ret_warn_once = !!(condition);   \
                              ^~~~~~~~~
   drivers/media/cec/i2c/ch7322.c: In function 'ch7322_send_message':
>> drivers/media/cec/i2c/ch7322.c:197:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for (i = 0; i < len; i++) {
                   ^

vim +197 drivers/media/cec/i2c/ch7322.c

   163	
   164	static int ch7322_send_message(struct ch7322 *ch7322, const struct cec_msg *msg)
   165	{
   166		unsigned int val;
   167		unsigned int len = msg->len;
   168		int ret;
   169		int i;
   170	
   171		WARN_ON(!mutex_is_locked(&ch7322->mutex));
   172	
   173		if (len > CH7322_WRBUF_LEN || len < 1)
   174			return -EINVAL;
   175	
   176		ret = regmap_read(ch7322->regmap, CH7322_WRITE, &val);
   177		if (ret)
   178			return ret;
   179	
   180		/* Buffer not ready */
   181		if (!(val & CH7322_WRITE_MSENT))
   182			return -EBUSY;
   183	
   184		if (cec_msg_opcode(msg) == -1 &&
   185		    cec_msg_initiator(msg) == cec_msg_destination(msg)) {
   186			ch7322->tx_flags = CH7322_TX_FLAG_NACK | CH7322_TX_FLAG_RETRY;
   187		} else if (cec_msg_is_broadcast(msg)) {
   188			ch7322->tx_flags = CH7322_TX_FLAG_NACK;
   189		} else {
   190			ch7322->tx_flags = CH7322_TX_FLAG_RETRY;
   191		}
   192	
   193		ret = regmap_write(ch7322->regmap, CH7322_WRITE, len - 1);
   194		if (ret)
   195			return ret;
   196	
 > 197		for (i = 0; i < len; i++) {
   198			ret = regmap_write(ch7322->regmap,
   199					   CH7322_WRBUF + i, msg->msg[i]);
   200			if (ret)
   201				return ret;
   202		}
   203	
   204		return 0;
   205	}
   206	

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