diff mbox

[v5,4/4] arm64: dts: salvator-x: Add current sense amplifiers

Message ID 1491488454-22468-5-git-send-email-jacopo+renesas@jmondi.org (mailing list archive)
State Accepted
Commit c3dc4604131940908b922e3b3c36c3ea3ee4df55
Delegated to: Simon Horman
Headers show

Commit Message

Jacopo Mondi April 6, 2017, 2:20 p.m. UTC
Add device nodes for two Maxim max961x current sense amplifiers
sensing VDD_08 and DVFS_08 lines.

Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

Comments

Jonathan Cameron April 8, 2017, 4:02 p.m. UTC | #1
On 06/04/17 15:20, Jacopo Mondi wrote:
> Add device nodes for two Maxim max961x current sense amplifiers
> sensing VDD_08 and DVFS_08 lines.
> 
> Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Driver now applied and should make the coming merge window easily,
so I'll assume this will head upstream via an appropriate tree.

Thanks,

Jonathan
> ---
>  arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> index c7f40f8..c4860a1 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> @@ -257,6 +257,24 @@
>  	status = "okay";
>  };
>  
> +&i2c4 {
> +	status = "okay";
> +
> +	csa_vdd: adc@7c {
> +		compatible = "maxim,max9611";
> +		reg = <0x7c>;
> +
> +		shunt-resistor-micro-ohms = <5000>;
> +	};
> +
> +	csa_dvfs: adc@7f {
> +		compatible = "maxim,max9611";
> +		reg = <0x7f>;
> +
> +		shunt-resistor-micro-ohms = <5000>;
> +	};
> +};
> +
>  &wdt0 {
>  	timeout-sec = <60>;
>  	status = "okay";
>
Geert Uytterhoeven April 19, 2017, 8:40 a.m. UTC | #2
Hi Jacopo,

On Thu, Apr 6, 2017 at 4:20 PM, Jacopo Mondi <jacopo+renesas@jmondi.org> wrote:
> [PATCH v5 4/4] arm64: dts: salvator-x: Add current sense amplifiers

This should be "arm64: dts: r8a7796: salvator-x: Add current sense amplifiers".
Perhaps Simon can just fix that himself while applying?

> Add device nodes for two Maxim max961x current sense amplifiers
> sensing VDD_08 and DVFS_08 lines.
>
> Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Simon Horman April 20, 2017, 9:16 a.m. UTC | #3
On Wed, Apr 19, 2017 at 10:40:21AM +0200, Geert Uytterhoeven wrote:
> Hi Jacopo,
> 
> On Thu, Apr 6, 2017 at 4:20 PM, Jacopo Mondi <jacopo+renesas@jmondi.org> wrote:
> > [PATCH v5 4/4] arm64: dts: salvator-x: Add current sense amplifiers
> 
> This should be "arm64: dts: r8a7796: salvator-x: Add current sense amplifiers".
> Perhaps Simon can just fix that himself while applying?

Done :)
Geert Uytterhoeven May 2, 2017, 6:46 p.m. UTC | #4
Hi Simon,

On Thu, Apr 20, 2017 at 11:16 AM, Simon Horman <horms@verge.net.au> wrote:
> On Wed, Apr 19, 2017 at 10:40:21AM +0200, Geert Uytterhoeven wrote:
>> Hi Jacopo,
>>
>> On Thu, Apr 6, 2017 at 4:20 PM, Jacopo Mondi <jacopo+renesas@jmondi.org> wrote:
>> > [PATCH v5 4/4] arm64: dts: salvator-x: Add current sense amplifiers
>>
>> This should be "arm64: dts: r8a7796: salvator-x: Add current sense amplifiers".
>> Perhaps Simon can just fix that himself while applying?
>
> Done :)

Apparently not...
Resending for board breakout compensation...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
index c7f40f8..c4860a1 100644
--- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
@@ -257,6 +257,24 @@ 
 	status = "okay";
 };
 
+&i2c4 {
+	status = "okay";
+
+	csa_vdd: adc@7c {
+		compatible = "maxim,max9611";
+		reg = <0x7c>;
+
+		shunt-resistor-micro-ohms = <5000>;
+	};
+
+	csa_dvfs: adc@7f {
+		compatible = "maxim,max9611";
+		reg = <0x7f>;
+
+		shunt-resistor-micro-ohms = <5000>;
+	};
+};
+
 &wdt0 {
 	timeout-sec = <60>;
 	status = "okay";