[PATCH 4/5] thermal: imx_tmu: Fix th temperature unit
Tom Rini
trini at konsulko.com
Wed Aug 23 19:37:10 CEST 2023
On Wed, Aug 23, 2023 at 02:26:36PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam at denx.de>
>
> 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 <festevam at denx.de>
> ---
> 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, &cpu_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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230823/99a7f03f/attachment.sig>
More information about the U-Boot
mailing list