[PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ T104x

[PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ T104x

[Jia Hongtao]

This is Thermal Monitoring Unit for QorIQ platform.

Signed-off-by: Jia Hongtao <hongtao.jia-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
---
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

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

RE: [PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ T104x

[Hongtao Jia]

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-KZfg59tc24xl57MIdRCFDg@public.gmane.org]
> Sent: Wednesday, April 15, 2015 4:18 PM
> To: Wood Scott-B07421
> Cc: linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org; devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org;
> robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org; 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-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> ---
> 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

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ T104x

[Scott Wood]

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


_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

RE: [PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ T104x

[Hongtao Jia]

> -----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
> 

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ T104x

[Scott Wood]

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-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org; linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org;
> > devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org
> > 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


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

RE: [PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ T104x

[Hongtao Jia]

> -----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
> 

Re: [PATCH V2] QorIQ/TMU: add TMU node to device tree for QorIQ T104x

[Scott Wood]

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-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org; linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org;
> > devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org
> > 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-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org; linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org;
> > > > devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; 
> > > > rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org
> > > > 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

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html