Hi Amit,

On 30/03/2020 12:34, Amit Kucheria wrote:

[ ... ]

quoted

quoted

I don't know why it's not consistent with the actual code in
of-thermal.c, where there is even a comment stated: /* For now,
thermal framework supports only 1 sensor per zone */

I think this is the place where developers should be informed
about the limitation and not even try to put more sensors into
the list.

That is a good point. I'm currently "porting" the existing
binding as described in thermal.txt to yaml. If you look at some
of the example (c) in there, the bindings allow many sensors to a
zone mapping but the thermal core doesn't implement that
functionality.

So should we fix the core code or change the bindings? Thoughts -
Rob, Daniel, Rui?

Rob, Daniel: Any comments? We don't have any concerns for Linux
backward compatibility since multiple sensors per zone isn't used
anywhere. But asking since bindings are supposed to be
OS-agnostic.

IMO, we should remove it as it is not used anywhere.

We still have to decide how we aggregate multiple sensors.


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog