From: kbuild test robot <lkp@intel.com>
To: Bhumika Goyal <bhumirks@gmail.com>
Cc: kbuild-all@01.org, julia.lawall@lip6.fr, lgirdwood@gmail.com,
broonie@kernel.org, michael.hennerich@analog.com, wens@csie.org,
cw00.choi@samsung.com, krzk@kernel.org, b.zolnierkie@samsung.com,
javier@osg.samsung.com, tony@atomide.com, sbkim73@samsung.com,
liviu.dudau@arm.com, sudeep.holla@arm.com,
lorenzo.pieralisi@arm.com, linux-kernel@vger.kernel.org,
linux-omap@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
Bhumika Goyal <bhumirks@gmail.com>
Subject: Re: [PATCH] regulator: constify regulator_ops structures
Date: Thu, 26 Jan 2017 19:57:35 +0800 [thread overview]
Message-ID: <201701261903.hqeTiDIx%fengguang.wu@intel.com> (raw)
In-Reply-To: <1485427668-2816-1-git-send-email-bhumirks@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 10145 bytes --]
Hi Bhumika,
[auto build test ERROR on regulator/for-next]
[cannot apply to v4.10-rc5 next-20170125]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Bhumika-Goyal/regulator-constify-regulator_ops-structures/20170126-185451
base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 6.2.0
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=ia64
All error/warnings (new ones prefixed by >>):
drivers/regulator/anatop-regulator.c: In function 'anatop_regulator_probe':
>> drivers/regulator/anatop-regulator.c:310:24: error: assignment of member 'enable' in read-only object
anatop_rops.enable = regulator_enable_regmap;
^
>> drivers/regulator/anatop-regulator.c:311:24: error: assignment of member 'disable' in read-only object
anatop_rops.disable = regulator_disable_regmap;
^
>> drivers/regulator/anatop-regulator.c:312:27: error: assignment of member 'is_enabled' in read-only object
anatop_rops.is_enabled = regulator_is_enabled_regmap;
^
--
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
>> drivers/regulator/qcom_spmi-regulator.c:1169:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(BUCK, GP_CTL, 0, INF, SMPS, smps, smps, 100000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1170:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, N300, 0, INF, LDO, ldo, nldo1, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1171:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, N600, 0, 0, LDO, ldo, nldo2, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1172:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, N1200, 0, 0, LDO, ldo, nldo2, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1173:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, N600, 1, INF, LDO, ldo, nldo3, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1174:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, N1200, 1, INF, LDO, ldo, nldo3, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1175:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, N600_ST, 0, 0, LDO, ldo, nldo2, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1176:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, N1200_ST, 0, 0, LDO, ldo, nldo2, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1177:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, N600_ST, 1, INF, LDO, ldo, nldo3, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1178:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, N1200_ST, 1, INF, LDO, ldo, nldo3, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1179:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, P50, 0, INF, LDO, ldo, pldo, 5000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1180:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, P150, 0, INF, LDO, ldo, pldo, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1181:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, P300, 0, INF, LDO, ldo, pldo, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1182:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, P600, 0, INF, LDO, ldo, pldo, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1183:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, P1200, 0, INF, LDO, ldo, pldo, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1184:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, LN, 0, INF, LN_LDO, ln_ldo, ln_ldo, 0),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1185:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, LV_P50, 0, INF, LDO, ldo, pldo, 5000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1186:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, LV_P150, 0, INF, LDO, ldo, pldo, 10000),
^~~~~~~~~
>> drivers/regulator/qcom_spmi-regulator.c:350:11: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ops = &spmi_##_ops_val##_ops, \
^
drivers/regulator/qcom_spmi-regulator.c:1187:2: note: in expansion of macro 'SPMI_VREG'
SPMI_VREG(LDO, LV_P300, 0, INF, LDO, ldo, pldo, 10000),
^~~~~~~~~
vim +/enable +310 drivers/regulator/anatop-regulator.c
ca7734ad Andrey Smirnov 2017-01-10 304 u32 enable_bit;
ca7734ad Andrey Smirnov 2017-01-10 305
605ebd35 Philipp Zabel 2014-02-11 306 rdesc->ops = &anatop_rops;
ca7734ad Andrey Smirnov 2017-01-10 307
ca7734ad Andrey Smirnov 2017-01-10 308 if (!of_property_read_u32(np, "anatop-enable-bit",
ca7734ad Andrey Smirnov 2017-01-10 309 &enable_bit)) {
ca7734ad Andrey Smirnov 2017-01-10 @310 anatop_rops.enable = regulator_enable_regmap;
ca7734ad Andrey Smirnov 2017-01-10 @311 anatop_rops.disable = regulator_disable_regmap;
ca7734ad Andrey Smirnov 2017-01-10 @312 anatop_rops.is_enabled = regulator_is_enabled_regmap;
ca7734ad Andrey Smirnov 2017-01-10 313
ca7734ad Andrey Smirnov 2017-01-10 314 rdesc->enable_reg = sreg->control_reg;
ca7734ad Andrey Smirnov 2017-01-10 315 rdesc->enable_mask = BIT(enable_bit);
:::::: The code at line 310 was first introduced by commit
:::::: ca7734ad77b4ffb83a03c80b8d64d40c7ef49263 regulator: anatop: Add support for "anatop-enable-bit"
:::::: TO: Andrey Smirnov <andrew.smirnov@gmail.com>
:::::: CC: Mark Brown <broonie@kernel.org>
---
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: 45045 bytes --]
next prev parent reply other threads:[~2017-01-26 11:57 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-26 10:47 [PATCH] regulator: constify regulator_ops structures Bhumika Goyal
2017-01-26 11:57 ` kbuild test robot [this message]
2017-01-26 12:30 ` kbuild test robot
2017-01-26 12:37 ` Mark Brown
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=201701261903.hqeTiDIx%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=b.zolnierkie@samsung.com \
--cc=bhumirks@gmail.com \
--cc=broonie@kernel.org \
--cc=cw00.choi@samsung.com \
--cc=javier@osg.samsung.com \
--cc=julia.lawall@lip6.fr \
--cc=kbuild-all@01.org \
--cc=krzk@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=liviu.dudau@arm.com \
--cc=lorenzo.pieralisi@arm.com \
--cc=michael.hennerich@analog.com \
--cc=sbkim73@samsung.com \
--cc=sudeep.holla@arm.com \
--cc=tony@atomide.com \
--cc=wens@csie.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).