Hi Amit, On 30/03/2020 12:34, Amit Kucheria wrote: [ ... ] >>> 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
