On Tue, Jun 9, 2015 at 6:40 AM, Tim Harvey <thar...@gateworks.com> wrote: > Remove the check for temperature being within the min/max range and enter the > busywait cooling loop whenever the CPU temperature is over the critical temp. > This fixes the issue where if a board was booted at a temp greater than > the CPU temperature max, it would skip the loop and never indicate or try > to address the overtemp issue. > > Cc: Ye Li <b37...@freescale.com> > Cc: Jason Liu <r64...@freescale.com> > Signed-off-by: Tim Harvey <thar...@gateworks.com> > --- > drivers/thermal/imx_thermal.c | 16 ++++++---------- > 1 file changed, 6 insertions(+), 10 deletions(-) > > diff --git a/drivers/thermal/imx_thermal.c b/drivers/thermal/imx_thermal.c > index 0d893c9..42ca8d0 100644 > --- a/drivers/thermal/imx_thermal.c > +++ b/drivers/thermal/imx_thermal.c > @@ -130,16 +130,12 @@ int imx_thermal_get_temp(struct udevice *dev, int *temp) > int cpu_tmp = 0; > > cpu_tmp = read_cpu_temperature(dev); > - while (cpu_tmp > priv->minc && cpu_tmp < priv->maxc) { > - if (cpu_tmp >= priv->critical) { > - printf("CPU Temperature (%dC) too close to max (%dC)", > - cpu_tmp, priv->maxc); > - puts(" waiting...\n"); > - udelay(5000000); > - cpu_tmp = read_cpu_temperature(dev); > - } else { > - break; > - } > + while (cpu_tmp >= priv->critical) { > + printf("CPU Temperature (%dC) too close to max (%dC)", > + cpu_tmp, priv->maxc); > + puts(" waiting...\n"); > + udelay(5000000); > + cpu_tmp = read_cpu_temperature(dev); > } > > *temp = cpu_tmp; > -- > 1.9.1 >
Stefano, Any feedback on this? Regards, Tim _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot