From: kernel test robot <yujie.liu@intel.com>
To: Frank Li <Frank.Li@nxp.com>, <miquel.raynal@bootlin.com>,
<conor.culhane@silvaco.com>, <alexandre.belloni@bootlin.com>,
<robh+dt@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>,
<conor+dt@kernel.org>, <corbet@lwn.net>, <joe@perches.com>,
<linux-i3c@lists.infradead.org>, <devicetree@vger.kernel.org>,
<linux-kernel@vger.kernel.org>, <linux-doc@vger.kernel.org>
Cc: <oe-kbuild-all@lists.linux.dev>, <gregkh@linuxfoundation.org>,
<imx@lists.linux.dev>, <jirislaby@kernel.org>,
<linux-serial@vger.kernel.org>
Subject: Re: [PATCH 1/5] i3c: add slave mode support
Date: Mon, 23 Oct 2023 09:20:37 +0800 [thread overview]
Message-ID: <202310211527.TeVTMxA7-lkp@intel.com> (raw)
In-Reply-To: <20231018215809.3477437-2-Frank.Li@nxp.com>
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
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2023-10-23 1:25 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-18 21:58 [PATCH 0/5] I3C slave mode support Frank Li
2023-10-18 21:58 ` [PATCH 1/5] i3c: add " Frank Li
2023-10-19 7:00 ` Krzysztof Kozlowski
2023-10-19 15:02 ` Frank Li
2023-10-19 15:46 ` Greg KH
2023-10-19 17:06 ` Greg KH
2023-10-23 1:11 ` kernel test robot
2023-10-23 1:20 ` kernel test robot [this message]
2023-10-18 21:58 ` [PATCH 2/5] dt-bindings: i3c: svc: add compatible string i3c: silvaco,i3c-slave Frank Li
2023-10-19 7:00 ` Krzysztof Kozlowski
2023-10-19 12:07 ` Rob Herring
2023-10-18 21:58 ` [PATCH 3/5] i3c: slave: add svc slave controller support Frank Li
2023-10-23 1:29 ` kernel test robot
2023-10-18 21:58 ` [PATCH 4/5] i3c: slave: func: add tty driver Frank Li
2023-10-19 7:21 ` Jiri Slaby
2023-10-18 21:58 ` [PATCH 5/5] Documentation: i3c: Add I3C slave mode controller and function Frank Li
2023-10-18 22:07 ` Frank Li
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202310211527.TeVTMxA7-lkp@intel.com \
--to=yujie.liu@intel.com \
--cc=Frank.Li@nxp.com \
--cc=alexandre.belloni@bootlin.com \
--cc=conor+dt@kernel.org \
--cc=conor.culhane@silvaco.com \
--cc=corbet@lwn.net \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=imx@lists.linux.dev \
--cc=jirislaby@kernel.org \
--cc=joe@perches.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-i3c@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=miquel.raynal@bootlin.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).