* [pm:bleeding-edge 59/60] drivers/power/avs/qcom-cpr.c:1081:15: sparse: sparse: symbol 'cpr_get_opp_hz_for_req' was not declared. Should it be static?
@ 2019-12-24 22:18 kbuild test robot
2019-12-24 22:18 ` [RFC PATCH pm] power: avs: cpr_get_opp_hz_for_req() can be static kbuild test robot
0 siblings, 1 reply; 2+ messages in thread
From: kbuild test robot @ 2019-12-24 22:18 UTC (permalink / raw)
To: Niklas Cassel
Cc: kbuild-all, linux-acpi, devel, linux-pm, Rafael J. Wysocki,
Jorge Ramirez-Ortiz, Bjorn Andersson, Ulf Hansson
tree: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git bleeding-edge
head: 89eba9e38dc4ed4576eaf7711c60403568663291
commit: bf6910abf54871b0e976e52f56fb3b3dd1b90e48 [59/60] power: avs: Add support for CPR (Core Power Reduction)
reproduce:
# apt-get install sparse
# sparse version: v0.6.1-129-g341daf20-dirty
git checkout bf6910abf54871b0e976e52f56fb3b3dd1b90e48
make ARCH=x86_64 allmodconfig
make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
sparse warnings: (new ones prefixed by >>)
>> drivers/power/avs/qcom-cpr.c:1081:15: sparse: sparse: symbol 'cpr_get_opp_hz_for_req' was not declared. Should it be static?
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation
^ permalink raw reply [flat|nested] 2+ messages in thread
* [RFC PATCH pm] power: avs: cpr_get_opp_hz_for_req() can be static
2019-12-24 22:18 [pm:bleeding-edge 59/60] drivers/power/avs/qcom-cpr.c:1081:15: sparse: sparse: symbol 'cpr_get_opp_hz_for_req' was not declared. Should it be static? kbuild test robot
@ 2019-12-24 22:18 ` kbuild test robot
0 siblings, 0 replies; 2+ messages in thread
From: kbuild test robot @ 2019-12-24 22:18 UTC (permalink / raw)
To: Niklas Cassel
Cc: kbuild-all, linux-acpi, devel, linux-pm, Rafael J. Wysocki,
Jorge Ramirez-Ortiz, Bjorn Andersson, Ulf Hansson
Fixes: bf6910abf548 ("power: avs: Add support for CPR (Core Power Reduction)")
Signed-off-by: kbuild test robot <lkp@intel.com>
---
qcom-cpr.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/power/avs/qcom-cpr.c b/drivers/power/avs/qcom-cpr.c
index a3187973bbb5fd..98c9ee4075ae9d 100644
--- a/drivers/power/avs/qcom-cpr.c
+++ b/drivers/power/avs/qcom-cpr.c
@@ -1078,8 +1078,8 @@ static unsigned int cpr_get_fuse_corner(struct dev_pm_opp *opp)
return fuse_corner;
}
-unsigned long cpr_get_opp_hz_for_req(struct dev_pm_opp *ref,
- struct device *cpu_dev)
+static unsigned long cpr_get_opp_hz_for_req(struct dev_pm_opp *ref,
+ struct device *cpu_dev)
{
u64 rate = 0;
struct device_node *ref_np;
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-12-24 22:19 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-12-24 22:18 [pm:bleeding-edge 59/60] drivers/power/avs/qcom-cpr.c:1081:15: sparse: sparse: symbol 'cpr_get_opp_hz_for_req' was not declared. Should it be static? kbuild test robot
2019-12-24 22:18 ` [RFC PATCH pm] power: avs: cpr_get_opp_hz_for_req() can be static kbuild test robot
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).