From nobody Fri Dec 19 07:57:03 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 27019C46CA3 for ; Wed, 6 Dec 2023 11:31:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1377711AbjLFLa7 (ORCPT ); Wed, 6 Dec 2023 06:30:59 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54776 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1377585AbjLFLao (ORCPT ); Wed, 6 Dec 2023 06:30:44 -0500 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 62AE7D42; Wed, 6 Dec 2023 03:30:50 -0800 (PST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 2B402C15; Wed, 6 Dec 2023 03:31:36 -0800 (PST) Received: from e129166.arm.com (unknown [10.57.1.114]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id E01B03F5A1; Wed, 6 Dec 2023 03:30:48 -0800 (PST) From: Lukasz Luba To: linux-kernel@vger.kernel.org, daniel.lezcano@linaro.org, rafael@kernel.org Cc: linux-pm@vger.kernel.org, rui.zhang@intel.com, lukasz.luba@arm.com Subject: [PATCH 4/5] thermal: gov_power_allocator: Simplify checks for valid power actor Date: Wed, 6 Dec 2023 11:31:37 +0000 Message-Id: <20231206113138.3576492-5-lukasz.luba@arm.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20231206113138.3576492-1-lukasz.luba@arm.com> References: <20231206113138.3576492-1-lukasz.luba@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" There is a need to check if the cooling device in the thermal zone supports IPA callback and is set for control trip point. Refactor the code which validates the power actor capabilities and make it more consistent in all places. No intentional functional impact. Signed-off-by: Lukasz Luba --- drivers/thermal/gov_power_allocator.c | 41 +++++++++++---------------- 1 file changed, 17 insertions(+), 24 deletions(-) diff --git a/drivers/thermal/gov_power_allocator.c b/drivers/thermal/gov_po= wer_allocator.c index 1b55d00fc12b..8f2d2ee3def0 100644 --- a/drivers/thermal/gov_power_allocator.c +++ b/drivers/thermal/gov_power_allocator.c @@ -85,6 +85,13 @@ struct power_allocator_params { u32 *weighted_req_power; }; =20 +static bool power_actor_is_valid(struct power_allocator_params *params, + struct thermal_instance *instance) +{ + return ((instance->trip =3D=3D params->trip_max) && + cdev_is_power_actor(instance->cdev)); +} + /** * estimate_sustainable_power() - Estimate the sustainable power of a ther= mal zone * @tz: thermal zone we are operating in @@ -105,14 +112,10 @@ static u32 estimate_sustainable_power(struct thermal_= zone_device *tz) u32 min_power; =20 list_for_each_entry(instance, &tz->thermal_instances, tz_node) { - cdev =3D instance->cdev; - - if (instance->trip !=3D params->trip_max) - continue; - - if (!cdev_is_power_actor(cdev)) + if (!power_actor_is_valid(params, instance)) continue; =20 + cdev =3D instance->cdev; if (cdev->ops->state2power(cdev, instance->upper, &min_power)) continue; =20 @@ -407,8 +410,7 @@ static int allocate_power(struct thermal_zone_device *t= z, int control_temp) return -ENODEV; =20 list_for_each_entry(instance, &tz->thermal_instances, tz_node) - if ((instance->trip =3D=3D params->trip_max) && - cdev_is_power_actor(instance->cdev)) + if (power_actor_is_valid(params, instance)) total_weight +=3D instance->weight; =20 /* Clean all buffers for new power estimations */ @@ -421,14 +423,10 @@ static int allocate_power(struct thermal_zone_device = *tz, int control_temp) weighted_req_power =3D params->weighted_req_power; =20 list_for_each_entry(instance, &tz->thermal_instances, tz_node) { - cdev =3D instance->cdev; - - if (instance->trip !=3D params->trip_max) - continue; - - if (!cdev_is_power_actor(cdev)) + if (!power_actor_is_valid(params, instance)) continue; =20 + cdev =3D instance->cdev; if (cdev->ops->get_requested_power(cdev, &req_power[i])) continue; =20 @@ -458,10 +456,7 @@ static int allocate_power(struct thermal_zone_device *= tz, int control_temp) =20 i =3D 0; list_for_each_entry(instance, &tz->thermal_instances, tz_node) { - if (instance->trip !=3D params->trip_max) - continue; - - if (!cdev_is_power_actor(instance->cdev)) + if (!power_actor_is_valid(params, instance)) continue; =20 power_actor_set_power(instance->cdev, instance, @@ -546,12 +541,11 @@ static void allow_maximum_power(struct thermal_zone_d= evice *tz, bool update) u32 req_power; =20 list_for_each_entry(instance, &tz->thermal_instances, tz_node) { - cdev =3D instance->cdev; - - if (instance->trip !=3D params->trip_max || - !cdev_is_power_actor(instance->cdev)) + if (!power_actor_is_valid(params, instance)) continue; =20 + cdev =3D instance->cdev; + instance->target =3D 0; mutex_lock(&cdev->lock); /* @@ -661,8 +655,7 @@ static void power_allocator_instances_update(struct the= rmal_zone_device *tz) int num_actors =3D 0; =20 list_for_each_entry(instance, &tz->thermal_instances, tz_node) - if ((instance->trip =3D=3D params->trip_max) && - cdev_is_power_actor(instance->cdev)) + if (power_actor_is_valid(params, instance)) num_actors++; =20 if (num_actors =3D=3D params->num_actors) --=20 2.25.1