diff mbox

[V5,02/30] thermal: exynos: Add ARCH_HAS_TMU config to know the supported soc's

Message ID 1370955220-2949-3-git-send-email-amit.daniel@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Amit Kachhap June 11, 2013, 12:53 p.m. UTC
This patch adds config sybmol ARCH_HAS_TMU to enable the TMU driver.
This will allow adding support for new soc easily as now it is the
platform responsibility to enable this config symbol.

Acked-by: Kukjin Kim <kgene.kim@samsung.com>
Signed-off-by: Amit Daniel Kachhap <amit.daniel@samsung.com>
---
 drivers/thermal/samsung/Kconfig |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

Comments

Eduardo Valentin June 17, 2013, 3:05 a.m. UTC | #1
Hey Amit,

On 11-06-2013 08:53, Amit Daniel Kachhap wrote:
> This patch adds config sybmol ARCH_HAS_TMU to enable the TMU driver.
> This will allow adding support for new soc easily as now it is the
> platform responsibility to enable this config symbol.
> 
> Acked-by: Kukjin Kim <kgene.kim@samsung.com>
> Signed-off-by: Amit Daniel Kachhap <amit.daniel@samsung.com>
> ---
>  drivers/thermal/samsung/Kconfig |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/thermal/samsung/Kconfig b/drivers/thermal/samsung/Kconfig
> index 2d3d9dc..145a55d 100644
> --- a/drivers/thermal/samsung/Kconfig
> +++ b/drivers/thermal/samsung/Kconfig
> @@ -1,6 +1,9 @@
> +config ARCH_HAS_TMU
> +	bool
> +
>  config EXYNOS_THERMAL
>  	tristate "Temperature sensor on Samsung EXYNOS"
> -	depends on (ARCH_EXYNOS4 || ARCH_EXYNOS5)
> +	depends on ARCH_HAS_TMU

I would rather use ARCH_HAS_BANDGAP as discussed and agreed during your
v4 review. I managed push this patch through RMK's system:
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=7758/1

>  	depends on CPU_THERMAL
>  	help
>  	  If you say yes here you get support for TMU (Thermal Management
>
Amit Kachhap June 17, 2013, 3:46 a.m. UTC | #2
Hi Eduardo,

On Mon, Jun 17, 2013 at 8:35 AM, Eduardo Valentin
<eduardo.valentin@ti.com> wrote:
> Hey Amit,
>
> On 11-06-2013 08:53, Amit Daniel Kachhap wrote:
>> This patch adds config sybmol ARCH_HAS_TMU to enable the TMU driver.
>> This will allow adding support for new soc easily as now it is the
>> platform responsibility to enable this config symbol.
>>
>> Acked-by: Kukjin Kim <kgene.kim@samsung.com>
>> Signed-off-by: Amit Daniel Kachhap <amit.daniel@samsung.com>
>> ---
>>  drivers/thermal/samsung/Kconfig |    5 ++++-
>>  1 files changed, 4 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/thermal/samsung/Kconfig b/drivers/thermal/samsung/Kconfig
>> index 2d3d9dc..145a55d 100644
>> --- a/drivers/thermal/samsung/Kconfig
>> +++ b/drivers/thermal/samsung/Kconfig
>> @@ -1,6 +1,9 @@
>> +config ARCH_HAS_TMU
>> +     bool
>> +
>>  config EXYNOS_THERMAL
>>       tristate "Temperature sensor on Samsung EXYNOS"
>> -     depends on (ARCH_EXYNOS4 || ARCH_EXYNOS5)
>> +     depends on ARCH_HAS_TMU
>
> I would rather use ARCH_HAS_BANDGAP as discussed and agreed during your
> v4 review. I managed push this patch through RMK's system:
> http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=7758/1
Agreed I will post my TMU re-structured patches with this config flag.

Thanks,
Amit Daniel

>
>>       depends on CPU_THERMAL
>>       help
>>         If you say yes here you get support for TMU (Thermal Management
>>
>
>
> --
> 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-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/thermal/samsung/Kconfig b/drivers/thermal/samsung/Kconfig
index 2d3d9dc..145a55d 100644
--- a/drivers/thermal/samsung/Kconfig
+++ b/drivers/thermal/samsung/Kconfig
@@ -1,6 +1,9 @@ 
+config ARCH_HAS_TMU
+	bool
+
 config EXYNOS_THERMAL
 	tristate "Temperature sensor on Samsung EXYNOS"
-	depends on (ARCH_EXYNOS4 || ARCH_EXYNOS5)
+	depends on ARCH_HAS_TMU
 	depends on CPU_THERMAL
 	help
 	  If you say yes here you get support for TMU (Thermal Management