Thread (10 messages) 10 messages, 4 authors, 2015-02-25

Re: [PATCH] thermal: armada: read stable temp on Armada XP

From: Gregory CLEMENT <hidden>
Date: 2015-02-25 18:38:38
Also in: linux-pm, lkml

Hi Ezequiel,

On 25/02/2015 19:17, Ezequiel Garcia wrote:
On 02/25/2015 02:04 PM, Gregory CLEMENT wrote:
quoted
quoted
My conclusions about these registers are based on experimental data. The
documentation is very sparse, but the Thermal Manager Control and Status
Register looks like the preferred register given the way it is laid out in the
public spec.
Ezequiel,

as you worked on this do you know why we used the Thermal Sensor Status Register
instead of the Thermal Manager Control and Status Register ?
My first guess is that the giving the name of the registers the 1st one made
more sens to use for a thermal sensor.
Actually, we based this driver in the vendor bootloader. The specs weren't
of much use back then.

Thanks for your prompt feedback. So we don't have much more information about
the sensor. :(

I will try to get information from Marvell.


Tyler,
In the meantime could you double check your values? The temperature on my board
seemed broken on my board. If needed I can check on an other board. By the way
on which board/product did you try it?


Thanks,

Gregory

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help