Message-ID: <54EE11C5.8000307@free-electrons.com>
Date: Wed, 25 Feb 2015 15:17:41 -0300
From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0
MIME-Version: 1.0
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
References: <1423608615-6575-1-git-send-email-tylerwhall@gmail.com> <54EE009E.8070206@free-electrons.com>
In-Reply-To: <54EE009E.8070206@free-electrons.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1041
Lines: 26

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/