Re: [PATCH v2 08/17] thermal: exynos: dts: Add default definition of the TMU sensor parameter

2015-01-12 Thread Lukasz Majewski
Hi Eduardo,

 On Fri, Jan 02, 2015 at 02:11:41PM -0400, Eduardo Valentin wrote:
  Lukasz,
  
  On Wed, Dec 10, 2014 at 01:09:47PM +0100, Lukasz Majewski wrote:
   Exynos 4 and 5 family of SoCs uses almost identical TMU sensor to
   measure the on chip temperature. For this reason it is possible
   to group TMU configuration parameters in one dts file.
   
   Signed-off-by: Lukasz Majewski l.majew...@samsung.com
   ---
   Changes for v2:
   - None
   ---
arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi | 24
   +++ 1 file changed, 24 insertions(+)
create mode 100644
   arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
   
   diff --git a/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
   b/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi new file mode
   100644 index 000..ee6d8bb
   --- /dev/null
   +++ b/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
   @@ -0,0 +1,24 @@
   +/*
   + * Device tree sources for Exynos4412 TMU sensor configuration
   + *
   + * Copyright (c) 2014 Lukasz Majewski l.majew...@samsung.com
   + *
   + * This program is free software; you can redistribute it and/or
   modify
   + * it under the terms of the GNU General Public License version
   2 as
   + * published by the Free Software Foundation.
   + *
   + */
   +
   +#include dt-bindings/thermal/thermal_exynos.h
   +
   +#thermal-sensor-cells = 0;
   +gain = 8;
   +reference_voltage = 16;
   +noise_cancel_mode = 4;
   +efuse_value = 55;
   +min_efuse_value = 40;
   +max_efuse_value = 100;
   +first_point_trim = 25;
   +second_point_trim = 85;
   +default_temp_offset = 50;
   +cal_type = TYPE_ONE_POINT_TRIMMING;
  
  Are the above properties specific to exynos? 

Yes, they are specific for Exynos.

  For specific DT
  entries, they need to be marked with a prefix. Please read the
  'Device Specific Data' section of Device tree Documentation [1].

Ok, I will. Thanks for tip.

  
  [1] - http://devicetree.org/Device_Tree_Usage
 
 
 BTW, you have to update:
 Documentation/devicetree/bindings/thermal/exynos-thermal.txt

I've forgotten to update this entry. Will be fixed at v3.

 
 
   -- 
   2.0.0.rc2
   
 
 



-- 
Best regards,

Lukasz Majewski

Samsung RD Institute Poland (SRPOL) | Linux Platform Group
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 08/17] thermal: exynos: dts: Add default definition of the TMU sensor parameter

2015-01-02 Thread Eduardo Valentin
Lukasz,

On Wed, Dec 10, 2014 at 01:09:47PM +0100, Lukasz Majewski wrote:
 Exynos 4 and 5 family of SoCs uses almost identical TMU sensor to measure the
 on chip temperature. For this reason it is possible to group TMU configuration
 parameters in one dts file.
 
 Signed-off-by: Lukasz Majewski l.majew...@samsung.com
 ---
 Changes for v2:
 - None
 ---
  arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi | 24 
 +++
  1 file changed, 24 insertions(+)
  create mode 100644 arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
 
 diff --git a/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi 
 b/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
 new file mode 100644
 index 000..ee6d8bb
 --- /dev/null
 +++ b/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
 @@ -0,0 +1,24 @@
 +/*
 + * Device tree sources for Exynos4412 TMU sensor configuration
 + *
 + * Copyright (c) 2014 Lukasz Majewski l.majew...@samsung.com
 + *
 + * This program is free software; you can redistribute it and/or modify
 + * it under the terms of the GNU General Public License version 2 as
 + * published by the Free Software Foundation.
 + *
 + */
 +
 +#include dt-bindings/thermal/thermal_exynos.h
 +
 +#thermal-sensor-cells = 0;
 +gain = 8;
 +reference_voltage = 16;
 +noise_cancel_mode = 4;
 +efuse_value = 55;
 +min_efuse_value = 40;
 +max_efuse_value = 100;
 +first_point_trim = 25;
 +second_point_trim = 85;
 +default_temp_offset = 50;
 +cal_type = TYPE_ONE_POINT_TRIMMING;

Are the above properties specific to exynos? For specific DT entries,
they need to be marked with a prefix. Please read the 'Device Specific
Data' section of Device tree Documentation [1].

[1] - http://devicetree.org/Device_Tree_Usage
 -- 
 2.0.0.rc2
 


signature.asc
Description: Digital signature


Re: [PATCH v2 08/17] thermal: exynos: dts: Add default definition of the TMU sensor parameter

2015-01-02 Thread Eduardo Valentin
On Fri, Jan 02, 2015 at 02:11:41PM -0400, Eduardo Valentin wrote:
 Lukasz,
 
 On Wed, Dec 10, 2014 at 01:09:47PM +0100, Lukasz Majewski wrote:
  Exynos 4 and 5 family of SoCs uses almost identical TMU sensor to measure 
  the
  on chip temperature. For this reason it is possible to group TMU 
  configuration
  parameters in one dts file.
  
  Signed-off-by: Lukasz Majewski l.majew...@samsung.com
  ---
  Changes for v2:
  - None
  ---
   arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi | 24 
  +++
   1 file changed, 24 insertions(+)
   create mode 100644 arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
  
  diff --git a/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi 
  b/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
  new file mode 100644
  index 000..ee6d8bb
  --- /dev/null
  +++ b/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
  @@ -0,0 +1,24 @@
  +/*
  + * Device tree sources for Exynos4412 TMU sensor configuration
  + *
  + * Copyright (c) 2014 Lukasz Majewski l.majew...@samsung.com
  + *
  + * This program is free software; you can redistribute it and/or modify
  + * it under the terms of the GNU General Public License version 2 as
  + * published by the Free Software Foundation.
  + *
  + */
  +
  +#include dt-bindings/thermal/thermal_exynos.h
  +
  +#thermal-sensor-cells = 0;
  +gain = 8;
  +reference_voltage = 16;
  +noise_cancel_mode = 4;
  +efuse_value = 55;
  +min_efuse_value = 40;
  +max_efuse_value = 100;
  +first_point_trim = 25;
  +second_point_trim = 85;
  +default_temp_offset = 50;
  +cal_type = TYPE_ONE_POINT_TRIMMING;
 
 Are the above properties specific to exynos? For specific DT entries,
 they need to be marked with a prefix. Please read the 'Device Specific
 Data' section of Device tree Documentation [1].
 
 [1] - http://devicetree.org/Device_Tree_Usage


BTW, you have to update:
Documentation/devicetree/bindings/thermal/exynos-thermal.txt


  -- 
  2.0.0.rc2
  




signature.asc
Description: Digital signature


[PATCH v2 08/17] thermal: exynos: dts: Add default definition of the TMU sensor parameter

2014-12-10 Thread Lukasz Majewski
Exynos 4 and 5 family of SoCs uses almost identical TMU sensor to measure the
on chip temperature. For this reason it is possible to group TMU configuration
parameters in one dts file.

Signed-off-by: Lukasz Majewski l.majew...@samsung.com
---
Changes for v2:
- None
---
 arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi | 24 +++
 1 file changed, 24 insertions(+)
 create mode 100644 arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi

diff --git a/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi 
b/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
new file mode 100644
index 000..ee6d8bb
--- /dev/null
+++ b/arch/arm/boot/dts/exynos4412-tmu-sensor-conf.dtsi
@@ -0,0 +1,24 @@
+/*
+ * Device tree sources for Exynos4412 TMU sensor configuration
+ *
+ * Copyright (c) 2014 Lukasz Majewski l.majew...@samsung.com
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ */
+
+#include dt-bindings/thermal/thermal_exynos.h
+
+#thermal-sensor-cells = 0;
+gain = 8;
+reference_voltage = 16;
+noise_cancel_mode = 4;
+efuse_value = 55;
+min_efuse_value = 40;
+max_efuse_value = 100;
+first_point_trim = 25;
+second_point_trim = 85;
+default_temp_offset = 50;
+cal_type = TYPE_ONE_POINT_TRIMMING;
-- 
2.0.0.rc2

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