On Mon, Jan 04, 2016 at 03:17:09PM +0100, Sascha Hauer wrote: > On Wed, Nov 25, 2015 at 03:09:44PM +0000, Javi Merino wrote: > > The thermal-sensor property of the thermal zone node accepts phandles to > > thermal sensors. However, thermal zones can be created as an > > + > > + thermal-sensors = <&cpu_thermal &gpu_thermal &lcd_thermal> > > This seems inconsistent. Why can a thermal zone only have multiple > thermal sensors when they are thermal zones themselves? Either we assume > that one thermal zone has a single sensor or we assume that it can have > multiple sensors, but this should not depend on the zone being a sub zone > or not. > > I think the thermal-sensors property should always point to one or > multiple sensors. I see no point in "This property either points to > exactly one sensor or multiple other thermal zones (from which we only > use the temperature)" Agreed here. In fact, if we are going to allow thermal zones to be treated as sensors, it means there should be no limits on what you put of there, as long as all items have #thermal-sensor-cells. So, mixing one (or more) regular sensors, with other thermal zones shall be allowed, if we agree in this semantics. > > Sascha > > > -- > Pengutronix e.K. | | > Industrial Linux Solutions | http://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html