linux-kernel - Re: [PATCH 2/4] thermal: rcar: enable to use thermal-zone on DT

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAMuHMdUcQFcQuEmQZ4yMGwm=ouH+aQQhG3V95p=Pb56qWberQA@mail.gmail.com>
Date:	Fri, 4 Dec 2015 09:32:36 +0100
From:	Geert Uytterhoeven <geert@...ux-m68k.org>
To:	Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
Cc:	Simon <horms@...ge.net.au>, Zhang Rui <rui.zhang@...el.com>,
	Eduardo Valentin <edubezval@...il.com>,
	Magnus <magnus.damm@...il.com>,
	Linux-sh list <linux-sh@...r.kernel.org>,
	Linux PM list <linux-pm@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: Re: [PATCH 2/4] thermal: rcar: enable to use thermal-zone on DT

Hi Morimoto-san,

CC devicetree

On Fri, Dec 4, 2015 at 4:17 AM, Kuninori Morimoto
<kuninori.morimoto.gx@...esas.com> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
>
> This patch enables to use thermal-zone on DT if it was call as
> "renesas,rcar-thermal-gen2".
> Previous style is still supported by "renesas,rcar-thermal".
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
> ---
>  .../devicetree/bindings/thermal/rcar-thermal.txt   | 37 +++++++++++++++++-
>  drivers/thermal/rcar_thermal.c                     | 45 +++++++++++++++++++---
>  2 files changed, 74 insertions(+), 8 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt
> index 332e625..904f204 100644
> --- a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt
> +++ b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt
> @@ -1,8 +1,9 @@
>  * Renesas R-Car Thermal
>
>  Required properties:
> -- compatible           : "renesas,thermal-<soctype>", "renesas,rcar-thermal"
> -                         as fallback.
> +- compatible           : "renesas,thermal-<soctype>",
> +                          "renesas,rcar-thermal-gen2" (with thermal-zone) or

Ugh, another different scheme "<manuf>,<family>-<device>-<familyversion>"...

What about "renesas,rcar-gen2-thermal"?

> +                          "renesas,rcar-thermal" (without thermal-zone) as fallback.
>                           Examples with soctypes are:
>                             - "renesas,thermal-r8a73a4" (R-Mobile APE6)
>                             - "renesas,thermal-r8a7779" (R-Car H1)
> @@ -36,3 +37,35 @@ thermal@...f0000 {
>                 0xe61f0300 0x38>;
>         interrupts = <0 69 IRQ_TYPE_LEVEL_HIGH>;
>  };
> +
> +Example (with thermal-zone):
> +
> +thermal-zones {
> +       cpu_thermal: cpu-thermal {
> +               polling-delay-passive   = <1000>;
> +               polling-delay           = <5000>;
> +
> +               thermal-sensors = <&thermal>;
> +
> +               trips {
> +                       cpu-crit {
> +                               temperature     = <1150000>;
> +                               hysteresis      = <0>;
> +                               type            = "critical";
> +                       };
> +               };
> +               cooling-maps {
> +               };
> +       };
> +};
> +
> +thermal: thermal@...f0000 {
> +       compatible =    "renesas,thermal-r8a7790",
> +                       "renesas,rcar-thermal-gen2",
> +                       "renesas,rcar-thermal";
> +       reg = <0 0xe61f0000 0 0x14>, <0 0xe61f0100 0 0x38>;
> +       interrupts = <0 69 IRQ_TYPE_LEVEL_HIGH>;
> +       clocks = <&mstp5_clks R8A7790_CLK_THERMAL>;
> +       power-domains = <&cpg_clocks>;
> +       #thermal-sensor-cells = <0>;
> +};

How are the two nodes above linked?

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/