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

From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
To: Gregory CLEMENT <gregory.clement@free-electrons.com>,
	Tyler Hall <tylerwhall@gmail.com>
Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
	devicetree@vger.kernel.org, Zhang Rui <rui.zhang@intel.com>,
	Eduardo Valentin <edubezval@gmail.com>
Subject: Re: [PATCH] thermal: armada: read stable temp on Armada XP
Date: Wed, 25 Feb 2015 15:17:41 -0300	[thread overview]
Message-ID: <54EE11C5.8000307@free-electrons.com> (raw)
In-Reply-To: <54EE009E.8070206@free-electrons.com>

On 02/25/2015 02:04 PM, Gregory CLEMENT wrote:
>>
>> 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.
-- 
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com