<DuClare>
If it were a 0-based index, 6 should refer to 816MHz? Yet I'm never going below 1GHz (generally hanging about 1200-1416 MHz) when at the 70degC trip point
<DuClare>
Also, the OPPs are in incrementing frequency
<DuClare>
While veyron-mickey dts refers to increasingly higher states as the cpu gets hotter and hotter
<DuClare>
If those were opp indices, it would be running at higher clocks when the cpu gets hotter?
<DuClare>
Is there some kind of descending sort going on behind the scenes
<DuClare>
But if it were a descending sort, then rk3288.dtsi would be allowing the highest possible CPU speed when it reaches the 70degC trip point?
<DuClare>
That is, if my interpretation were correct and THERMAL_NO_LIMIT to 6 included states 0 to 6 and 0 had highest frequency..
<DuClare>
It makes no sense to me
<DuClare>
Also the definition of THERMAL_NO_LIMIT makes no sense to me with respect to the way it's actually used
<DuClare>
Note: Using the THERMAL_NO_LIMIT (-1UL) constant in the cooling-device phandle
<DuClare>
limit specifier means:
<DuClare>
(i) - minimum state allowed for minimum cooling state used in the reference.
<DuClare>
(ii) - maximum state allowed for maximum cooling state used in the reference.
<DuClare>
so <&cooling_device THERMAL_NO_LIMIT THERMAL_NO_LIMIT> is equal to <&cooling_device MIN MAX>?
<DuClare>
So dtses are allowing all possible cooling states when the cpu gets very hot..
<DuClare>
Oh, hmm
<DuClare>
The values in veyron-mickey at least make sense if you assume descending sort
nsaenz has quit [Read error: Connection reset by peer]
nsaenz has joined #linux-rockchip
<DuClare>
At least they match the stated clocks in the comment
<DuClare>
Ok, I think this makes sense
<DuClare>
But then I don't really get the point of cooling maps
<DuClare>
If rk3288.dtsi allows states 0-6 (highest possible frequency + six steps down), then there must be another mechanism for actually deciding which frequency to choose
<DuClare>
(At the 70degC point)
<DuClare>
If there is already such a mechanism in place, why not just allow all states?
<DuClare>
Like it presumably does at 75degC point
<DuClare>
Ok I guess I get it
<DuClare>
Thanks
sigmaris has quit [Ping timeout: 246 seconds]
sigmaris has joined #linux-rockchip
field^Mop has joined #linux-rockchip
sigmaris_ has joined #linux-rockchip
sigmaris has quit [Read error: Connection reset by peer]