[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAMuHMdX5UuFTwXHY08DJyGS_deK5iyf1QkGW1OeG5QPD4K3k+A@mail.gmail.com>
Date: Mon, 20 Jun 2016 09:49:38 +0200
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Khiem Nguyen <khiem.nguyen.xt@....renesas.com>
Cc: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>,
Wolfram Sang <wsa@...-dreams.de>,
Geert Uytterhoeven <geert+renesas@...der.be>,
Magnus Damm <damm+renesas@...nsource.se>,
Zhang Rui <rui.zhang@...el.com>,
Eduardo Valentin <edubezval@...il.com>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"Thao Phuong Le. Nguyen" <thao.nguyen.yb@....renesas.com>,
"Hien Duy. Dang" <hien.dang.eb@....renesas.com>,
Toru Oishi <toru.oishi.zj@....renesas.com>
Subject: Re: [PATCH 1/4] thermal: rcar_gen3_thermal: Document the R-Car Gen3
thermal bindings
Hi Khiem,
On Sun, Jun 19, 2016 at 5:33 AM, Khiem Nguyen
<khiem.nguyen.xt@....renesas.com> wrote:
>
> Signed-off-by: Hien Dang <hien.dang.eb@....renesas.com>
> Signed-off-by: Khiem Nguyen <khiem.nguyen.xt@....renesas.com>
Thanks for your patch!
> ---
> .../bindings/thermal/rcar-gen3-thermal.txt | 79 ++++++++++++++++++++++
> 1 file changed, 79 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/thermal/rcar-gen3-thermal.txt
>
> diff --git a/Documentation/devicetree/bindings/thermal/rcar-gen3-thermal.txt b/Documentation/devicetree/bindings/thermal/rcar-gen3-thermal.txt
> new file mode 100644
> index 0000000..ed6ce45
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/thermal/rcar-gen3-thermal.txt
> @@ -0,0 +1,79 @@
> +* DT bindings for Renesas R-Car Gen3 Thermal Sensor driver
> +
> +Required properties:
> +- compatible : "renesas,thermal-<soctype>",
Please use "renesas,<soctype>-thermal", as this is the preferred order for
new bindings.
> + Examples with soctypes are:
> + - "renesas,thermal-r8a7795" (R-Car H3)
renesas,r8a7795-thermal
> + - "renesas,thermal-r8a7796" (R-Car M3)
renesas,r8a7796-thermal, M3-W
> + - "renesas,rcar-gen3-thermal" as fallback
> +- reg : Address range of the thermal registers.
> +- clocks : Must contain a reference to the functional clock.
According to the datasheet, the USB_EXTAL clock is also an input for the
thermal block. Should it be added?
> +- #thermal-sensor-cells : Please see ./thermal.txt
> +
> +Option properties:
Optional
> +
> +- interrupts : Use interrupt
> +- power-domain : Must contain a reference to the power domain. This property is
> + mandatory if the thermal sensor instance is part of a controllable power
> + domain.
> +
> +Example (non interrupt support):
Is there a good reason why you wouldn't want to use interrupts?
If not, please make it mandatory.
> +
> + tsc1: thermal@...98000 {
> + compatible = "renesas,thermal-r8a7795",
> + "renesas,rcar-gen3-thermal";
> + reg = <0 0xe6198000 0 0x5c>;
According to the datasheet, there are more registers beyond this range.
Shouldn't it be "reg = <0 0xe6198000 0 0x68>;"?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-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
Powered by blists - more mailing lists