Re: [PATCH] RFC : mikroBUS driver for add-on boards

[kernel test robot <lkp@intel.com>]

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

Hi Vaishnav,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on char-misc/char-misc-testing]
[also build test WARNING on linux/master soc/for-next linus/master v5.8-rc6 next-20200724]
[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/Vaishnav-M-A/RFC-mikroBUS-driver-for-add-on-boards/20200724-201058
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git 92ca3dd4867bafbfd026b06d53737d61ded1bd27
config: alpha-allyesconfig (attached as .config)
compiler: alpha-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
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.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 >>):

   drivers/misc/mikrobus/mikrobus_manifest.c: In function 'mikrobus_manifest_parse':
>> drivers/misc/mikrobus/mikrobus_manifest.c:344:6: warning: variable 'dev_count' set but not used [-Wunused-but-set-variable]
     344 |  int dev_count;
         |      ^~~~~~~~~

vim +/dev_count +344 drivers/misc/mikrobus/mikrobus_manifest.c

   336	
   337	bool mikrobus_manifest_parse(struct addon_board_info *info, void *data,
   338								 size_t size)
   339	{
   340		struct greybus_manifest *manifest;
   341		struct greybus_manifest_header *header;
   342		struct greybus_descriptor *desc;
   343		u16 manifest_size;
 > 344		int dev_count;
   345		int desc_size;
   346	
   347		if (WARN_ON(!list_empty(&info->manifest_descs)))
   348			return false;
   349		if (size < sizeof(*header))
   350			return false;
   351		manifest = data;
   352		header = &manifest->header;
   353		manifest_size = le16_to_cpu(header->size);
   354		if (manifest_size != size)
   355			return false;
   356		if (header->version_major > MIKROBUS_VERSION_MAJOR)
   357			return false;
   358		desc = manifest->descriptors;
   359		size -= sizeof(*header);
   360		while (size) {
   361			desc_size = identify_descriptor(info, desc, size);
   362			if (desc_size < 0) {
   363				pr_err("invalid manifest descriptor");
   364			return -EINVAL;
   365			}
   366			desc = (struct greybus_descriptor *)((char *)desc + desc_size);
   367			size -= desc_size;
   368		}
   369		mikrobus_state_get(info);
   370		dev_count = mikrobus_manifest_parse_devices(info);
   371		pr_info(" %s manifest parsed with %d device(s)\n", info->name,
   372			info->num_devices);
   373		release_manifest_descriptors(info);
   374		return true;
   375	}
   376	

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