diff mbox

[V2,1/7] dt-bindings: Update QorIQ TMU thermal bindings

Message ID 1465270054-31512-1-git-send-email-hongtao.jia@nxp.com (mailing list archive)
State Accepted, archived
Delegated to: Zhang Rui
Headers show

Commit Message

Jia Hongtao June 7, 2016, 3:27 a.m. UTC
For different types of SoC the sensor id and endianness may vary.
"#thermal-sensor-cells" is used to provide sensor id information.
"little-endian" property is to tell the endianness of TMU.

Signed-off-by: Jia Hongtao <hongtao.jia@nxp.com>
---
Changes for V2:
* Remove formatting chnages.

 Documentation/devicetree/bindings/thermal/qoriq-thermal.txt | 7 +++++++
 1 file changed, 7 insertions(+)

--
2.1.0.27.g96db324

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

Comments

Rob Herring (Arm) June 8, 2016, 7:52 p.m. UTC | #1
On Tue, Jun 07, 2016 at 11:27:34AM +0800, Jia Hongtao wrote:
> For different types of SoC the sensor id and endianness may vary.
> "#thermal-sensor-cells" is used to provide sensor id information.
> "little-endian" property is to tell the endianness of TMU.
> 
> Signed-off-by: Jia Hongtao <hongtao.jia@nxp.com>
> ---
> Changes for V2:
> * Remove formatting chnages.
> 
>  Documentation/devicetree/bindings/thermal/qoriq-thermal.txt | 7 +++++++
>  1 file changed, 7 insertions(+)

Acked-by: Rob Herring <robh@kernel.org>
--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Yang Li Sept. 14, 2016, 4:40 p.m. UTC | #2
On Wed, Jun 8, 2016 at 2:52 PM, Rob Herring <robh@kernel.org> wrote:
> On Tue, Jun 07, 2016 at 11:27:34AM +0800, Jia Hongtao wrote:
>> For different types of SoC the sensor id and endianness may vary.
>> "#thermal-sensor-cells" is used to provide sensor id information.
>> "little-endian" property is to tell the endianness of TMU.
>>
>> Signed-off-by: Jia Hongtao <hongtao.jia@nxp.com>
>> ---
>> Changes for V2:
>> * Remove formatting chnages.
>>
>>  Documentation/devicetree/bindings/thermal/qoriq-thermal.txt | 7 +++++++
>>  1 file changed, 7 insertions(+)
>
> Acked-by: Rob Herring <robh@kernel.org>

Hi Zhang Rui,

Since you have applied the driver patch, can you also apply the
binding patch?  The binding is supposed to go with the driver.

Regards,
Leo
--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Zhang Rui Sept. 19, 2016, 2:44 a.m. UTC | #3
On 三, 2016-09-14 at 11:40 -0500, Leo Li wrote:
> On Wed, Jun 8, 2016 at 2:52 PM, Rob Herring <robh@kernel.org> wrote:
> > 
> > On Tue, Jun 07, 2016 at 11:27:34AM +0800, Jia Hongtao wrote:
> > > 
> > > For different types of SoC the sensor id and endianness may vary.
> > > "#thermal-sensor-cells" is used to provide sensor id information.
> > > "little-endian" property is to tell the endianness of TMU.
> > > 
> > > Signed-off-by: Jia Hongtao <hongtao.jia@nxp.com>
> > > ---
> > > Changes for V2:
> > > * Remove formatting chnages.
> > > 
> > >  Documentation/devicetree/bindings/thermal/qoriq-thermal.txt | 7
> > > +++++++
> > >  1 file changed, 7 insertions(+)
> > Acked-by: Rob Herring <robh@kernel.org>
> Hi Zhang Rui,
> 
> Since you have applied the driver patch, can you also apply the
> binding patch?  The binding is supposed to go with the driver.
> 

Do you mean I should take both patch 1/7 and 7/7? I can not see the
other patches in this patch set.

thanks,
rui

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jia Hongtao Oct. 9, 2016, 7:12 a.m. UTC | #4
> -----Original Message-----

> From: Zhang Rui [mailto:rui.zhang@intel.com]

> Sent: Monday, September 19, 2016 10:44 AM

> To: Leo Li <pku.leo@gmail.com>; Rob Herring <robh@kernel.org>

> Cc: Troy Jia <hongtao.jia@nxp.com>; devicetree@vger.kernel.org; linux-

> pm@vger.kernel.org; linuxppc-dev <linuxppc-dev@lists.ozlabs.org>; lkml <linux-

> kernel@vger.kernel.org>; Scott Wood <scott.wood@nxp.com>; Eduardo Valentin

> <edubezval@gmail.com>; Kumar Gala <galak@codeaurora.org>; Shawn Guo

> <shawnguo@kernel.org>; linux-arm-kernel@lists.infradead.org

> Subject: Re: [PATCH V2 1/7] dt-bindings: Update QorIQ TMU thermal bindings

> 

> On 三, 2016-09-14 at 11:40 -0500, Leo Li wrote:

> > On Wed, Jun 8, 2016 at 2:52 PM, Rob Herring <robh@kernel.org> wrote:

> > >

> > > On Tue, Jun 07, 2016 at 11:27:34AM +0800, Jia Hongtao wrote:

> > > >

> > > > For different types of SoC the sensor id and endianness may vary.

> > > > "#thermal-sensor-cells" is used to provide sensor id information.

> > > > "little-endian" property is to tell the endianness of TMU.

> > > >

> > > > Signed-off-by: Jia Hongtao <hongtao.jia@nxp.com>

> > > > ---

> > > > Changes for V2:

> > > > * Remove formatting chnages.

> > > >

> > > >  Documentation/devicetree/bindings/thermal/qoriq-thermal.txt | 7

> > > > +++++++

> > > >  1 file changed, 7 insertions(+)

> > > Acked-by: Rob Herring <robh@kernel.org>

> > Hi Zhang Rui,

> >

> > Since you have applied the driver patch, can you also apply the

> > binding patch?  The binding is supposed to go with the driver.

> >

> 

> Do you mean I should take both patch 1/7 and 7/7? I can not see the other patches

> in this patch set.


Yes, please take 7/7 as well.
I also reworked the device tree patches and you can find them here:
https://patchwork.kernel.org/patch/9368377/
https://patchwork.kernel.org/patch/9368381/
https://patchwork.kernel.org/patch/9368383/
https://patchwork.kernel.org/patch/9368379/
https://patchwork.kernel.org/patch/9368385/

-Hongtao.

> 

> thanks,

> rui
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt b/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt
index 66223d5..20ca4ef 100644
--- a/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt
+++ b/Documentation/devicetree/bindings/thermal/qoriq-thermal.txt
@@ -17,6 +17,12 @@  Required properties:
 	calibration data, as specified by the SoC reference manual.
 	The first cell of each pair is the value to be written to TTCFGR,
 	and the second is the value to be written to TSCFGR.
+- #thermal-sensor-cells : Must be 1. The sensor specifier is the monitoring
+	site ID, and represents the "n" in TRITSRn and TRATSRn.
+
+Optional property:
+- little-endian : If present, the TMU registers are little endian. If absent,
+	the default is big endian.

 Example:

@@ -60,4 +66,5 @@  tmu@f0000 {

 			       0x00030000 0x00000012
 			       0x00030001 0x0000001d>;
+	#thermal-sensor-cells = <1>;
 };