Re: [PATCH] dt-bindings: thermal: Fix definition of cooling-maps contribution property

[Rob Herring <robh@kernel.org>]

On Tue, 09 Nov 2021 11:30:45 +0100, Niklas Söderlund wrote:
> When converting the thermal-zones bindings to yaml the definition of the
> contribution property changed. The intention is the same, an integer
> value expressing a ratio of a sum on how much cooling is provided by the
> device to the zone. But after the conversion the integer value is
> limited to the range 0 to 100 and expressed as a percentage.
> 
> This is problematic for two reasons.
> 
> - This do not match how the binding is used. Out of the 18 files that
>   make use of the property only two (ste-dbx5x0.dtsi and
>   ste-hrefv60plus.dtsi) sets it at a value that satisfy the binding,
>   100. The remaining 16 files set the value higher and fail to validate.
> 
> - Expressing the value as a percentage instead of a ratio of the sum is
>   confusing as there is nothing to enforce the sum in the zone is not
>   greater then 100.
> 
> This patch restore the pre yaml conversion description and removes the
> value limitation allowing the usage of the bindings to validate.
> 
> Fixes: 1202a442a31fd2e5 ("dt-bindings: thermal: Add yaml bindings for thermal zones")
> Reported-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
>  .../devicetree/bindings/thermal/thermal-zones.yaml       | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 

Applied, thanks!