On Wed, Aug 23, 2023 at 02:26:36PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam
>
> The temperature unit is millidegree Celsius, so divide by 1000 to correctly
> print the temperature values in Celsius.
>
> While at it, also change a typo: "has beyond" to "is beyond".
>
> Signed-off-by: Fabio Estevam
> ---
> drivers/thermal/imx_tmu.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/thermal/imx_tmu.c b/drivers/thermal/imx_tmu.c
> index d9a04eaf79..f79b583811 100644
> --- a/drivers/thermal/imx_tmu.c
> +++ b/drivers/thermal/imx_tmu.c
> @@ -238,8 +238,8 @@ int imx_tmu_get_temp(struct udevice *dev, int *temp)
> return ret;
>
> while (cpu_tmp >= pdata->alert) {
> - dev_info(dev, "CPU Temperature (%dC) has beyond alert (%dC),
> close to critical (%dC) waiting...\n",
> - cpu_tmp, pdata->alert, pdata->critical);
> + dev_info(dev, "CPU Temperature (%dC) is beyond alert (%dC),
> close to critical (%dC) waiting...\n",
> + cpu_tmp / 1000, pdata->alert / 1000, pdata->critical /
> 1000);
> mdelay(pdata->polling_delay);
> ret = read_temperature(dev, _tmp);
> if (ret)
To follow-up on the patch about select'ing LOG, this shouldn't be
dev_info, it should be higher, at least warning if not critical. And
given the default LOGLEVEL of 4, it will be included and printed.
--
Tom
signature.asc
Description: PGP signature