| Message ID | 20180625151239.20976-18-miquel.raynal@bootlin.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Eduardo Valentin |
| Headers | show |
On Mon, Jun 25, 2018 at 05:12:33PM +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@bootlin.com> > --- > .../arm/marvell/cp110-system-controller.txt | 37 ++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/marvell/cp110-system-controller.txt b/Documentation/devicetree/bindings/arm/marvell/cp110-system-controller.txt > index 54b0d64ce819..cd4a7a57491e 100644 > --- a/Documentation/devicetree/bindings/arm/marvell/cp110-system-controller.txt > +++ b/Documentation/devicetree/bindings/arm/marvell/cp110-system-controller.txt > @@ -11,6 +11,9 @@ For the top level node: > - compatible: must be: "syscon", "simple-mfd"; > - reg: register area of the CP110 system controller > > +SYSTEM CONTROLLER 0 > +=================== > + > Clocks: > ------- > > @@ -186,3 +189,37 @@ CP110_LABEL(syscon0): system-controller@440000 { > }; > > }; > + > +SYSTEM CONTROLLER 1 > +=================== > + > +Thermal: > +-------- > + > +The thermal IP can probe the temperature all around the processor. It > +may feature several channels, each of them wired to one sensor. > + > +For common binding part and usage, refer to > +Documentation/devicetree/bindings/thermal/thermal.txt > + > +Required properties: > +- compatible: must be one of: > + * marvell,armada-cp110-thermal > +- reg: register range associated with the thermal functions. > + > +Optional properties: > +- #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. > + > +Example: > +CP110_LABEL(syscon1): system-controller@6f8000 { > + compatible = "syscon", "simple-mfd"; > + reg = <0x6f8000 0x1000>; > + > + CP110_LABEL(thermal): ap-thermal@70 { thermal-sensor@... With that, Reviewed-by: Rob Herring <robh@kernel.org> > + compatible = "marvell,armada-cp110-thermal"; > + reg = <0x70 0x10>; > + #thermal-sensor-cells = <1>; > + }; > +}; > -- > 2.14.1 >
diff --git a/Documentation/devicetree/bindings/arm/marvell/cp110-system-controller.txt b/Documentation/devicetree/bindings/arm/marvell/cp110-system-controller.txt index 54b0d64ce819..cd4a7a57491e 100644 --- a/Documentation/devicetree/bindings/arm/marvell/cp110-system-controller.txt +++ b/Documentation/devicetree/bindings/arm/marvell/cp110-system-controller.txt @@ -11,6 +11,9 @@ For the top level node: - compatible: must be: "syscon", "simple-mfd"; - reg: register area of the CP110 system controller +SYSTEM CONTROLLER 0 +=================== + Clocks: ------- @@ -186,3 +189,37 @@ CP110_LABEL(syscon0): system-controller@440000 { }; }; + +SYSTEM CONTROLLER 1 +=================== + +Thermal: +-------- + +The thermal IP can probe the temperature all around the processor. It +may feature several channels, each of them wired to one sensor. + +For common binding part and usage, refer to +Documentation/devicetree/bindings/thermal/thermal.txt + +Required properties: +- compatible: must be one of: + * marvell,armada-cp110-thermal +- reg: register range associated with the thermal functions. + +Optional properties: +- #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. + +Example: +CP110_LABEL(syscon1): system-controller@6f8000 { + compatible = "syscon", "simple-mfd"; + reg = <0x6f8000 0x1000>; + + CP110_LABEL(thermal): ap-thermal@70 { + compatible = "marvell,armada-cp110-thermal"; + reg = <0x70 0x10>; + #thermal-sensor-cells = <1>; + }; +};
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@bootlin.com> --- .../arm/marvell/cp110-system-controller.txt | 37 ++++++++++++++++++++++ 1 file changed, 37 insertions(+)