Re: [PATCH v4 0/5] thermal: fixes the rockchip thermal

[ayaka]

On 01/03/2017 09:13 AM, Randy Li wrote:
> On 01/03/2017 09:02 AM, Caesar Wang wrote:
> > å 2017å01æ03æ 07:57, Randy Li åé:
> > > On 01/02/2017 09:16 PM, Caesar Wang wrote:
> > > > å 2016å12æ31æ 00:11, ayaka åé:
> > > > > BTW, Caesar have you ever met this at RK3288 at booting time?
> > > > > [    8.430582] thermal thermal_zone1: critical temperature
> > > > > reached(125 C),shutting down
> > > > > [    8.439038] thermal thermal_zone2: critical temperature
> > > > > reached(125 C),shutting down
> > > > > [    8.456344] thermal thermal_zone1: critical temperature
> > > > > reached(125 C),shutting down
> > > > > [    8.465298] thermal thermal_zone2: critical temperature
> > > > > reached(125 C),shutting down
> > > >
> > > > 125C? the thermal zone isn't the upstream kernel, what's the kernel
> > > > version?
> > > They have been merged into the linux-next.
> >
> > Really?
> > I saw the 90 degree is the critical temperature on rk3288 dts .
> Yes I do.
> > kernel$ vi arch/arm/boot/dts/rk3288.dtsi
> >             cpu_crit: cpu_crit {
> >                     temperature = <90000>; /* millicelsius */
> >                     hysteresis = <2000>; /* millicelsius */
> >                     type = "critical";
> >                 };
> >
> > > > Anyway, look like,  the TSHUT issue. Do you have the below patches
> > > > for your linux kernel?
> > > > http://lists.infradead.org/pipermail/linux-arm-kernel/2015-October/380446.html 
> > > No, could you resubmit those patches ?
> >
> > These patches had merged for upstream.
> Sorry, it is my fault, I mistake that commit. Those patches are merged 
> in next-20161224. I would bring a board for you later.
I check the schematic, the otp interrupt is not used at firefly reload 
nor connected to the control of PMIC.
I also check power supply, the power is PD_PERI domain comes from 
VD_LOGIC which is always on all the platform.
It more likely the tsadc doesn't report correctly temperature value?
> > -Caesar