From patchwork Mon Dec 7 19:08:59 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Daniel Lezcano X-Patchwork-Id: 11956531 X-Patchwork-Delegate: rui.zhang@intel.com Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 497F1C4167B for ; Mon, 7 Dec 2020 19:10:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 17C38238A0 for ; Mon, 7 Dec 2020 19:10:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726647AbgLGTKC (ORCPT ); Mon, 7 Dec 2020 14:10:02 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55700 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726190AbgLGTKC (ORCPT ); Mon, 7 Dec 2020 14:10:02 -0500 Received: from mail-wm1-x344.google.com (mail-wm1-x344.google.com [IPv6:2a00:1450:4864:20::344]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 43F28C061794 for ; Mon, 7 Dec 2020 11:09:16 -0800 (PST) Received: by mail-wm1-x344.google.com with SMTP id d3so220685wmb.4 for ; Mon, 07 Dec 2020 11:09:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=jLzBGlSW3uUTIEu7vncLkrlWKsGqZuKn9AcbkvV5Usk=; b=KUKwbIk9HYGTBhmuBYLvw5fsabq33U6zbk//YQgkHImTT+LqTI4cXeMgHXWZUhryap vZoA+aS8JzLxTY1oH3rGUcMEZOPkeCOq+B6UewDiYKWJB3/e2/6168Jg0N6H0dt6Wfv6 XcF9hOk6TpBmw3t0BfrjitHcxTxx0alVgPqCq4UNHwebDIHYs4S7FNbpVhBYpvjZCQ63 HMWX2nOASHriSxDlfWSsvmfqJNY4E5Qm3MNvOI+g45xSrlen83M8NYkiUGgDk/v9Kzmm cZ6Er6sMOvjnMk15qpYDo6EYMw9xEAdPxAkgSzZhr7eFELnYIH6K5+49SX/NL2CshCUQ aGDg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=jLzBGlSW3uUTIEu7vncLkrlWKsGqZuKn9AcbkvV5Usk=; b=ZTXZvzDPCDrnkPJOE4HYY+k9TaRO6LgSxeALg0/VGisSG9xi02M/x1pqZbizXlhL75 O21BPbivqbCVab0stKsbW9d0d5duiTepUfW067UxVOt5BXiFsKX454i61Bk95LNF6gC2 tr/OFjLOJxuqqh17c7EPDkpNSHovudsNgNCavdgNOc3PWAY3NH8nSPBATkP3j8+eL7In F7kpzjJzFYp9tTGKHNy46Fq5YB14rIoLRh17MeLkBfbGQzXsh5YkK787jrEQ7xITfjVw /6Uj3qYulP1OCPBOlmp3MBFBLbwfj4qxoRzDYMqqDAL46DsUgPx1CvKX0PMlu9uMWXbZ ptog== X-Gm-Message-State: AOAM533560jWgZ+Xy1tpUMB/nZN6dNayjWaOyQZ7wWvd+/T6s0sLX2HC zvCRJYPoIUSXtX5gkCW+6027Nw== X-Google-Smtp-Source: ABdhPJx7x44xXvaS39ZrK68URTaLEpb2c8ii/QSsSruqIFpQ9Gr/e5PtaKuwoMnm+QYYFlNUHEyQjg== X-Received: by 2002:a1c:f00a:: with SMTP id a10mr273464wmb.83.1607368154795; Mon, 07 Dec 2020 11:09:14 -0800 (PST) Received: from localhost.localdomain (lns-bzn-59-82-252-158-132.adsl.proxad.net. [82.252.158.132]) by smtp.gmail.com with ESMTPSA id v1sm16335827wrr.48.2020.12.07.11.09.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Dec 2020 11:09:14 -0800 (PST) From: Daniel Lezcano To: daniel.lezcano@linaro.org, rui.zhang@intel.com Cc: amitk@kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org Subject: [PATCH v2 1/4] thermal/core: Rename passive_delay and polling_delay with units Date: Mon, 7 Dec 2020 20:08:59 +0100 Message-Id: <20201207190902.30464-1-daniel.lezcano@linaro.org> X-Mailer: git-send-email 2.17.1 Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org The input unit used by the thermal framework is the msec but it uses the jiffies to set the timers. As it is stored in the thermal zone device structure, everytime the timer is setup at each polling interval, the msecs to jiffies conversion happens. The jiffies is the unit the thermal framework is using, so keeping it under the jiffies instead of msecs will save some pointless conversion. Set the scene to directly store the delays under their jiffies form by adding to the variable name the 'ms' suffix. Signed-off-by: Daniel Lezcano Reviewed-by: Lukasz Luba --- drivers/platform/x86/acerhdf.c | 2 +- drivers/thermal/da9062-thermal.c | 4 ++-- drivers/thermal/gov_power_allocator.c | 2 +- drivers/thermal/thermal_core.c | 10 +++++----- drivers/thermal/thermal_of.c | 16 ++++++++-------- drivers/thermal/thermal_sysfs.c | 6 +++--- .../thermal/ti-soc-thermal/ti-thermal-common.c | 2 +- include/linux/thermal.h | 8 ++++---- 8 files changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/platform/x86/acerhdf.c b/drivers/platform/x86/acerhdf.c index b6aa6e5514f4..7b26f601b407 100644 --- a/drivers/platform/x86/acerhdf.c +++ b/drivers/platform/x86/acerhdf.c @@ -336,7 +336,7 @@ static void acerhdf_check_param(struct thermal_zone_device *thermal) pr_notice("interval changed to: %d\n", interval); if (thermal) - thermal->polling_delay = interval*1000; + thermal->polling_delay_ms = interval*1000; prev_interval = interval; } diff --git a/drivers/thermal/da9062-thermal.c b/drivers/thermal/da9062-thermal.c index 4d74994f160a..ebb3d0b4a5be 100644 --- a/drivers/thermal/da9062-thermal.c +++ b/drivers/thermal/da9062-thermal.c @@ -95,7 +95,7 @@ static void da9062_thermal_poll_on(struct work_struct *work) thermal_zone_device_update(thermal->zone, THERMAL_EVENT_UNSPECIFIED); - delay = msecs_to_jiffies(thermal->zone->passive_delay); + delay = msecs_to_jiffies(thermal->zone->passive_delay_ms); queue_delayed_work(system_freezable_wq, &thermal->work, delay); return; } @@ -245,7 +245,7 @@ static int da9062_thermal_probe(struct platform_device *pdev) dev_dbg(&pdev->dev, "TJUNC temperature polling period set at %d ms\n", - thermal->zone->passive_delay); + thermal->zone->passive_delay_ms); ret = platform_get_irq_byname(pdev, "THERMAL"); if (ret < 0) { diff --git a/drivers/thermal/gov_power_allocator.c b/drivers/thermal/gov_power_allocator.c index 7a4170a0b51f..f7a663f698d4 100644 --- a/drivers/thermal/gov_power_allocator.c +++ b/drivers/thermal/gov_power_allocator.c @@ -258,7 +258,7 @@ static u32 pid_controller(struct thermal_zone_device *tz, * power being applied, slowing down the controller) */ d = mul_frac(tz->tzp->k_d, err - params->prev_err); - d = div_frac(d, tz->passive_delay); + d = div_frac(d, tz->passive_delay_ms); params->prev_err = err; power_range = p + i + d; diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c index 1bd23ff2247b..5b500d72aab4 100644 --- a/drivers/thermal/thermal_core.c +++ b/drivers/thermal/thermal_core.c @@ -317,9 +317,9 @@ static void monitor_thermal_zone(struct thermal_zone_device *tz) mutex_lock(&tz->lock); if (!stop && tz->passive) - thermal_zone_device_set_polling(tz, tz->passive_delay); - else if (!stop && tz->polling_delay) - thermal_zone_device_set_polling(tz, tz->polling_delay); + thermal_zone_device_set_polling(tz, tz->passive_delay_ms); + else if (!stop && tz->polling_delay_ms) + thermal_zone_device_set_polling(tz, tz->polling_delay_ms); else thermal_zone_device_set_polling(tz, 0); @@ -1340,8 +1340,8 @@ thermal_zone_device_register(const char *type, int trips, int mask, tz->device.class = &thermal_class; tz->devdata = devdata; tz->trips = trips; - tz->passive_delay = passive_delay; - tz->polling_delay = polling_delay; + tz->passive_delay_ms = passive_delay; + tz->polling_delay_ms = polling_delay; /* sys I/F */ /* Add nodes that are always present via .groups */ diff --git a/drivers/thermal/thermal_of.c b/drivers/thermal/thermal_of.c index 69ef12f852b7..c01248e48c09 100644 --- a/drivers/thermal/thermal_of.c +++ b/drivers/thermal/thermal_of.c @@ -51,8 +51,8 @@ struct __thermal_bind_params { /** * struct __thermal_zone - internal representation of a thermal zone - * @passive_delay: polling interval while passive cooling is activated - * @polling_delay: zone polling interval + * @passive_delay_ms: polling interval while passive cooling is activated + * @polling_delay_ms: zone polling interval * @slope: slope of the temperature adjustment curve * @offset: offset of the temperature adjustment curve * @ntrips: number of trip points @@ -64,8 +64,8 @@ struct __thermal_bind_params { */ struct __thermal_zone { - int passive_delay; - int polling_delay; + int passive_delay_ms; + int polling_delay_ms; int slope; int offset; @@ -865,14 +865,14 @@ __init *thermal_of_build_thermal_zone(struct device_node *np) pr_err("%pOFn: missing polling-delay-passive property\n", np); goto free_tz; } - tz->passive_delay = prop; + tz->passive_delay_ms = prop; ret = of_property_read_u32(np, "polling-delay", &prop); if (ret < 0) { pr_err("%pOFn: missing polling-delay property\n", np); goto free_tz; } - tz->polling_delay = prop; + tz->polling_delay_ms = prop; /* * REVIST: for now, the thermal framework supports only @@ -1085,8 +1085,8 @@ int __init of_parse_thermal_zones(void) zone = thermal_zone_device_register(child->name, tz->ntrips, mask, tz, ops, tzp, - tz->passive_delay, - tz->polling_delay); + tz->passive_delay_ms, + tz->polling_delay_ms); if (IS_ERR(zone)) { pr_err("Failed to build %pOFn zone %ld\n", child, PTR_ERR(zone)); diff --git a/drivers/thermal/thermal_sysfs.c b/drivers/thermal/thermal_sysfs.c index 0866e949339b..f465462d8aa1 100644 --- a/drivers/thermal/thermal_sysfs.c +++ b/drivers/thermal/thermal_sysfs.c @@ -233,12 +233,12 @@ passive_store(struct device *dev, struct device_attribute *attr, return -EINVAL; if (state && !tz->forced_passive) { - if (!tz->passive_delay) - tz->passive_delay = 1000; + if (!tz->passive_delay_ms) + tz->passive_delay_ms = 1000; thermal_zone_device_rebind_exception(tz, "Processor", sizeof("Processor")); } else if (!state && tz->forced_passive) { - tz->passive_delay = 0; + tz->passive_delay_ms = 0; thermal_zone_device_unbind_exception(tz, "Processor", sizeof("Processor")); } diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c index 2ce4b19f312a..4baff19e1142 100644 --- a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c +++ b/drivers/thermal/ti-soc-thermal/ti-thermal-common.c @@ -185,7 +185,7 @@ int ti_thermal_expose_sensor(struct ti_bandgap *bgp, int id, ti_bandgap_set_sensor_data(bgp, id, data); ti_bandgap_write_update_interval(bgp, data->sensor_id, - data->ti_thermal->polling_delay); + data->ti_thermal->polling_delay_ms); return 0; } diff --git a/include/linux/thermal.h b/include/linux/thermal.h index d07ea27e72a9..230d451bf335 100644 --- a/include/linux/thermal.h +++ b/include/linux/thermal.h @@ -116,9 +116,9 @@ struct thermal_cooling_device { * @devdata: private pointer for device private data * @trips: number of trip points the thermal zone supports * @trips_disabled; bitmap for disabled trips - * @passive_delay: number of milliseconds to wait between polls when + * @passive_delay_ms: number of milliseconds to wait between polls when * performing passive cooling. - * @polling_delay: number of milliseconds to wait between polls when + * @polling_delay_ms: number of milliseconds to wait between polls when * checking whether trip points have been crossed (0 for * interrupt driven systems) * @temperature: current temperature. This is only for core code, @@ -159,8 +159,8 @@ struct thermal_zone_device { void *devdata; int trips; unsigned long trips_disabled; /* bitmap for disabled trips */ - int passive_delay; - int polling_delay; + int passive_delay_ms; + int polling_delay_ms; int temperature; int last_temperature; int emul_temperature;