linux-kernel - Re: [PATCH] thermal: of: Allow selection of thermal governor in 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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <CAL_JsqKGXGBQSh3OeG_-bBSgv+uoANxB1CYU8RS0O6FrtaKaJQ@mail.gmail.com>
Date:   Mon, 5 Mar 2018 14:08:04 -0600
From:   Rob Herring <robh+dt@...nel.org>
To:     Amit Kucheria <amit.kucheria@...aro.org>
Cc:     devicetree@...r.kernel.org,
        Ram Chandrasekar <rkumbako@...eaurora.org>,
        Lina Iyer <ilina@...eaurora.org>,
        Zhang Rui <rui.zhang@...el.com>,
        Eduardo Valentin <edubezval@...il.com>,
        Mark Rutland <mark.rutland@....com>, linux-pm@...r.kernel.org,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] thermal: of: Allow selection of thermal governor in DT

On Mon, Mar 5, 2018 at 12:36 PM, Amit Kucheria <amit.kucheria@...aro.org> wrote:
> From: Ram Chandrasekar <rkumbako@...eaurora.org>
>
> There is currently no way for the governor to be selected for each thermal
> zone in devicetree. This results in the default governor being used for all
> thermal zones even though no such restriction exists in the core code.
>
> Add support for specifying the thermal governor to be used for a thermal
> zone in the devicetree. The devicetree config should specify the governor
> name as a string that matches any available governors. If not specified, we
> maintain the current behaviour of using the default governor.
>
> Signed-off-by: Ram Chandrasekar <rkumbako@...eaurora.org>
> Signed-off-by: Amit Kucheria <amit.kucheria@...aro.org>
> ---
>  Documentation/devicetree/bindings/thermal/thermal.txt | 8 ++++++++
>  drivers/thermal/of-thermal.c                          | 6 ++++++
>  2 files changed, 14 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/thermal/thermal.txt b/Documentation/devicetree/bindings/thermal/thermal.txt
> index 1719d47..fced9d3 100644
> --- a/Documentation/devicetree/bindings/thermal/thermal.txt
> +++ b/Documentation/devicetree/bindings/thermal/thermal.txt
> @@ -168,6 +168,14 @@ Optional property:
>                         by means of sensor ID. Additional coefficients are
>                         interpreted as constant offset.
>
> +- thermal-governor:     Thermal governor to be used for this thermal zone.
> +                       Expected values are:
> +                       "step_wise": Use step wise governor.
> +                       "fair_share": Use fair share governor.
> +                       "user_space": Use user space governor.
> +                       "power_allocator": Use power allocator governor.

This looks pretty Linux specific. Not that we can't have Linux
specific properties, but we try to avoid them.

What determines the selection? I'd imagine only certain governors make
sense for certain devices. We should perhaps describe those
characteristics which can then infer the best governor. Not really
sure though...

Rob