Hi Jacopo,

On Thu, Mar 23, 2017 at 1:41 PM, Jacopo Mondi <jacopo+rene...@jmondi.org> wrote:
> Add device nodes for two Maxim max961x current sense amplifiers
> sensing VDD_08 and DVFS_08 lines.
>
> Signed-off-by: Jacopo Mondi <jacopo+rene...@jmondi.org>
> ---
>  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 fd05c2b..ef8b723 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> @@ -261,6 +261,24 @@
>         status = "okay";
>  };
>
> +&i2c4 {
> +       status = "okay";
> +
> +       csa_vdd: max9611_vdd@7c {

csa_vdd: adc@7c {

Node names should be generic, i.e. "adc" instead of "max9611_vdd"
(also in the DT binding example).

> +               compatible = "maxim,max9611";
> +               reg = <0x7c>;
> +
> +               maxim,shunt-resistor-uohm = <5000>;
> +       };
> +
> +       csa_dvfs: max9611_dvfs@7f {

csa_dvfs: adc@7f {

> +               compatible = "maxim,max9611";
> +               reg = <0x7f>;
> +
> +               maxim,shunt-resistor-uohm = <5000>;
> +       };
> +};

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@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

Reply via email to