aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>2020-02-11 23:58:26 +0100
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2020-02-13 11:25:48 +0100
commit87ad7356799622b69478076cd99df7a5024992cb (patch)
treebfa12ce81122cd93bec3821629d299dc1ef0ea52
parentPM: QoS: Drop pm_qos_update_request_timeout() (diff)
downloadlinux-dev-87ad7356799622b69478076cd99df7a5024992cb.tar.xz
linux-dev-87ad7356799622b69478076cd99df7a5024992cb.zip
PM: QoS: Drop the PM_QOS_SUM QoS type
The PM_QOS_SUM QoS type is not used, so drop it along with the code referring to it in pm_qos_get_value() and the related local variables in there. No intentional functional impact. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org> Reviewed-by: Amit Kucheria <amit.kucheria@linaro.org> Tested-by: Amit Kucheria <amit.kucheria@linaro.org>
-rw-r--r--include/linux/pm_qos.h1
-rw-r--r--kernel/power/qos.c9
2 files changed, 0 insertions, 10 deletions
diff --git a/include/linux/pm_qos.h b/include/linux/pm_qos.h
index 4747bdb6bed2..48bfb96a9360 100644
--- a/include/linux/pm_qos.h
+++ b/include/linux/pm_qos.h
@@ -53,7 +53,6 @@ enum pm_qos_type {
PM_QOS_UNITIALIZED,
PM_QOS_MAX, /* return the largest value */
PM_QOS_MIN, /* return the smallest value */
- PM_QOS_SUM /* return the sum */
};
/*
diff --git a/kernel/power/qos.c b/kernel/power/qos.c
index 67dab7f330e4..a6be7faa1974 100644
--- a/kernel/power/qos.c
+++ b/kernel/power/qos.c
@@ -101,9 +101,6 @@ static const struct file_operations pm_qos_power_fops = {
/* unlocked internal variant */
static inline int pm_qos_get_value(struct pm_qos_constraints *c)
{
- struct plist_node *node;
- int total_value = 0;
-
if (plist_head_empty(&c->list))
return c->no_constraint_value;
@@ -114,12 +111,6 @@ static inline int pm_qos_get_value(struct pm_qos_constraints *c)
case PM_QOS_MAX:
return plist_last(&c->list)->prio;
- case PM_QOS_SUM:
- plist_for_each(node, &c->list)
- total_value += node->prio;
-
- return total_value;
-
default:
/* runtime check for not using enum */
BUG();