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

From: [email-masked] (Sergei Shtylyov)
Date: Sat, 23 Mar 2013 18:18:33 +0400

Hello.

On 23-03-2013 2:25, Ezequiel Garcia 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 <[email-masked]>
> ---
> 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 at 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?

WBR, Sergei