Message-ID: <ff124b19-4a21-f9f8-4249-d72e3b8dbd6a@linaro.org>
Date:   Mon, 27 Feb 2023 19:14:17 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.1
Subject: Re: [PATCH v3 14/20] thermal/drivers/da9062: Don't access the thermal
 zone device fields
Content-Language: en-US
To:     DLG Adam Ward <DLG-Adam.Ward.opensource@dm.renesas.com>
Cc:     linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
        Support Opensource <support.opensource@diasemi.com>,
        Amit Kucheria <amitk@kernel.org>,
        Zhang Rui <rui.zhang@intel.com>, rafael@kernel.org
References: <20230226225406.979703-1-daniel.lezcano@linaro.org>
 <20230226225406.979703-15-daniel.lezcano@linaro.org>
From:   Daniel Lezcano <daniel.lezcano@linaro.org>
In-Reply-To: <20230226225406.979703-15-daniel.lezcano@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk


Hi Adam,

are you ok with this patch ?


On 26/02/2023 23:54, Daniel Lezcano wrote:
> The driver is reading the passive polling rate in the thermal zone
> structure. We want to prevent the drivers to rummage around in the
> thermal zone structure.
> 
> On the other side, the delay is what the driver passed to the
> thermal_zone_device_register() function, so it has already the
> information.
> 
> Reuse the information we have instead of reading the information we
> set.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> ---
>   drivers/thermal/da9062-thermal.c | 11 +++++++----
>   1 file changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/thermal/da9062-thermal.c b/drivers/thermal/da9062-thermal.c
> index e7097f354750..2d31b1f73423 100644
> --- a/drivers/thermal/da9062-thermal.c
> +++ b/drivers/thermal/da9062-thermal.c
> @@ -41,6 +41,8 @@
>   
>   #define DA9062_MILLI_CELSIUS(t)			((t) * 1000)
>   
> +static unsigned int pp_tmp = DA9062_DEFAULT_POLLING_MS_PERIOD;
> +
>   struct da9062_thermal_config {
>   	const char *name;
>   };
> @@ -95,7 +97,10 @@ static void da9062_thermal_poll_on(struct work_struct *work)
>   		thermal_zone_device_update(thermal->zone,
>   					   THERMAL_EVENT_UNSPECIFIED);
>   
> -		delay = thermal->zone->passive_delay_jiffies;
> +		/*
> +		 * pp_tmp is between 1s and 10s, so we can round the jiffies
> +		 */
> +		delay = round_jiffies(msecs_to_jiffies(pp_tmp));
>   		queue_delayed_work(system_freezable_wq, &thermal->work, delay);
>   		return;
>   	}
> @@ -155,7 +160,6 @@ static int da9062_thermal_probe(struct platform_device *pdev)
>   {
>   	struct da9062 *chip = dev_get_drvdata(pdev->dev.parent);
>   	struct da9062_thermal *thermal;
> -	unsigned int pp_tmp = DA9062_DEFAULT_POLLING_MS_PERIOD;
>   	const struct of_device_id *match;
>   	int ret = 0;
>   
> @@ -208,8 +212,7 @@ static int da9062_thermal_probe(struct platform_device *pdev)
>   	}
>   
>   	dev_dbg(&pdev->dev,
> -		"TJUNC temperature polling period set at %d ms\n",
> -		jiffies_to_msecs(thermal->zone->passive_delay_jiffies));
> +		"TJUNC temperature polling period set at %d ms\n", pp_tmp);
>   
>   	ret = platform_get_irq_byname(pdev, "THERMAL");
>   	if (ret < 0)

-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog