On Wed, Feb 27, 2019 at 08:46:21AM +0000, Anson Huang wrote: > NXP i.MX8QXP is an ARMv8 SoC with a Cortex-M4 core inside as > system controller, the system controller is in charge of system > power, clock and thermal sensors etc. management, Linux kernel > has to communicate with system controller via MU (message unit) > IPC to get temperature from thermal sensors, this patch adds > binding doc for i.MX system controller thermal driver. > > Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx> > --- > Changes since V9: > - change #thermal-sensor-cells value in example to 1, since there are other > thermal sensors inside system controller, it is just because there are still > some issue, so system controller does NOT expose them for now, they could > be exposed later, so it should be 1 from HW perspective. > --- > .../devicetree/bindings/arm/freescale/fsl,scu.txt | 21 +++++++++++++++++++++ > 1 file changed, 21 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt > index 72d481c..855270b 100644 > --- a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt > +++ b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt > @@ -122,6 +122,21 @@ RTC bindings based on SCU Message Protocol > Required properties: > - compatible: should be "fsl,imx8qxp-sc-rtc"; > > +Thermal bindings based on SCU Message Protocol > +------------------------------------------------------------ > + > +Required properties: > +- compatible: Should be : > + "fsl,imx8qxp-sc-thermal" > + followed by "fsl,imx-sc-thermal"; > + > +- #thermal-sensor-cells: See Documentation/devicetree/bindings/thermal/thermal.txt > + for a description. > + > +- imx,sensor-resource-id: A single integer for single thermal zone's resource ID or > + an array of integers to specify each thermal zone's sensor > + resource ID. Can't you put the resource ids in the thermal-sensor cells? Why do you need to list them here? Rob
