Re: [PATCH 1/5] i3c: add slave mode support

kernel test robot
2023-11-03T18:52:42Z

Hi Frank,

kernel test robot noticed the following build warnings:

[auto build test WARNING on robh/for-next]
[also build test WARNING on linus/master v6.6-rc6 next-20231020]
[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#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/Frank-Li/i3c-add-slave-mode-support/20231019-055940
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
patch link: https://lore.kernel.org/r/20231018215809.3477437-2-Frank.Li%40nxp.com
patch subject: [PATCH 1/5] i3c: add slave mode support
config: microblaze-allyesconfig (https://download.01.org/0day-ci/archive/20231021/202310211527.TeVTMxA7-lkp@intel.com/config)
compiler: microblaze-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231021/202310211527.TeVTMxA7-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <yujie.liu@intel.com>
| Closes: https://lore.kernel.org/r/202310211527.TeVTMxA7-lkp@intel.com/

All warnings (new ones prefixed by >>):

In file included from drivers/i3c/slave.c:11:
include/linux/i3c/slave.h: In function 'i3c_slave_ctrl_alloc_request':
include/linux/i3c/slave.h:272:23: error: implicit declaration of function 'kzalloc' [-Werror=implicit-function-declaration]
272 | req = kzalloc(sizeof(*req), gfp_flags);
| ^~~~~~~
>> include/linux/i3c/slave.h:272:21: warning: assignment to 'struct i3c_request *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
272 | req = kzalloc(sizeof(*req), gfp_flags);
| ^
include/linux/i3c/slave.h: In function 'i3c_slave_ctrl_free_request':
include/linux/i3c/slave.h:298:17: error: implicit declaration of function 'kfree'; did you mean 'kvfree'? [-Werror=implicit-function-declaration]
298 | kfree(req);
| ^~~~~
| kvfree
In file included from drivers/i3c/slave.c:13:
include/linux/slab.h: At top level:
>> include/linux/slab.h:227:6: warning: conflicting types for 'kfree'; have 'void(const void *)'
227 | void kfree(const void *objp);
| ^~~~~
include/linux/i3c/slave.h:298:17: note: previous implicit declaration of 'kfree' with type 'void(const void *)'
298 | kfree(req);
| ^~~~~
include/linux/slab.h:718:37: error: conflicting types for 'kzalloc'; have 'void *(size_t, gfp_t)' {aka 'void *(unsigned int, unsigned int)'}
718 | static inline __alloc_size(1) void *kzalloc(size_t size, gfp_t flags)
| ^~~~~~~
include/linux/i3c/slave.h:272:23: note: previous implicit declaration of 'kzalloc' with type 'int()'
272 | req = kzalloc(sizeof(*req), gfp_flags);
| ^~~~~~~
cc1: some warnings being treated as errors

vim +272 include/linux/i3c/slave.h

a63b2858bd837d Frank Li 2023-10-18 256
a63b2858bd837d Frank Li 2023-10-18 257 /**
a63b2858bd837d Frank Li 2023-10-18 258 * i3c_slave_ctrl_alloc_request() - Alloc an I3C transfer
a63b2858bd837d Frank Li 2023-10-18 259 * @ctrl: I3C slave controller device
a63b2858bd837d Frank Li 2023-10-18 260 * gfp_flags: additional gfp flags used when allocating the buffers
a63b2858bd837d Frank Li 2023-10-18 261 *
a63b2858bd837d Frank Li 2023-10-18 262 * Returns: Zero for success, or an error code in case of failure
a63b2858bd837d Frank Li 2023-10-18 263 */
a63b2858bd837d Frank Li 2023-10-18 264 static inline struct i3c_request *
a63b2858bd837d Frank Li 2023-10-18 265 i3c_slave_ctrl_alloc_request(struct i3c_slave_ctrl *ctrl, gfp_t gfp_flags)
a63b2858bd837d Frank Li 2023-10-18 266 {
a63b2858bd837d Frank Li 2023-10-18 267 struct i3c_request *req = NULL;
a63b2858bd837d Frank Li 2023-10-18 268
a63b2858bd837d Frank Li 2023-10-18 269 if (ctrl && ctrl->ops && ctrl->ops->alloc_request)
a63b2858bd837d Frank Li 2023-10-18 270 req = ctrl->ops->alloc_request(ctrl, gfp_flags);
a63b2858bd837d Frank Li 2023-10-18 271 else
a63b2858bd837d Frank Li 2023-10-18 @272 req = kzalloc(sizeof(*req), gfp_flags);
a63b2858bd837d Frank Li 2023-10-18 273
a63b2858bd837d Frank Li 2023-10-18 274 if (req)
a63b2858bd837d Frank Li 2023-10-18 275 req->ctrl = ctrl;
a63b2858bd837d Frank Li 2023-10-18 276
a63b2858bd837d Frank Li 2023-10-18 277 return req;
a63b2858bd837d Frank Li 2023-10-18 278 }
a63b2858bd837d Frank Li 2023-10-18 279

--
0-DAY CI Kernel Test Service