Hi "Bence,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on wsa/i2c/for-next]
[also build test WARNING on v5.12-rc3 next-20210317]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    
https://github.com/0day-ci/linux/commits/Bence-Cs-k-s/Adding-i2c-cp2615-i2c-support-for-Silicon-Labs-CP2615-Digital-Audio-Bridge/20210317-181539
base:   https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git 
i2c/for-next
config: ia64-allmodconfig (attached as .config)
compiler: ia64-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://github.com/0day-ci/linux/commit/c8c005a08175789b1874e69abf4c6da690d5b323
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review 
Bence-Cs-k-s/Adding-i2c-cp2615-i2c-support-for-Silicon-Labs-CP2615-Digital-Audio-Bridge/20210317-181539
        git checkout c8c005a08175789b1874e69abf4c6da690d5b323
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross 
ARCH=ia64 

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

All warnings (new ones prefixed by >>):

   drivers/i2c/busses/i2c-cp2615.c:82:5: warning: no previous prototype for 
'cp2615_init_iop_msg' [-Wmissing-prototypes]
      82 | int cp2615_init_iop_msg(struct cp2615_iop_msg *ret, enum 
cp2615_iop_msg_type msg, const void *data, size_t data_len)
         |     ^~~~~~~~~~~~~~~~~~~
   drivers/i2c/busses/i2c-cp2615.c:99:5: warning: no previous prototype for 
'cp2615_init_i2c_msg' [-Wmissing-prototypes]
      99 | int cp2615_init_i2c_msg(struct cp2615_iop_msg *ret, const struct 
cp2615_i2c_transfer *data)
         |     ^~~~~~~~~~~~~~~~~~~
   drivers/i2c/busses/i2c-cp2615.c:105:5: warning: no previous prototype for 
'cp2615_check_status' [-Wmissing-prototypes]
     105 | int cp2615_check_status(enum cp2615_i2c_status status)
         |     ^~~~~~~~~~~~~~~~~~~
   drivers/i2c/busses/i2c-cp2615.c: In function 'cp2615_i2c_probe':
>> drivers/i2c/busses/i2c-cp2615.c:247:2: warning: 'strncpy' specified bound 48 
>> equals destination size [-Wstringop-truncation]
     247 |  strncpy(adap->name, usbdev->serial, sizeof(adap->name));
         |  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Kconfig warnings: (for reference only)
   WARNING: unmet direct dependencies detected for FRAME_POINTER
   Depends on DEBUG_KERNEL && (M68K || UML || SUPERH) || 
ARCH_WANT_FRAME_POINTERS
   Selected by
   - FAULT_INJECTION_STACKTRACE_FILTER && FAULT_INJECTION_DEBUG_FS && 
STACKTRACE_SUPPORT && !X86_64 && !MIPS && !PPC && !S390 && !MICROBLAZE && !ARM 
&& !ARC && !X86


vim +/strncpy +247 drivers/i2c/busses/i2c-cp2615.c

   231  
   232  static int
   233  cp2615_i2c_probe(struct usb_interface *usbif, const struct 
usb_device_id *id)
   234  {
   235          int ret = 0;
   236          struct i2c_adapter *adap;
   237          struct usb_device *usbdev = interface_to_usbdev(usbif);
   238  
   239          ret = usb_set_interface(usbdev, IOP_IFN, IOP_ALTSETTING);
   240          if (ret)
   241                  return ret;
   242  
   243          adap = devm_kzalloc(&usbif->dev, sizeof(struct i2c_adapter), 
GFP_KERNEL);
   244          if (!adap)
   245                  return -ENOMEM;
   246  
 > 247          strncpy(adap->name, usbdev->serial, sizeof(adap->name));
   248          adap->owner = THIS_MODULE;
   249          adap->dev.parent = &usbif->dev;
   250          adap->dev.of_node = usbif->dev.of_node;
   251          adap->timeout = HZ;
   252          adap->algo = &cp2615_i2c_algo;
   253          adap->quirks = &cp2615_i2c_quirks;
   254          adap->algo_data = usbif;
   255  
   256          ret = i2c_add_adapter(adap);
   257          if (ret)
   258                  return ret;
   259  
   260          usb_set_intfdata(usbif, adap);
   261          return ret;
   262  }
   263  

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

Attachment: .config.gz
Description: application/gzip

Reply via email to