From mboxrd@z Thu Jan 1 00:00:00 1970 From: Javi Merino Subject: [PATCH v3 4/4] thermal: power_allocator: exit early if there are no cooling devices Date: Mon, 17 Aug 2015 18:36:48 +0100 Message-ID: <1439833008-26440-5-git-send-email-javi.merino@arm.com> References: <1439288493-19740-1-git-send-email-javi.merino@arm.com> <1439833008-26440-1-git-send-email-javi.merino@arm.com> Return-path: In-Reply-To: <1439833008-26440-1-git-send-email-javi.merino@arm.com> Sender: linux-kernel-owner@vger.kernel.org To: linux-pm@vger.kernel.org Cc: dmitry.torokhov@gmail.com, cywang@chromium.org, linux-kernel@vger.kernel.org, punit.agrawal@arm.com, djkurtz@chromium.org, Javi Merino , Zhang Rui , Eduardo Valentin List-Id: linux-pm@vger.kernel.org Don't waste cycles in the power allocator governor's throttle function if there are no cooling devices and exit early. This commit doesn't change any functionality, but should provide better performance for the odd case of a thermal zone with trip points but without cooling devices. Cc: Zhang Rui Cc: Eduardo Valentin Signed-off-by: Javi Merino --- drivers/thermal/power_allocator.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/drivers/thermal/power_allocator.c b/drivers/thermal/power_allocator.c index 6dcc4fedd4f2..6b536ffd5ef6 100644 --- a/drivers/thermal/power_allocator.c +++ b/drivers/thermal/power_allocator.c @@ -291,6 +291,11 @@ static int allocate_power(struct thermal_zone_device *tz, } } + if (!num_actors) { + ret = -ENODEV; + goto unlock; + } + /* * We need to allocate five arrays of the same size: * req_power, max_power, granted_power, extra_actor_power and -- 1.9.1