From: kbuild test robot <lkp@intel.com>
To: Ran Wang <ran.wang_1@nxp.com>
Cc: Mark Rutland <mark.rutland@arm.com>, Li Biwen <biwen.li@nxp.com>,
kbuild-all@lists.01.org, Len Brown <len.brown@intel.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-pm@vger.kernel.org,
"Rafael J . Wysocki" <rjw@rjwysocki.net>,
linux-kernel@vger.kernel.org, Li Yang <leoyang.li@nxp.com>,
devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
Pavel Machek <pavel@ucw.cz>, Ran Wang <ran.wang_1@nxp.com>,
linuxppc-dev@lists.ozlabs.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v7 3/3] soc: fsl: add RCPM driver
Date: Thu, 24 Oct 2019 10:57:13 +0800 [thread overview]
Message-ID: <201910241056.5U6SmK2P%lkp@intel.com> (raw)
In-Reply-To: <20191021034927.19300-3-ran.wang_1@nxp.com>
[-- Attachment #1: Type: text/plain, Size: 3182 bytes --]
Hi Ran,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[cannot apply to v5.4-rc4 next-20191023]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Ran-Wang/PM-wakeup-Add-routine-to-help-fetch-wakeup-source-object/20191024-101743
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 13b86bc4cd648eae69fdcf3d04b2750c76350053
config: mips-allmodconfig (attached as .config)
compiler: mips-linux-gcc (GCC) 7.4.0
reproduce:
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
GCC_VERSION=7.4.0 make.cross ARCH=mips
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/soc/fsl/rcpm.c: In function 'rcpm_pm_prepare':
>> drivers/soc/fsl/rcpm.c:72:11: warning: suggest parentheses around assignment used as truth value [-Wparentheses]
} while (ws = wakeup_source_get_next(ws));
^~
vim +72 drivers/soc/fsl/rcpm.c
24
25 static int rcpm_pm_prepare(struct device *dev)
26 {
27 int i, ret, idx;
28 void __iomem *base;
29 struct wakeup_source *ws;
30 struct rcpm *rcpm;
31 struct device_node *np = dev->of_node;
32 u32 value[RCPM_WAKEUP_CELL_MAX_SIZE + 1], tmp;
33
34 rcpm = dev_get_drvdata(dev);
35 if (!rcpm)
36 return -EINVAL;
37
38 base = rcpm->ippdexpcr_base;
39
40 /* Begin with first registered wakeup source */
41 ws = wakeup_source_get_start(&idx);
42 do {
43 /* skip object which is not attached to device */
44 if (!ws->dev->parent)
45 continue;
46
47 ret = device_property_read_u32_array(ws->dev->parent,
48 "fsl,rcpm-wakeup", value,
49 rcpm->wakeup_cells + 1);
50
51 /* Wakeup source should refer to current rcpm device */
52 if (ret || (np->phandle != value[0])) {
53 dev_info(dev, "%s doesn't refer to this rcpm\n",
54 ws->name);
55 continue;
56 }
57
58 for (i = 0; i < rcpm->wakeup_cells; i++) {
59 /* We can only OR related bits */
60 if (value[i + 1]) {
61 if (rcpm->little_endian) {
62 tmp = ioread32(base + i * 4);
63 tmp |= value[i + 1];
64 iowrite32(tmp, base + i * 4);
65 } else {
66 tmp = ioread32be(base + i * 4);
67 tmp |= value[i + 1];
68 iowrite32be(tmp, base + i * 4);
69 }
70 }
71 }
> 72 } while (ws = wakeup_source_get_next(ws));
73
74 wakeup_source_get_stop(idx);
75
76 return 0;
77 }
78
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 62100 bytes --]
next prev parent reply other threads:[~2019-10-24 2:59 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-21 3:49 [PATCH v7 1/3] PM: wakeup: Add routine to help fetch wakeup source object Ran Wang
2019-10-21 3:49 ` [PATCH v7 2/3] Documentation: dt: binding: fsl: Add 'little-endian' and update Chassis define Ran Wang
2019-10-24 18:33 ` Scott Wood
2019-10-25 2:40 ` Ran Wang
2019-10-21 3:49 ` [PATCH v7 3/3] soc: fsl: add RCPM driver Ran Wang
2019-10-24 2:57 ` kbuild test robot [this message]
2019-10-21 8:38 ` [PATCH v7 1/3] PM: wakeup: Add routine to help fetch wakeup source object Rafael J. Wysocki
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=201910241056.5U6SmK2P%lkp@intel.com \
--to=lkp@intel.com \
--cc=biwen.li@nxp.com \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=kbuild-all@lists.01.org \
--cc=len.brown@intel.com \
--cc=leoyang.li@nxp.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mark.rutland@arm.com \
--cc=pavel@ucw.cz \
--cc=ran.wang_1@nxp.com \
--cc=rjw@rjwysocki.net \
--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).