Re: [PATCH v11 0/4] thermal: samsung: Clean up and add support for Exynos5420

2014-04-08 Thread Javi Merino
On Thu, Mar 20, 2014 at 02:45:54AM +, Naveen Krishna Ch wrote:
 Hello Tomasz,
 
 On 20 March 2014 00:58, Tomasz Figa t.f...@samsung.com wrote:
  Hi Leela,
 
 
  On 19.03.2014 12:19, Leela Krishna Amudala wrote:
 
  Hi All,
 
  I didn't see this series in mainline, Any comments for this ?
 
 
  Naveen had posted v12 of this series and I believe all the patches got my
  Reviewed-by.
 
  Best regards,
  Tomasz
 Thanks for your reply.
 These patches are posted to the mailing list quite some time now.
 Anything i can do to speed up the process.

These patches create 5 thermal zones, one for each sensor.  Due to the
way the exynos code in drivers/thermal/samsung/exynos_thermal_common.c
registers thermal zones, each one gets a cpufreq_cooling_device.
Therefore you end up with 5 thermal zone each of which is controlling
the cpu frequency.

Wouldn't it be better to collate these 5 sensors to create one thermal
zone for the whole SoC?  Maybe a simplistic algorithm like reporting
the maximum temperature of all the sensors as the thermal zone
temperature is good enough.

Cheers,
Javi

--
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 v11 0/4] thermal: samsung: Clean up and add support for Exynos5420

2014-03-19 Thread Leela Krishna Amudala
Hi All,

I didn't see this series in mainline, Any comments for this ?

Thanks,
Leela Krishna Amudala.


On Tue, Dec 10, 2013 at 12:10 PM, Naveen Krishna Chatradhi
ch.nav...@samsung.com wrote:
 This patchset does a little clean up of the existing code (linux-soc-thermal)
 1.  [v11] thermal: samsung: replace inten_ bit fields with intclr_
 2.  [v11] thermal: samsung: change base_common to more meaningful base_second

 adds support for Exynos5420 in the driver and (linux-soc-thermal)
 3.  [v11] thermal: samsung: Add TMU support for Exynos5420 SoCs
 also adds the device tree nodes for the same to exynos5420.dtsi 
 (linux-samsung.git)
 4.  [v11] ARM: dts: Exynos5420: Add device nodes for TMU blocks

 (linux-soc-thermal)
 Naveen Krishna Chatradhi (3):
   thermal: samsung: replace inten_ bit fields with intclr_
   thermal: samsung: change base_common to more meaningful base_second
   thermal: samsung: Add TMU support for Exynos5420 SoCs

  .../devicetree/bindings/thermal/exynos-thermal.txt |   42 ++-
  drivers/thermal/samsung/exynos_tmu.c   |   72 +---
  drivers/thermal/samsung/exynos_tmu.h   |   21 ++--
  drivers/thermal/samsung/exynos_tmu_data.c  |  119 
 ++--
  drivers/thermal/samsung/exynos_tmu_data.h  |   12 +-
  5 files changed, 228 insertions(+), 38 deletions(-)

 (linux-samsung.git)
 Naveen Krishna Chatradhi (1):
   ARM: dts: Exynos5420: Add device nodes for TMU blocks

  arch/arm/boot/dts/exynos5420.dtsi |   40 
 +
  1 file changed, 40 insertions(+)
 --
 1.7.10.4

 --
 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
--
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 v11 0/4] thermal: samsung: Clean up and add support for Exynos5420

2014-03-19 Thread Tomasz Figa

Hi Leela,

On 19.03.2014 12:19, Leela Krishna Amudala wrote:

Hi All,

I didn't see this series in mainline, Any comments for this ?


Naveen had posted v12 of this series and I believe all the patches got 
my Reviewed-by.


Best regards,
Tomasz
--
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 v11 0/4] thermal: samsung: Clean up and add support for Exynos5420

2014-03-19 Thread Naveen Krishna Ch
Hello Tomasz,

On 20 March 2014 00:58, Tomasz Figa t.f...@samsung.com wrote:
 Hi Leela,


 On 19.03.2014 12:19, Leela Krishna Amudala wrote:

 Hi All,

 I didn't see this series in mainline, Any comments for this ?


 Naveen had posted v12 of this series and I believe all the patches got my
 Reviewed-by.

 Best regards,
 Tomasz
Thanks for your reply.
These patches are posted to the mailing list quite some time now.
Anything i can do to speed up the process.




-- 
Shine bright,
(: Nav :)
--
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


[PATCH v11 0/4] thermal: samsung: Clean up and add support for Exynos5420

2013-12-09 Thread Naveen Krishna Chatradhi
This patchset does a little clean up of the existing code (linux-soc-thermal)
1.  [v11] thermal: samsung: replace inten_ bit fields with intclr_
2.  [v11] thermal: samsung: change base_common to more meaningful base_second

adds support for Exynos5420 in the driver and (linux-soc-thermal)
3.  [v11] thermal: samsung: Add TMU support for Exynos5420 SoCs
also adds the device tree nodes for the same to exynos5420.dtsi 
(linux-samsung.git)
4.  [v11] ARM: dts: Exynos5420: Add device nodes for TMU blocks

(linux-soc-thermal)
Naveen Krishna Chatradhi (3):
  thermal: samsung: replace inten_ bit fields with intclr_
  thermal: samsung: change base_common to more meaningful base_second
  thermal: samsung: Add TMU support for Exynos5420 SoCs

 .../devicetree/bindings/thermal/exynos-thermal.txt |   42 ++-
 drivers/thermal/samsung/exynos_tmu.c   |   72 +---
 drivers/thermal/samsung/exynos_tmu.h   |   21 ++--
 drivers/thermal/samsung/exynos_tmu_data.c  |  119 ++--
 drivers/thermal/samsung/exynos_tmu_data.h  |   12 +-
 5 files changed, 228 insertions(+), 38 deletions(-)

(linux-samsung.git)
Naveen Krishna Chatradhi (1):
  ARM: dts: Exynos5420: Add device nodes for TMU blocks

 arch/arm/boot/dts/exynos5420.dtsi |   40 +
 1 file changed, 40 insertions(+)
-- 
1.7.10.4

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