Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751351Ab3FEDtL (ORCPT ); Tue, 4 Jun 2013 23:49:11 -0400 Received: from mail-ie0-f172.google.com ([209.85.223.172]:40070 "EHLO mail-ie0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750791Ab3FEDtI (ORCPT ); Tue, 4 Jun 2013 23:49:08 -0400 MIME-Version: 1.0 In-Reply-To: <51ADE515.5020802@ti.com> References: <1368525540-15034-1-git-send-email-amit.daniel@samsung.com> <51939F61.5020904@ti.com> <51ADE41E.3070202@ti.com> <51ADE515.5020802@ti.com> Date: Wed, 5 Jun 2013 09:19:05 +0530 X-Google-Sender-Auth: 61pdDCw7CS92-rYJFPqRMXRHJpg Message-ID: Subject: Re: [PATCH V4 00/30] thermal: exynos: Add thermal driver for exynos5440 From: amit daniel kachhap To: Eduardo Valentin Cc: linux-pm@vger.kernel.org, Zhang Rui , linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Kukjin Kim Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 8670 Lines: 185 On Tue, Jun 4, 2013 at 6:31 PM, Eduardo Valentin wrote: > > Hi, > > On 04-06-2013 08:57, Eduardo Valentin wrote: >> On 04-06-2013 00:55, amit daniel kachhap wrote: >>> Hi Eduardo, >>> >>> On Wed, May 15, 2013 at 8:14 PM, Eduardo Valentin >>> wrote: >>>> On 14-05-2013 05:58, Amit Daniel Kachhap wrote: >>>>> Changes in V4: >>>>> Almost all the changes in this version is as per suggestion from Eduardo.The >>>>> major ones are listed below, >>>>> * Added kconfig symbol ARCH_HAS_TMU which needs to be enabled by platform. With >>>>> this change existing symbol EXYNOS_TMU_DATA is not needed. >>>> >>>> I was more thinking if we could have a single flag that we could use in >>>> thermal drivers. Besides, my proposal for ARCH_HAS_BANDGAP is still not >>>> merged yet. >>> >>> yes a single flag might be useful such as ARCH_HAS_THERMAL but this >>> patch can work as an intermediate solution. >> >> I think bandgap is a generic enough term IMO. >> >> [1] - http://en.wikipedia.org/wiki/Bandgap_voltage_reference >> [2] - http://en.wikipedia.org/wiki/Silicon_bandgap_temperature_sensor >> > > Hit the send button too early. So, I think bandgap is a generic enough > term. Using bandgap I believe it is better because its meaning has the > implication of having a device to control thermal. > > But if you feel more confident with the thermal term we can switch to > something like HAS_THERMAL_CONTROL. Even in my case TMU uses bandgap voltage reference circuit. so ARCH_HAS_BANDGAP should be fine. Thanks, Amit Daniel > >> >>> >>> Thanks, >>> Amit Daniel >>>> >>>>> * Movement of freq_clip_table from exynos_tmu.h to exynos_thermal_common.h is >>>>> explained in the commit logs. >>>>> * Wrote all register description documentation. >>>>> * Split 5440 TMU support patch into controller change, configuration data and >>>>> feature addition patches. >>>>> * Remove all *LINUX_* in the header files. >>>>> * Still regulator enable is kept optional but a TODO: comment is added to fix >>>>> it later. >>>>> >>>>> Changes in V3: >>>>> * Added proper dependency of different exynos thermal Kconfig symbols. Basically 3 >>>>> Kconfig can be enabled now and corresponds to tmu driver. exynos common part >>>>> and exynos configuration data. This issue was raised by Rui Zhang. >>>>> >>>>> Changes in V2: >>>>> * Separated SOC data from TMU driver. This is as per suggestion from Eduardo. >>>>> * Merged the new file created for exynos5440 TMU controller with the existing >>>>> TMU controller code. >>>>> * Removed the DT parsing code as now the SOC specific data are cleanly put >>>>> inside the data specific file. >>>>> * Even the register definations/bitfields are treated as data as there is >>>>> some variation across SOC's. >>>>> >>>>> This patchset adds TMU(Thermal management Unit) driver support for >>>>> exynos5440 platform. There are 3 instances of the TMU controllers so >>>>> necessary cleanup/re-structure is done to handle multiple thermal zone. >>>>> >>>>> Patch (exynos4: Add documentation for Exynos SoC thermal bindings) from >>>>> Lukasz Majewski is already posted to mainline. Adding it here for completeness. >>>>> (http://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg17817.html) >>>>> >>>>> Patch (thermal: exynos: Support thermal tripping ) from Jonghwan Choi is >>>>> added here with some changes. >>>>> (https://patchwork.kernel.org/patch/1668371/) >>>>> >>>>> Patch (thermal: exynos: Support for TMU regulator defined at device tree) >>>>> is a repost of my earlier patch(https://patchwork-mail1.kernel.org/patch/2510771/) >>>>> and adds regulator support. >>>>> >>>>> Patch (ARM: dts: Add device tree node for exynos5440 TMU controller) and >>>>> patch (arm: exynos: enable ARCH_HAS_TMU) can be merged through exynos platform >>>>> maintainer as this can cause merge conflict. >>>>> >>>>> All these patches are based on thermal maintainers git tree, >>>>> git://git.kernel.org/pub/scm/linux/kernel/git/rzhang/linux.git next. >>>>> >>>>> Amit Daniel Kachhap (29): >>>>> thermal: exynos: Moving exynos thermal files into samsung directory >>>>> thermal: exynos: Add ARCH_HAS_TMU config to know the supported soc's >>>>> thermal: exynos: Remove CPU_THERMAL dependency for using TMU driver >>>>> thermal: exynos: Bifurcate exynos thermal common and tmu controller >>>>> code >>>>> thermal: exynos: Rename exynos_thermal.c to exynos_tmu.c >>>>> thermal: exynos: Move exynos_thermal.h from include/* to driver/* >>>>> folder >>>>> thermal: exynos: Bifurcate exynos tmu driver and configuration data >>>>> thermal: exynos: Add missing definations and code cleanup >>>>> thermal: exynos: Add extra entries in the tmu platform data >>>>> thermal: exynos: Support thermal tripping >>>>> thermal: exynos: Move register definitions from driver file to data >>>>> file >>>>> thermal: exynos: Fix to clear only the generated interrupts >>>>> thermal: exynos: Add support for instance based register/unregister >>>>> thermal: exynos: Modify private_data to appropriate name driver_data >>>>> thermal: exynos: Return success even if no cooling data supplied >>>>> thermal: exynos: Make the zone handling dependent on trip count >>>>> thermal: exynos: Add support to handle many instances of TMU >>>>> thermal: exynos: Add TMU features to check instead of using SOC type >>>>> thermal: exynos: use device resource management infrastructure >>>>> thermal: exynos: Add support to access common register for >>>>> multistance >>>>> thermal: exynos: Add support for exynos5440 TMU sensor. >>>>> thermal: exynos: Fix to set the second point correction value >>>>> properly >>>>> thermal: exynos: Add thermal configuration data for exynos5440 TMU >>>>> sensor >>>>> thermal: exynos: Add a compensation logic on swapped e-fuse values >>>>> thermal: exynos: Add hardware mode thermal calibration support >>>>> Documentation: thermal: Explain the exynos thermal driver model >>>>> thermal: exynos: Support for TMU regulator defined at device tree >>>>> ARM: dts: Add device tree node for exynos5440 TMU controller >>>>> arm: exynos: enable ARCH_HAS_TMU >>>>> >>>>> Lukasz Majewski (1): >>>>> ARM: dts: thermal: exynos4: Add documentation for Exynos SoC thermal >>>>> bindings >>>>> >>>>> .../devicetree/bindings/thermal/exynos-thermal.txt | 53 + >>>>> Documentation/thermal/exynos_thermal | 43 +- >>>>> arch/arm/boot/dts/exynos5440.dtsi | 30 + >>>>> arch/arm/mach-exynos/Kconfig | 5 + >>>>> drivers/thermal/Kconfig | 13 +- >>>>> drivers/thermal/Makefile | 2 +- >>>>> drivers/thermal/exynos_thermal.c | 1066 -------------------- >>>>> drivers/thermal/samsung/Kconfig | 21 + >>>>> drivers/thermal/samsung/Makefile | 7 + >>>>> drivers/thermal/samsung/exynos_thermal_common.c | 407 ++++++++ >>>>> drivers/thermal/samsung/exynos_thermal_common.h | 98 ++ >>>>> drivers/thermal/samsung/exynos_tmu.c | 741 ++++++++++++++ >>>>> drivers/thermal/samsung/exynos_tmu.h | 294 ++++++ >>>>> drivers/thermal/samsung/exynos_tmu_data.c | 225 ++++ >>>>> drivers/thermal/samsung/exynos_tmu_data.h | 155 +++ >>>>> include/linux/platform_data/exynos_thermal.h | 119 --- >>>>> 16 files changed, 2076 insertions(+), 1203 deletions(-) >>>>> create mode 100644 Documentation/devicetree/bindings/thermal/exynos-thermal.txt >>>>> delete mode 100644 drivers/thermal/exynos_thermal.c >>>>> create mode 100644 drivers/thermal/samsung/Kconfig >>>>> create mode 100644 drivers/thermal/samsung/Makefile >>>>> create mode 100644 drivers/thermal/samsung/exynos_thermal_common.c >>>>> create mode 100644 drivers/thermal/samsung/exynos_thermal_common.h >>>>> create mode 100644 drivers/thermal/samsung/exynos_tmu.c >>>>> create mode 100644 drivers/thermal/samsung/exynos_tmu.h >>>>> create mode 100644 drivers/thermal/samsung/exynos_tmu_data.c >>>>> create mode 100644 drivers/thermal/samsung/exynos_tmu_data.h >>>>> delete mode 100644 include/linux/platform_data/exynos_thermal.h >>>>> >>>>> >>>>> >>>> >>>> >>> >>> >> >> > > > -- > You have got to be excited about what you are doing. (L. Lamport) > > Eduardo Valentin > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/