Message ID | 1429085868-47577-1-git-send-email-hongtao.jia@freescale.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers | show |
Hi Scott, Eduardo indicated that "calibration" property should add "fsl" prefix. I updated the patch. Any other comments? If not I will send V3 soon. Thanks. --- Best Regards, Hongtao > -----Original Message----- > From: Jia Hongtao [mailto:hongtao.jia@freescale.com] > Sent: Wednesday, April 15, 2015 4:18 PM > To: Wood Scott-B07421 > Cc: linuxppc-dev@lists.ozlabs.org; devicetree@vger.kernel.org; > robh+dt@kernel.org; rui.zhang@intel.com; Jia Hongtao-B38951 > Subject: [PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ > T104x > > This is Thermal Monitoring Unit for QorIQ platform. > > Signed-off-by: Jia Hongtao <hongtao.jia@freescale.com> > --- > Changes from V1: > * Provide more details on compatible and calibration properties. > > .../devicetree/bindings/thermal/qoriq-thermal.txt | 68 > ++++++++++++++++++ > arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi | 82 > ++++++++++++++++++++++ > arch/powerpc/boot/dts/fsl/t1040si-post.dtsi | 1 + > 3 files changed, 151 insertions(+) > create mode 100644 Documentation/devicetree/bindings/thermal/qoriq- > thermal.txt > create mode 100644 arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > > diff --git a/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt > b/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt > new file mode 100644 > index 0000000..48e7fff > --- /dev/null > +++ b/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt > @@ -0,0 +1,68 @@ > +* Thermal Monitoring Unit (TMU) on Freescale QorIQ SoCs > + > +Required properties: > +- compatible : Must include "fsl,qoriq-tmu". The version of the device > is > + determined by the TMU IP Block Revision Register (IPBRR0) at > + offset 0x0BF8. > + Table of correspondences between IPBRR0 values and example chips: > + Value Device > + ---------- ----- > + 0x01900102 T1040 > +- reg : Address range of TMU registers. > +- interrupts : Contains the interrupt for TMU. > +- calibration : Calibration table for TMU. This is required to determine > + what a specific sensor reading translates to in degrees Celsius. > + There are four calibration config ranges, for each range we need > + a couple of calibration data formatted like <TTCFGR, TSCFGR>. > + Freescale provides the data required. > + > +Example: > + > +tmu@f0000 { > + compatible = "fsl,qoriq-tmu"; > + reg = <0xf0000 0x1000>; > + interrupts = <18 2 0 0>; > + calibration = <0x00000000 0x0000000f > + 0x00000001 0x00000017 > + 0x00000002 0x0000001e > + 0x00000003 0x00000026 > + 0x00000004 0x0000002e > + 0x00000005 0x00000035 > + 0x00000006 0x0000003d > + 0x00000007 0x00000044 > + 0x00000008 0x0000004c > + 0x00000009 0x00000053 > + 0x0000000a 0x0000005b > + 0x0000000b 0x00000062 > + > + 0x00010000 0x00000011 > + 0x00010001 0x00000019 > + 0x00010002 0x00000021 > + 0x00010003 0x0000002a > + 0x00010004 0x00000032 > + 0x00010005 0x0000003a > + 0x00010006 0x00000042 > + 0x00010007 0x0000004b > + 0x00010008 0x00000053 > + 0x00010009 0x0000005b > + 0x0001000a 0x00000063 > + > + 0x00020000 0x00000012 > + 0x00020001 0x0000001b > + 0x00020002 0x00000024 > + 0x00020003 0x0000002c > + 0x00020004 0x00000035 > + 0x00020005 0x0000003d > + 0x00020006 0x00000046 > + 0x00020007 0x0000004e > + 0x00020008 0x00000057 > + > + 0x00030000 0x00000009 > + 0x00030001 0x00000011 > + 0x00030002 0x00000019 > + 0x00030003 0x00000021 > + 0x00030004 0x00000029 > + 0x00030005 0x00000031 > + 0x00030006 0x00000039 > + 0x00030007 0x00000041>; > +}; > diff --git a/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > b/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > new file mode 100644 > index 0000000..d85c77d > --- /dev/null > +++ b/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > @@ -0,0 +1,82 @@ > +/* > + * QorIQ TMU device tree stub [ controller @ offset 0xf0000 ] > + * > + * Copyright 2015 Freescale Semiconductor Inc. > + * > + * Redistribution and use in source and binary forms, with or without > + * modification, are permitted provided that the following conditions > are met: > + * * Redistributions of source code must retain the above copyright > + * notice, this list of conditions and the following disclaimer. > + * * Redistributions in binary form must reproduce the above > copyright > + * notice, this list of conditions and the following disclaimer in > the > + * documentation and/or other materials provided with the > distribution. > + * * Neither the name of Freescale Semiconductor nor the > + * names of its contributors may be used to endorse or promote > products > + * derived from this software without specific prior written > permission. > + * > + * > + * ALTERNATIVELY, this software may be distributed under the terms of > the > + * GNU General Public License ("GPL") as published by the Free Software > + * Foundation, either version 2 of that License or (at your option) any > + * later version. > + * > + * THIS SOFTWARE IS PROVIDED BY Freescale Semiconductor ``AS IS'' AND > ANY > + * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE > IMPLIED > + * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE > ARE > + * DISCLAIMED. IN NO EVENT SHALL Freescale Semiconductor BE LIABLE FOR > ANY > + * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL > DAMAGES > + * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR > SERVICES; > + * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER > CAUSED AND > + * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR > TORT > + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE > OF THIS > + * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > + */ > + > +tmu@f0000 { > + compatible = "fsl,qoriq-tmu"; > + reg = <0xf0000 0x1000>; > + interrupts = <18 2 0 0>; > + calibration = <0x00000000 0x0000000f > + 0x00000001 0x00000017 > + 0x00000002 0x0000001e > + 0x00000003 0x00000026 > + 0x00000004 0x0000002e > + 0x00000005 0x00000035 > + 0x00000006 0x0000003d > + 0x00000007 0x00000044 > + 0x00000008 0x0000004c > + 0x00000009 0x00000053 > + 0x0000000a 0x0000005b > + 0x0000000b 0x00000062 > + > + 0x00010000 0x00000011 > + 0x00010001 0x00000019 > + 0x00010002 0x00000021 > + 0x00010003 0x0000002a > + 0x00010004 0x00000032 > + 0x00010005 0x0000003a > + 0x00010006 0x00000042 > + 0x00010007 0x0000004b > + 0x00010008 0x00000053 > + 0x00010009 0x0000005b > + 0x0001000a 0x00000063 > + > + 0x00020000 0x00000012 > + 0x00020001 0x0000001b > + 0x00020002 0x00000024 > + 0x00020003 0x0000002c > + 0x00020004 0x00000035 > + 0x00020005 0x0000003d > + 0x00020006 0x00000046 > + 0x00020007 0x0000004e > + 0x00020008 0x00000057 > + > + 0x00030000 0x00000009 > + 0x00030001 0x00000011 > + 0x00030002 0x00000019 > + 0x00030003 0x00000021 > + 0x00030004 0x00000029 > + 0x00030005 0x00000031 > + 0x00030006 0x00000039 > + 0x00030007 0x00000041>; > +}; > diff --git a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > index 15ae462..bb9f792 100644 > --- a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > +++ b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > @@ -345,6 +345,7 @@ > reg = <0xea000 0x4000>; > }; > > +/include/ "qoriq-tmu-t104xsi.dtsi" > /include/ "elo3-dma-0.dtsi" > /include/ "elo3-dma-1.dtsi" > /include/ "qoriq-espi-0.dtsi" > -- > 2.1.0.27.g96db324
On Tue, 2015-05-26 at 01:33 -0500, Jia Hongtao-B38951 wrote: > Hi Scott, > > Eduardo indicated that "calibration" property should add "fsl" > prefix. > I updated the patch. > Any other comments? > If not I will send V3 soon. > > Thanks. I'd go with "fsl,tmu-calibration" rather than "fsl,calibration" but not a big deal either way. > > diff --git a/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > > b/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > > new file mode 100644 > > index 0000000..d85c77d > > --- /dev/null > > +++ b/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > > @@ -0,0 +1,82 @@ > > +/* > > + * QorIQ TMU device tree stub [ controller @ offset 0xf0000 ] > > + * > > + * Copyright 2015 Freescale Semiconductor Inc. > > + * > > + * Redistribution and use in source and binary forms, with or > > without > > + * modification, are permitted provided that the following > > conditions > > are met: > > + * * Redistributions of source code must retain the above > > copyright > > + * notice, this list of conditions and the following > > disclaimer. > > + * * Redistributions in binary form must reproduce the above > > copyright > > + * notice, this list of conditions and the following > > disclaimer in > > the > > + * documentation and/or other materials provided with the > > distribution. > > + * * Neither the name of Freescale Semiconductor nor the > > + * names of its contributors may be used to endorse or > > promote > > products > > + * derived from this software without specific prior written > > permission. > > + * > > + * > > + * ALTERNATIVELY, this software may be distributed under the > > terms of > > the > > + * GNU General Public License ("GPL") as published by the Free > > Software > > + * Foundation, either version 2 of that License or (at your > > option) any > > + * later version. > > + * > > + * THIS SOFTWARE IS PROVIDED BY Freescale Semiconductor ``AS IS'' > > AND > > ANY > > + * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, > > THE > > IMPLIED > > + * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR > > PURPOSE > > ARE > > + * DISCLAIMED. IN NO EVENT SHALL Freescale Semiconductor BE > > LIABLE FOR > > ANY > > + * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR > > CONSEQUENTIAL > > DAMAGES > > + * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE > > GOODS OR > > SERVICES; > > + * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) > > HOWEVER > > CAUSED AND > > + * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT > > LIABILITY, OR > > TORT > > + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF > > THE USE > > OF THIS > > + * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > > + */ > > + > > +tmu@f0000{ > > + compatible = "fsl,qoriq-tmu"; > > + reg = <0xf0000 0x1000>; > > + interrupts = <18 2 0 0>; > > + calibration = <0x00000000 0x0000000f > > + 0x00000001 0x00000017 > > + 0x00000002 0x0000001e > > + 0x00000003 0x00000026 > > + 0x00000004 0x0000002e > > + 0x00000005 0x00000035 > > + 0x00000006 0x0000003d > > + 0x00000007 0x00000044 > > + 0x00000008 0x0000004c > > + 0x00000009 0x00000053 > > + 0x0000000a 0x0000005b > > + 0x0000000b 0x00000062 > > + > > + 0x00010000 0x00000011 > > + 0x00010001 0x00000019 > > + 0x00010002 0x00000021 > > + 0x00010003 0x0000002a > > + 0x00010004 0x00000032 > > + 0x00010005 0x0000003a > > + 0x00010006 0x00000042 > > + 0x00010007 0x0000004b > > + 0x00010008 0x00000053 > > + 0x00010009 0x0000005b > > + 0x0001000a 0x00000063 > > + > > + 0x00020000 0x00000012 > > + 0x00020001 0x0000001b > > + 0x00020002 0x00000024 > > + 0x00020003 0x0000002c > > + 0x00020004 0x00000035 > > + 0x00020005 0x0000003d > > + 0x00020006 0x00000046 > > + 0x00020007 0x0000004e > > + 0x00020008 0x00000057 > > + > > + 0x00030000 0x00000009 > > + 0x00030001 0x00000011 > > + 0x00030002 0x00000019 > > + 0x00030003 0x00000021 > > + 0x00030004 0x00000029 > > + 0x00030005 0x00000031 > > + 0x00030006 0x00000039 > > + 0x00030007 0x00000041>; > > +}; > > diff --git a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > index 15ae462..bb9f792 100644 > > --- a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > +++ b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > @@ -345,6 +345,7 @@ > > reg = <0xea000 0x4000>; > > }; > > > > +/include/ "qoriq-tmu-t104xsi.dtsi" > > /include/ "elo3-dma-0.dtsi" > > /include/ "elo3-dma-1.dtsi" > > /include/ "qoriq-espi-0.dtsi" Why is this chip-specific enough to be called "t104xsi", but not chip- specific enough to just go directly into t1040si-post? -SCott
> -----Original Message----- > From: Wood Scott-B07421 > Sent: Wednesday, May 27, 2015 3:38 AM > To: Jia Hongtao-B38951 > Cc: edubezval@gmail.com; linuxppc-dev@lists.ozlabs.org; > devicetree@vger.kernel.org; robh+dt@kernel.org; rui.zhang@intel.com > Subject: Re: [PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ > T104x > > On Tue, 2015-05-26 at 01:33 -0500, Jia Hongtao-B38951 wrote: > > Hi Scott, > > > > Eduardo indicated that "calibration" property should add "fsl" > > prefix. > > I updated the patch. > > Any other comments? > > If not I will send V3 soon. > > > > Thanks. > > I'd go with "fsl,tmu-calibration" rather than "fsl,calibration" but not a > big deal either way. Thanks for the advice. > > > > diff --git a/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > > > b/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > > > new file mode 100644 > > > index 0000000..d85c77d > > > --- /dev/null > > > +++ b/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi > > > @@ -0,0 +1,82 @@ > > > +/* > > > + * QorIQ TMU device tree stub [ controller @ offset 0xf0000 ] > > > + * > > > + * Copyright 2015 Freescale Semiconductor Inc. > > > + * > > > + * Redistribution and use in source and binary forms, with or > > > without > > > + * modification, are permitted provided that the following > > > conditions > > > are met: > > > + * * Redistributions of source code must retain the above > > > copyright > > > + * notice, this list of conditions and the following > > > disclaimer. > > > + * * Redistributions in binary form must reproduce the above > > > copyright > > > + * notice, this list of conditions and the following > > > disclaimer in > > > the > > > + * documentation and/or other materials provided with the > > > distribution. > > > + * * Neither the name of Freescale Semiconductor nor the > > > + * names of its contributors may be used to endorse or > > > promote > > > products > > > + * derived from this software without specific prior written > > > permission. > > > + * > > > + * > > > + * ALTERNATIVELY, this software may be distributed under the > > > terms of > > > the > > > + * GNU General Public License ("GPL") as published by the Free > > > Software > > > + * Foundation, either version 2 of that License or (at your > > > option) any > > > + * later version. > > > + * > > > + * THIS SOFTWARE IS PROVIDED BY Freescale Semiconductor ``AS IS'' > > > AND > > > ANY > > > + * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, > > > THE > > > IMPLIED > > > + * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR > > > PURPOSE > > > ARE > > > + * DISCLAIMED. IN NO EVENT SHALL Freescale Semiconductor BE > > > LIABLE FOR > > > ANY > > > + * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR > > > CONSEQUENTIAL > > > DAMAGES > > > + * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE > > > GOODS OR > > > SERVICES; > > > + * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) > > > HOWEVER > > > CAUSED AND > > > + * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT > > > LIABILITY, OR > > > TORT > > > + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF > > > THE USE > > > OF THIS > > > + * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > > > + */ > > > + > > > +tmu@f0000{ > > > + compatible = "fsl,qoriq-tmu"; > > > + reg = <0xf0000 0x1000>; > > > + interrupts = <18 2 0 0>; > > > + calibration = <0x00000000 0x0000000f > > > + 0x00000001 0x00000017 > > > + 0x00000002 0x0000001e > > > + 0x00000003 0x00000026 > > > + 0x00000004 0x0000002e > > > + 0x00000005 0x00000035 > > > + 0x00000006 0x0000003d > > > + 0x00000007 0x00000044 > > > + 0x00000008 0x0000004c > > > + 0x00000009 0x00000053 > > > + 0x0000000a 0x0000005b > > > + 0x0000000b 0x00000062 > > > + > > > + 0x00010000 0x00000011 > > > + 0x00010001 0x00000019 > > > + 0x00010002 0x00000021 > > > + 0x00010003 0x0000002a > > > + 0x00010004 0x00000032 > > > + 0x00010005 0x0000003a > > > + 0x00010006 0x00000042 > > > + 0x00010007 0x0000004b > > > + 0x00010008 0x00000053 > > > + 0x00010009 0x0000005b > > > + 0x0001000a 0x00000063 > > > + > > > + 0x00020000 0x00000012 > > > + 0x00020001 0x0000001b > > > + 0x00020002 0x00000024 > > > + 0x00020003 0x0000002c > > > + 0x00020004 0x00000035 > > > + 0x00020005 0x0000003d > > > + 0x00020006 0x00000046 > > > + 0x00020007 0x0000004e > > > + 0x00020008 0x00000057 > > > + > > > + 0x00030000 0x00000009 > > > + 0x00030001 0x00000011 > > > + 0x00030002 0x00000019 > > > + 0x00030003 0x00000021 > > > + 0x00030004 0x00000029 > > > + 0x00030005 0x00000031 > > > + 0x00030006 0x00000039 > > > + 0x00030007 0x00000041>; }; > > > diff --git a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > index 15ae462..bb9f792 100644 > > > --- a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > +++ b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > @@ -345,6 +345,7 @@ > > > reg = <0xea000 0x4000>; > > > }; > > > > > > +/include/ "qoriq-tmu-t104xsi.dtsi" > > > /include/ "elo3-dma-0.dtsi" > > > /include/ "elo3-dma-1.dtsi" > > > /include/ "qoriq-espi-0.dtsi" > > Why is this chip-specific enough to be called "t104xsi", but not chip- > specific enough to just go directly into t1040si-post? T104x platform share the same calibration data. -Hongtao. > > -SCott >
On Mon, 2015-06-08 at 21:19 -0500, Jia Hongtao-B38951 wrote: > > -----Original Message----- > > From: Wood Scott-B07421 > > Sent: Wednesday, May 27, 2015 3:38 AM > > To: Jia Hongtao-B38951 > > Cc: edubezval@gmail.com; linuxppc-dev@lists.ozlabs.org; > > devicetree@vger.kernel.org; robh+dt@kernel.org; rui.zhang@intel.com > > Subject: Re: [PATCH V2] QorIQ/TMU: add TMU node to device tree for > > QorIQ > > T104x > > > > On Tue, 2015-05-26 at 01:33 -0500, Jia Hongtao-B38951 wrote: > > > diff --git a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > index 15ae462..bb9f792 100644 > > > > --- a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > +++ b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > @@ -345,6 +345,7 @@ > > > > reg = <0xea000 0x4000>; > > > > }; > > > > > > > > +/include/ "qoriq-tmu-t104xsi.dtsi" > > > > /include/ "elo3-dma-0.dtsi" > > > > /include/ "elo3-dma-1.dtsi" > > > > /include/ "qoriq-espi-0.dtsi" > > > > Why is this chip-specific enough to be called "t104xsi", but not > > chip- > > specific enough to just go directly into t1040si-post? > > T104x platform share the same calibration data. Which t104x are you referring to that doesn't use t1040si-post.dtsi? Plus, given the numbering scheme it's always possible that a new t104x comes out that has little to do with existing t104x. -Scott
> -----Original Message----- > From: Wood Scott-B07421 > Sent: Wednesday, June 10, 2015 1:47 AM > To: Jia Hongtao-B38951 > Cc: edubezval@gmail.com; linuxppc-dev@lists.ozlabs.org; > devicetree@vger.kernel.org; robh+dt@kernel.org; rui.zhang@intel.com > Subject: Re: [PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ > T104x > > On Mon, 2015-06-08 at 21:19 -0500, Jia Hongtao-B38951 wrote: > > > -----Original Message----- > > > From: Wood Scott-B07421 > > > Sent: Wednesday, May 27, 2015 3:38 AM > > > To: Jia Hongtao-B38951 > > > Cc: edubezval@gmail.com; linuxppc-dev@lists.ozlabs.org; > > > devicetree@vger.kernel.org; robh+dt@kernel.org; rui.zhang@intel.com > > > Subject: Re: [PATCH V2] QorIQ/TMU: add TMU node to device tree for > > > QorIQ T104x > > > > > > On Tue, 2015-05-26 at 01:33 -0500, Jia Hongtao-B38951 wrote: > > > > diff --git a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > > b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > > index 15ae462..bb9f792 100644 > > > > > --- a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > > +++ b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > > @@ -345,6 +345,7 @@ > > > > > reg = <0xea000 0x4000>; > > > > > }; > > > > > > > > > > +/include/ "qoriq-tmu-t104xsi.dtsi" > > > > > /include/ "elo3-dma-0.dtsi" > > > > > /include/ "elo3-dma-1.dtsi" > > > > > /include/ "qoriq-espi-0.dtsi" > > > > > > Why is this chip-specific enough to be called "t104xsi", but not > > > chip- > > > specific enough to just go directly into t1040si-post? The tmu node includes lots of contents. Is it looks messy if just go directly into t1040si-post? > > > > T104x platform share the same calibration data. > > Which t104x are you referring to that doesn't use t1040si-post.dtsi? > Plus, given the numbering scheme it's always possible that a new t104x > comes out that has little to do with existing t104x. Yes, right. The name contains "t104x" keyword seems not perfect. I'd like to change "qoriq-tmu-t104xsi.dtsi" to "qoriq-tmu-t1040si.dtsi" and included in t1040si-post. Is this OK with you? -Hongtao. > > -Scott >
On Wed, 2015-06-10 at 03:56 -0500, Jia Hongtao-B38951 wrote: > > -----Original Message----- > > From: Wood Scott-B07421 > > Sent: Wednesday, June 10, 2015 1:47 AM > > To: Jia Hongtao-B38951 > > Cc: edubezval@gmail.com; linuxppc-dev@lists.ozlabs.org; > > devicetree@vger.kernel.org; robh+dt@kernel.org; rui.zhang@intel.com > > Subject: Re: [PATCH V2] QorIQ/TMU: add TMU node to device tree for > > QorIQ > > T104x > > > > On Mon, 2015-06-08 at 21:19 -0500, Jia Hongtao-B38951 wrote: > > > > -----Original Message----- > > > > From: Wood Scott-B07421 > > > > Sent: Wednesday, May 27, 2015 3:38 AM > > > > To: Jia Hongtao-B38951 > > > > Cc: edubezval@gmail.com; linuxppc-dev@lists.ozlabs.org; > > > > devicetree@vger.kernel.org; robh+dt@kernel.org; > > > > rui.zhang@intel.com > > > > Subject: Re: [PATCH V2] QorIQ/TMU: add TMU node to device tree > > > > for > > > > QorIQ T104x > > > > > > > > On Tue, 2015-05-26 at 01:33 -0500, Jia Hongtao-B38951 wrote: > > > > > diff --git a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > > > b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > > > index 15ae462..bb9f792 100644 > > > > > > --- a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > > > +++ b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi > > > > > > @@ -345,6 +345,7 @@ > > > > > > reg = <0xea000 0x4000>; > > > > > > }; > > > > > > > > > > > > +/include/ "qoriq-tmu-t104xsi.dtsi" > > > > > > /include/ "elo3-dma-0.dtsi" > > > > > > /include/ "elo3-dma-1.dtsi" > > > > > > /include/ "qoriq-espi-0.dtsi" > > > > > > > > Why is this chip-specific enough to be called "t104xsi", but > > > > not > > > > chip- > > > > specific enough to just go directly into t1040si-post? > > The tmu node includes lots of contents. > Is it looks messy if just go directly into t1040si-post? I don't think it's too much to go into the main file. Having too many files is a hassle as well... -Scott
diff --git a/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt b/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt new file mode 100644 index 0000000..48e7fff --- /dev/null +++ b/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt @@ -0,0 +1,68 @@ +* Thermal Monitoring Unit (TMU) on Freescale QorIQ SoCs + +Required properties: +- compatible : Must include "fsl,qoriq-tmu". The version of the device is + determined by the TMU IP Block Revision Register (IPBRR0) at + offset 0x0BF8. + Table of correspondences between IPBRR0 values and example chips: + Value Device + ---------- ----- + 0x01900102 T1040 +- reg : Address range of TMU registers. +- interrupts : Contains the interrupt for TMU. +- calibration : Calibration table for TMU. This is required to determine + what a specific sensor reading translates to in degrees Celsius. + There are four calibration config ranges, for each range we need + a couple of calibration data formatted like <TTCFGR, TSCFGR>. + Freescale provides the data required. + +Example: + +tmu@f0000 { + compatible = "fsl,qoriq-tmu"; + reg = <0xf0000 0x1000>; + interrupts = <18 2 0 0>; + calibration = <0x00000000 0x0000000f + 0x00000001 0x00000017 + 0x00000002 0x0000001e + 0x00000003 0x00000026 + 0x00000004 0x0000002e + 0x00000005 0x00000035 + 0x00000006 0x0000003d + 0x00000007 0x00000044 + 0x00000008 0x0000004c + 0x00000009 0x00000053 + 0x0000000a 0x0000005b + 0x0000000b 0x00000062 + + 0x00010000 0x00000011 + 0x00010001 0x00000019 + 0x00010002 0x00000021 + 0x00010003 0x0000002a + 0x00010004 0x00000032 + 0x00010005 0x0000003a + 0x00010006 0x00000042 + 0x00010007 0x0000004b + 0x00010008 0x00000053 + 0x00010009 0x0000005b + 0x0001000a 0x00000063 + + 0x00020000 0x00000012 + 0x00020001 0x0000001b + 0x00020002 0x00000024 + 0x00020003 0x0000002c + 0x00020004 0x00000035 + 0x00020005 0x0000003d + 0x00020006 0x00000046 + 0x00020007 0x0000004e + 0x00020008 0x00000057 + + 0x00030000 0x00000009 + 0x00030001 0x00000011 + 0x00030002 0x00000019 + 0x00030003 0x00000021 + 0x00030004 0x00000029 + 0x00030005 0x00000031 + 0x00030006 0x00000039 + 0x00030007 0x00000041>; +}; diff --git a/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi b/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi new file mode 100644 index 0000000..d85c77d --- /dev/null +++ b/arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi @@ -0,0 +1,82 @@ +/* + * QorIQ TMU device tree stub [ controller @ offset 0xf0000 ] + * + * Copyright 2015 Freescale Semiconductor Inc. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions are met: + * * Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * * Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution. + * * Neither the name of Freescale Semiconductor nor the + * names of its contributors may be used to endorse or promote products + * derived from this software without specific prior written permission. + * + * + * ALTERNATIVELY, this software may be distributed under the terms of the + * GNU General Public License ("GPL") as published by the Free Software + * Foundation, either version 2 of that License or (at your option) any + * later version. + * + * THIS SOFTWARE IS PROVIDED BY Freescale Semiconductor ``AS IS'' AND ANY + * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED + * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE + * DISCLAIMED. IN NO EVENT SHALL Freescale Semiconductor BE LIABLE FOR ANY + * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES + * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; + * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND + * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS + * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + */ + +tmu@f0000 { + compatible = "fsl,qoriq-tmu"; + reg = <0xf0000 0x1000>; + interrupts = <18 2 0 0>; + calibration = <0x00000000 0x0000000f + 0x00000001 0x00000017 + 0x00000002 0x0000001e + 0x00000003 0x00000026 + 0x00000004 0x0000002e + 0x00000005 0x00000035 + 0x00000006 0x0000003d + 0x00000007 0x00000044 + 0x00000008 0x0000004c + 0x00000009 0x00000053 + 0x0000000a 0x0000005b + 0x0000000b 0x00000062 + + 0x00010000 0x00000011 + 0x00010001 0x00000019 + 0x00010002 0x00000021 + 0x00010003 0x0000002a + 0x00010004 0x00000032 + 0x00010005 0x0000003a + 0x00010006 0x00000042 + 0x00010007 0x0000004b + 0x00010008 0x00000053 + 0x00010009 0x0000005b + 0x0001000a 0x00000063 + + 0x00020000 0x00000012 + 0x00020001 0x0000001b + 0x00020002 0x00000024 + 0x00020003 0x0000002c + 0x00020004 0x00000035 + 0x00020005 0x0000003d + 0x00020006 0x00000046 + 0x00020007 0x0000004e + 0x00020008 0x00000057 + + 0x00030000 0x00000009 + 0x00030001 0x00000011 + 0x00030002 0x00000019 + 0x00030003 0x00000021 + 0x00030004 0x00000029 + 0x00030005 0x00000031 + 0x00030006 0x00000039 + 0x00030007 0x00000041>; +}; diff --git a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi index 15ae462..bb9f792 100644 --- a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi +++ b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi @@ -345,6 +345,7 @@ reg = <0xea000 0x4000>; }; +/include/ "qoriq-tmu-t104xsi.dtsi" /include/ "elo3-dma-0.dtsi" /include/ "elo3-dma-1.dtsi" /include/ "qoriq-espi-0.dtsi"
This is Thermal Monitoring Unit for QorIQ platform. Signed-off-by: Jia Hongtao <hongtao.jia@freescale.com> --- Changes from V1: * Provide more details on compatible and calibration properties. .../devicetree/bindings/thermal/qoriq-thermal.txt | 68 ++++++++++++++++++ arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi | 82 ++++++++++++++++++++++ arch/powerpc/boot/dts/fsl/t1040si-post.dtsi | 1 + 3 files changed, 151 insertions(+) create mode 100644 Documentation/devicetree/bindings/thermal/qoriq-thermal.txt create mode 100644 arch/powerpc/boot/dts/fsl/qoriq-tmu-t104xsi.dtsi