Re: [PATCH v2 13/14] ARM: mvebu: Add thermal support to Armada 370 device tree

[Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>]

On 23-03-2013 18:18, Sergei Shtylyov wrote:

>> This patch adds support for the thermal controller available in
>> all Armada 370 boards. This controller has two 4-byte registers:
>> one to read the thermal sensor, the other for sensor initialization.

>> Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
>> ---
>> This patch depends on the patch to add Armada 370 support
>> in the mvebu_thermal driver:
>> 'thermal: mvebu: Add support for Armada 370 thermal sensor'

>>   arch/arm/boot/dts/armada-370.dtsi |    6 ++++++
>>   1 files changed, 6 insertions(+), 0 deletions(-)

>> diff --git a/arch/arm/boot/dts/armada-370.dtsi
>> b/arch/arm/boot/dts/armada-370.dtsi
>> index 8188d13..5831994 100644
>> --- a/arch/arm/boot/dts/armada-370.dtsi
>> +++ b/arch/arm/boot/dts/armada-370.dtsi
>> @@ -153,5 +153,11 @@
>>               clocks = <&coreclk 0>;
>>           };
>>
>> +        thermal@d0018300 {
>> +            compatible = "marvell,armada370-thermal";
>> +            reg = <0xd0018300 0x4
>> +                   0xd0018304 0x4>;

>     As the two registers are adjacent to each other, why have two register
> regions ISO one?

    Ah, I just saw the previous patch where the registers are not adjacent and 
it became clear why.

WBR, Sergei