From: kernel test robot <lkp@intel.com>
To: Deepak Kumar Singh <deesin@codeaurora.org>,
bjorn.andersson@linaro.org, clew@codeaurora.org,
mathieu.poirier@linaro.org
Cc: kbuild-all@lists.01.org, rampraka@codeaurora.org,
Arun Kumar Neelakantam <aneela@codeaurora.org>,
Deepak Kumar Singh <deesin@codeaurora.org>,
Ohad Ben-Cohen <ohad@wizery.com>,
"open list:REMOTE PROCESSOR MESSAGING (RPMSG) SUBSYSTEM"
<linux-remoteproc@vger.kernel.org>,
open list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH V5 4/4] rpmsg: char: Add signal callback and POLLPRI support
Date: Thu, 25 Jun 2020 06:26:49 +0800 [thread overview]
Message-ID: <202006250635.S7vKsAc7%lkp@intel.com> (raw)
In-Reply-To: <1593020701-23778-5-git-send-email-deesin@codeaurora.org>
[-- Attachment #1: Type: text/plain, Size: 2000 bytes --]
Hi Deepak,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v5.8-rc2 next-20200624]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Deepak-Kumar-Singh/rpmsg-core-Add-signal-API-support/20200625-024527
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 3e08a95294a4fb3702bb3d35ed08028433c37fe6
config: riscv-randconfig-s031-20200624 (attached as .config)
compiler: riscv32-linux-gcc (GCC) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.2-dirty
# save the attached .config to linux build tree
make W=1 C=1 ARCH=riscv CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
sparse warnings: (new ones prefixed by >>)
>> drivers/rpmsg/rpmsg_char.c:288:22: sparse: sparse: invalid assignment: |=
>> drivers/rpmsg/rpmsg_char.c:288:22: sparse: left side has type restricted __poll_t
>> drivers/rpmsg/rpmsg_char.c:288:22: sparse: right side has type int
vim +288 drivers/rpmsg/rpmsg_char.c
273
274 static __poll_t rpmsg_eptdev_poll(struct file *filp, poll_table *wait)
275 {
276 struct rpmsg_eptdev *eptdev = filp->private_data;
277 __poll_t mask = 0;
278
279 if (!eptdev->ept)
280 return EPOLLERR;
281
282 poll_wait(filp, &eptdev->readq, wait);
283
284 if (!skb_queue_empty(&eptdev->queue))
285 mask |= EPOLLIN | EPOLLRDNORM;
286
287 if (eptdev->sig_pending)
> 288 mask |= POLLPRI;
289
290 mask |= rpmsg_poll(eptdev->ept, filp, wait);
291
292 return mask;
293 }
294
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 27844 bytes --]
prev parent reply other threads:[~2020-06-24 22:40 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1593020701-23778-1-git-send-email-deesin@codeaurora.org>
2020-06-24 17:44 ` [PATCH V5 1/4] rpmsg: core: Add signal API support Deepak Kumar Singh
2020-06-24 17:44 ` [PATCH V5 2/4] rpmsg: glink: Add support to handle signals command Deepak Kumar Singh
2020-06-24 17:45 ` [PATCH V5 3/4] rpmsg: char: Add TIOCMGET/TIOCMSET ioctl support Deepak Kumar Singh
2020-06-24 17:45 ` [PATCH V5 4/4] rpmsg: char: Add signal callback and POLLPRI support Deepak Kumar Singh
2020-06-24 22:26 ` kernel test robot [this message]
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=202006250635.S7vKsAc7%lkp@intel.com \
--to=lkp@intel.com \
--cc=aneela@codeaurora.org \
--cc=bjorn.andersson@linaro.org \
--cc=clew@codeaurora.org \
--cc=deesin@codeaurora.org \
--cc=kbuild-all@lists.01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-remoteproc@vger.kernel.org \
--cc=mathieu.poirier@linaro.org \
--cc=ohad@wizery.com \
--cc=rampraka@codeaurora.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