Re: [PATCH 1/4] dt-bindings: opp: Introduce opp-sustainable bindings

Nishanth Menon
2020-10-28T21:47:32Z

On 14:08-20201028, Lukasz Luba wrote:
> Add opp-sustainable as an additional property in the OPP node to describe
> the sustainable performance level of the device. This will help to
> estimate the sustainable performance of the whole system.
>
> Signed-off-by: Lukasz Luba <lukasz.luba@arm.com>
> ---
> Documentation/devicetree/bindings/opp/opp.txt | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/opp/opp.txt b/Documentation/devicetree/bindings/opp/opp.txt
> index 9847dfeeffcb..cd01028de305 100644
> --- a/Documentation/devicetree/bindings/opp/opp.txt
> +++ b/Documentation/devicetree/bindings/opp/opp.txt
> @@ -154,6 +154,10 @@ Optional properties:
> - opp-suspend: Marks the OPP to be used during device suspend. If multiple OPPs
> in the table have this, the OPP with highest opp-hz will be used.
>
> +- opp-sustainable: Marks the OPP as sustainable. This property can be used for
> + estimating sustainable performance of the whole system. If multiple OPPs in
> + the table have this, the OPP with highest opp-hz will be used.

By "sustainable", do you mean sustainable across Process, Voltage and
Temperature corners upto the max rated operational Power-ON hours
without IDLE state being achieved on the processor?

OR do you mean to leave it up to interpretation?
> +
> - opp-supported-hw: This property allows a platform to enable only a subset of
> the OPPs from the larger set present in the OPP table, based on the current
> version of the hardware (already known to the operating system).
> --
> 2.17.1
>

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D