On Sat, Apr 21, 2018 at 05:12:45PM +0200, Miquel Raynal wrote: > Explain the thermal bindings now that the thermal IP is described being > inside of a system controller. Add a reference to the thermal-zone node. > > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> > --- > .../arm/marvell/ap806-system-controller.txt | 43 ++++++++++++++++++++++ > 1 file changed, 43 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt b/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt > index a856eb9a4e05..c95f3ac5c728 100644 > --- a/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt > +++ b/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt > @@ -11,6 +11,9 @@ For the top level node: > - compatible: must be: "syscon", "simple-mfd"; > - reg: register area of the AP806 system controller > > +SYSTEM CONTROLLER 0 > +=================== > + > Clocks: > ------- > > @@ -98,3 +101,43 @@ ap_syscon: system-controller@6f4000 { > gpio-ranges = <&ap_pinctrl 0 0 19>; > }; > }; > + > +SYSTEM CONTROLLER 1 > +=================== > + > +Thermal: > +-------- > + > +For common binding part and usage, refer to > +Documentation/devicetree/bindings/thermal/thermal.txt > + > +The thermal IP can probe the temperature all around the processor. It > +may feature several channels, each of them wired to one sensor. > + > +It is possible to setup an overheat interrupt by giving at least one > +critical point to any subnode of the thermal-zone node. > + > +Required properties: > +- compatible: "marvell,armada-ap806-thermal" > + > +Optional properties: > +- interrupt-parent/interrupts: overheat interrupt handle. Should point to > + line 18 of the SEI irqchip. > + See interrupt-controller/interrupts.txt > +- #thermal-sensor-cells: shall be <1> when thermal-zones subnodes refer > + to this IP and represents the channel ID. There is one sensor per > + channel. O refers to the thermal IP internal channel, while positive > + IDs refer to each CPU. > + > +Example: > +ap_syscon1: system-controller@6f8000 { > + compatible = "syscon", "simple-mfd"; > + reg = <0x6f8000 0x1000>; > + > + ap_thermal: ap-thermal { > + compatible = "marvell,armada-ap806-thermal"; Is there a register range associated with the thermal functions? > + interrupt-parent = <&sei>; > + interrupts = <18>; > + #thermal-sensor-cells = <1>; > + }; > +}; > -- > 2.14.1 > -- 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