All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/17] thermal: add COMPILE_TEST (on drivers)
@ 2015-09-10  4:10 Eduardo Valentin
  2015-09-10  4:10 ` [PATCH 01/17] thermal: hisi: allow compile test Eduardo Valentin
                   ` (17 more replies)
  0 siblings, 18 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:10 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Hello all,

This is a simple series to add the COMPILE_TEST on thermal drivers.
The idea is to help compiling and maintaining.

Split into smaller patches so people can have their own voices on their drivers.

Let me know if this is a problem.

BR,

Eduardo Valentin (17):
  thermal: hisi: allow compile test
  thermal: spear: allow compile test
  thermal: rockchip: allow compile test
  thermal: kirkwood: allow compile test
  thermal: dove: allow compile test
  thermal: u8500: allow compile test
  thermal: armada: allow compile test
  thermal: tegra: allow compile test
  thermal: powerclamp: allow compile test
  thermal: intel_soc_dts_iosf: allow compile test
  thermal: intel_soc_dts: allow compile test
  thermal: intel_quark: allow compile test
  thermal: int340x: allow compile test
  thermal: exynos: allow compile test
  thermal: st: allow compile test
  thermal: qcom_spmi: allow compile test
  thermal: ti-soc: allow compile test

 drivers/thermal/Kconfig                | 34 +++++++++++++++++-----------------
 drivers/thermal/ti-soc-thermal/Kconfig |  8 ++++----
 2 files changed, 21 insertions(+), 21 deletions(-)

-- 
2.5.0


^ permalink raw reply	[flat|nested] 34+ messages in thread

* [PATCH 01/17] thermal: hisi: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
@ 2015-09-10  4:10 ` Eduardo Valentin
  2015-09-10  4:10 ` [PATCH 02/17] thermal: spear: " Eduardo Valentin
                   ` (16 subsequent siblings)
  17 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:10 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to hisi driver to facilitate maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 118938e..f2b1d31 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -163,7 +163,7 @@ config THERMAL_EMULATION
 
 config HISI_THERMAL
 	tristate "Hisilicon thermal driver"
-	depends on ARCH_HISI && CPU_THERMAL && OF
+	depends on (ARCH_HISI && CPU_THERMAL && OF) || COMPILE_TEST
 	help
 	  Enable this to plug hisilicon's thermal sensor driver into the Linux
 	  thermal framework. cpufreq is used as the cooling device to throttle
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 02/17] thermal: spear: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
  2015-09-10  4:10 ` [PATCH 01/17] thermal: hisi: allow compile test Eduardo Valentin
@ 2015-09-10  4:10 ` Eduardo Valentin
  2015-09-10  4:23   ` Viresh Kumar
  2015-09-10  4:10   ` Eduardo Valentin
                   ` (15 subsequent siblings)
  17 siblings, 1 reply; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:10 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to spear driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index f2b1d31..0334886 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -182,7 +182,7 @@ config IMX_THERMAL
 
 config SPEAR_THERMAL
 	bool "SPEAr thermal sensor driver"
-	depends on PLAT_SPEAR
+	depends on PLAT_SPEAR || COMPILE_TEST
 	depends on OF
 	help
 	  Enable this to plug the SPEAr thermal sensor driver into the Linux
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 03/17] thermal: rockchip: allow compile test
@ 2015-09-10  4:10   ` Eduardo Valentin
  0 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:10 UTC (permalink / raw)
  To: Rui Zhang
  Cc: Linux PM, LKML, Eduardo Valentin, Heiko Stuebner,
	linux-arm-kernel, linux-rockchip

Adding COMPILE_TEST flag to rockchip driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: Heiko Stuebner <heiko@sntech.de>
Cc: linux-pm@vger.kernel.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-rockchip@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 0334886..2e0aaee 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -190,7 +190,7 @@ config SPEAR_THERMAL
 
 config ROCKCHIP_THERMAL
 	tristate "Rockchip thermal driver"
-	depends on ARCH_ROCKCHIP
+	depends on ARCH_ROCKCHIP || COMPILE_TEST
 	depends on RESET_CONTROLLER
 	help
 	  Rockchip thermal driver provides support for Temperature sensor
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 03/17] thermal: rockchip: allow compile test
@ 2015-09-10  4:10   ` Eduardo Valentin
  0 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:10 UTC (permalink / raw)
  To: Rui Zhang
  Cc: Heiko Stuebner, Linux PM, LKML, Eduardo Valentin,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

Adding COMPILE_TEST flag to rockchip driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Heiko Stuebner <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>
Cc: linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Signed-off-by: Eduardo Valentin <edubezval-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 0334886..2e0aaee 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -190,7 +190,7 @@ config SPEAR_THERMAL
 
 config ROCKCHIP_THERMAL
 	tristate "Rockchip thermal driver"
-	depends on ARCH_ROCKCHIP
+	depends on ARCH_ROCKCHIP || COMPILE_TEST
 	depends on RESET_CONTROLLER
 	help
 	  Rockchip thermal driver provides support for Temperature sensor
-- 
2.5.0

^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 03/17] thermal: rockchip: allow compile test
@ 2015-09-10  4:10   ` Eduardo Valentin
  0 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:10 UTC (permalink / raw)
  To: linux-arm-kernel

Adding COMPILE_TEST flag to rockchip driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: Heiko Stuebner <heiko@sntech.de>
Cc: linux-pm at vger.kernel.org
Cc: linux-arm-kernel at lists.infradead.org
Cc: linux-rockchip at lists.infradead.org
Cc: linux-kernel at vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 0334886..2e0aaee 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -190,7 +190,7 @@ config SPEAR_THERMAL
 
 config ROCKCHIP_THERMAL
 	tristate "Rockchip thermal driver"
-	depends on ARCH_ROCKCHIP
+	depends on ARCH_ROCKCHIP || COMPILE_TEST
 	depends on RESET_CONTROLLER
 	help
 	  Rockchip thermal driver provides support for Temperature sensor
-- 
2.5.0

^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 04/17] thermal: kirkwood: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (2 preceding siblings ...)
  2015-09-10  4:10   ` Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  4:11 ` [PATCH 05/17] thermal: dove: " Eduardo Valentin
                   ` (13 subsequent siblings)
  17 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to kirkwood driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 2e0aaee..f1da24e 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -208,7 +208,7 @@ config RCAR_THERMAL
 
 config KIRKWOOD_THERMAL
 	tristate "Temperature sensor on Marvell Kirkwood SoCs"
-	depends on MACH_KIRKWOOD
+	depends on MACH_KIRKWOOD || COMPILE_TEST
 	depends on OF
 	help
 	  Support for the Kirkwood thermal sensor driver into the Linux thermal
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 05/17] thermal: dove: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (3 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 04/17] thermal: kirkwood: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  4:11 ` [PATCH 06/17] thermal: u8500: " Eduardo Valentin
                   ` (12 subsequent siblings)
  17 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to dove driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index f1da24e..b38ce41 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -216,7 +216,7 @@ config KIRKWOOD_THERMAL
 
 config DOVE_THERMAL
 	tristate "Temperature sensor on Marvell Dove SoCs"
-	depends on ARCH_DOVE || MACH_DOVE
+	depends on ARCH_DOVE || MACH_DOVE || COMPILE_TEST
 	depends on OF
 	help
 	  Support for the Dove thermal sensor driver in the Linux thermal
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 06/17] thermal: u8500: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (4 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 05/17] thermal: dove: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  6:35   ` Geert Uytterhoeven
  2015-09-10  4:11 ` [PATCH 07/17] thermal: armada: " Eduardo Valentin
                   ` (11 subsequent siblings)
  17 siblings, 1 reply; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to u8500 driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index b38ce41..f810bad 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -224,7 +224,7 @@ config DOVE_THERMAL
 
 config DB8500_THERMAL
 	bool "DB8500 thermal management"
-	depends on ARCH_U8500
+	depends on ARCH_U8500 || COMPILE_TEST
 	default y
 	help
 	  Adds DB8500 thermal management implementation according to the thermal
@@ -252,7 +252,7 @@ config TEGRA_SOCTHERM
 
 config DB8500_CPUFREQ_COOLING
 	tristate "DB8500 cpufreq cooling"
-	depends on ARCH_U8500
+	depends on ARCH_U8500 || COMPILE_TEST
 	depends on CPU_THERMAL
 	default y
 	help
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 07/17] thermal: armada: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (5 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 06/17] thermal: u8500: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  4:11 ` [PATCH 08/17] thermal: tegra: " Eduardo Valentin
                   ` (10 subsequent siblings)
  17 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to armada driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index f810bad..0e0d0de 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -234,7 +234,7 @@ config DB8500_THERMAL
 
 config ARMADA_THERMAL
 	tristate "Armada 370/XP thermal management"
-	depends on ARCH_MVEBU
+	depends on ARCH_MVEBU || COMPILE_TEST
 	depends on OF
 	help
 	  Enable this option if you want to have support for thermal management
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 08/17] thermal: tegra: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (6 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 07/17] thermal: armada: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
       [not found]   ` <1441858273-4017-9-git-send-email-edubezval-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
  2015-09-10  4:11 ` [PATCH 09/17] thermal: powerclamp: " Eduardo Valentin
                   ` (9 subsequent siblings)
  17 siblings, 1 reply; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang
  Cc: Linux PM, LKML, Eduardo Valentin, Stephen Warren, Thierry Reding,
	Alexandre Courbot, linux-tegra

Adding COMPILE_TEST flag to tegra driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: Stephen Warren <swarren@wwwdotorg.org>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Alexandre Courbot <gnurou@gmail.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-tegra@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 0e0d0de..c94efd3 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -242,7 +242,7 @@ config ARMADA_THERMAL
 
 config TEGRA_SOCTHERM
 	tristate "Tegra SOCTHERM thermal management"
-	depends on ARCH_TEGRA
+	depends on ARCH_TEGRA || COMPILE_TEST
 	help
 	  Enable this option for integrated thermal management support on NVIDIA
 	  Tegra124 systems-on-chip. The driver supports four thermal zones
-- 
2.5.0

^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 09/17] thermal: powerclamp: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (7 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 08/17] thermal: tegra: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  4:11 ` [PATCH 10/17] thermal: intel_soc_dts_iosf: " Eduardo Valentin
                   ` (8 subsequent siblings)
  17 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to powerclamp driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index c94efd3..449508f 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -264,7 +264,7 @@ config DB8500_CPUFREQ_COOLING
 config INTEL_POWERCLAMP
 	tristate "Intel PowerClamp idle injection driver"
 	depends on THERMAL
-	depends on X86
+	depends on X86 || COMPILE_TEST
 	depends on CPU_SUP_INTEL
 	help
 	  Enable this to enable Intel PowerClamp idle injection driver. This
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 10/17] thermal: intel_soc_dts_iosf: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (8 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 09/17] thermal: powerclamp: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  6:25   ` Geert Uytterhoeven
  2015-09-10  4:11 ` [PATCH 11/17] thermal: intel_soc_dts: " Eduardo Valentin
                   ` (7 subsequent siblings)
  17 siblings, 1 reply; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to intel_soc_dts_iosf driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 449508f..eb2978d 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -284,7 +284,7 @@ config X86_PKG_TEMP_THERMAL
 
 config INTEL_SOC_DTS_IOSF_CORE
 	tristate
-	depends on X86
+	depends on X86 || COMPILE_TEST
 	select IOSF_MBI
 	help
 	  This is becoming a common feature for Intel SoCs to expose the additional
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 11/17] thermal: intel_soc_dts: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (9 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 10/17] thermal: intel_soc_dts_iosf: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  6:25   ` Geert Uytterhoeven
  2015-09-10  4:11 ` [PATCH 12/17] thermal: intel_quark: " Eduardo Valentin
                   ` (6 subsequent siblings)
  17 siblings, 1 reply; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to intel_soc_dts driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index eb2978d..027b662 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -294,7 +294,7 @@ config INTEL_SOC_DTS_IOSF_CORE
 
 config INTEL_SOC_DTS_THERMAL
 	tristate "Intel SoCs DTS thermal driver"
-	depends on X86
+	depends on X86 || COMPILE_TEST
 	select INTEL_SOC_DTS_IOSF_CORE
 	help
 	  Enable this to register Intel SoCs (e.g. Bay Trail) platform digital
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 12/17] thermal: intel_quark: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (10 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 11/17] thermal: intel_soc_dts: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  6:25   ` Geert Uytterhoeven
  2015-09-10  4:11 ` [PATCH 13/17] thermal: int340x: " Eduardo Valentin
                   ` (5 subsequent siblings)
  17 siblings, 1 reply; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to intel_quark driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 027b662..d0ff238 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -307,7 +307,7 @@ config INTEL_SOC_DTS_THERMAL
 
 config INTEL_QUARK_DTS_THERMAL
 	tristate "Intel Quark DTS thermal driver"
-	depends on X86_INTEL_QUARK
+	depends on X86_INTEL_QUARK || COMPILE_TEST
 	help
 	  Enable this to register Intel Quark SoC (e.g. X1000) platform digital
 	  temperature sensor (DTS). For X1000 SoC, it has one on-die DTS.
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 13/17] thermal: int340x: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (11 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 12/17] thermal: intel_quark: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  6:31   ` Geert Uytterhoeven
  2015-09-10  4:11 ` [PATCH 14/17] thermal: exynos: " Eduardo Valentin
                   ` (4 subsequent siblings)
  17 siblings, 1 reply; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang
  Cc: Linux PM, LKML, Eduardo Valentin, Srinivas Pandruvada, Jacob Pan,
	lan,Tianyu, Ilkka Koskinen

Adding COMPILE_TEST flag to int340x driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: Jacob Pan <jacob.jun.pan@linux.intel.com>
Cc: "lan,Tianyu" <tianyu.lan@intel.com>
Cc: Ilkka Koskinen <ilkka.koskinen@linux.intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index d0ff238..0fa5d20 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -317,7 +317,7 @@ config INTEL_QUARK_DTS_THERMAL
 
 config INT340X_THERMAL
 	tristate "ACPI INT340X thermal drivers"
-	depends on X86 && ACPI
+	depends on (X86 && ACPI) || COMPILE_TEST
 	select THERMAL_GOV_USER_SPACE
 	select ACPI_THERMAL_REL
 	select ACPI_FAN
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 14/17] thermal: exynos: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (12 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 13/17] thermal: int340x: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  4:11 ` [PATCH 15/17] thermal: st: " Eduardo Valentin
                   ` (3 subsequent siblings)
  17 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang
  Cc: Linux PM, LKML, Eduardo Valentin, Lukasz Majewski, linux-samsung-soc

Adding COMPILE_TEST flag to exynos driver to facilitate
maintenance.

Cc: Lukasz Majewski <l.majewski@samsung.com>
Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-samsung-soc@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 0fa5d20..9069cc7 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -345,7 +345,7 @@ source "drivers/thermal/ti-soc-thermal/Kconfig"
 endmenu
 
 menu "Samsung thermal drivers"
-depends on ARCH_EXYNOS
+depends on ARCH_EXYNOS || COMPILE_TEST
 source "drivers/thermal/samsung/Kconfig"
 endmenu
 
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 15/17] thermal: st: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (13 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 14/17] thermal: exynos: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-11  9:23   ` Lee Jones
  2015-09-10  4:11 ` [PATCH 16/17] thermal: qcom_spmi: " Eduardo Valentin
                   ` (2 subsequent siblings)
  17 siblings, 1 reply; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang
  Cc: Linux PM, LKML, Eduardo Valentin, Nicolas Boichat, Mark Brown,
	Fabian Frederick, Wolfram Sang, Lee Jones

Adding COMPILE_TEST flag to st driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: Nicolas Boichat <drinkcat@chromium.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Fabian Frederick <fabf@skynet.be>
Cc: Wolfram Sang <wsa@the-dreams.de>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 9069cc7..d94b4e9 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -350,7 +350,7 @@ source "drivers/thermal/samsung/Kconfig"
 endmenu
 
 menu "STMicroelectronics thermal drivers"
-depends on ARCH_STI && OF
+depends on (ARCH_STI || COMPILE_TEST) && OF
 source "drivers/thermal/st/Kconfig"
 endmenu
 
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 16/17] thermal: qcom_spmi: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (14 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 15/17] thermal: st: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  4:11 ` [PATCH 17/17] thermal: ti-soc: " Eduardo Valentin
  2015-09-10  6:23 ` [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Geert Uytterhoeven
  17 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin

Adding COMPILE_TEST flag to qcom_spmi driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index d94b4e9..a752bef 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -356,7 +356,7 @@ endmenu
 
 config QCOM_SPMI_TEMP_ALARM
 	tristate "Qualcomm SPMI PMIC Temperature Alarm"
-	depends on OF && SPMI && IIO
+	depends on OF && (SPMI || COMPILE_TEST) && IIO
 	select REGMAP_SPMI
 	help
 	  This enables a thermal sysfs driver for Qualcomm plug-and-play (QPNP)
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* [PATCH 17/17] thermal: ti-soc: allow compile test
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (15 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 16/17] thermal: qcom_spmi: " Eduardo Valentin
@ 2015-09-10  4:11 ` Eduardo Valentin
  2015-09-10  6:23 ` [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Geert Uytterhoeven
  17 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10  4:11 UTC (permalink / raw)
  To: Rui Zhang; +Cc: Linux PM, LKML, Eduardo Valentin, linux-omap

Adding COMPILE_TEST flag to ti-soc driver to facilitate
maintenance.

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-omap@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
---
 drivers/thermal/ti-soc-thermal/Kconfig | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/thermal/ti-soc-thermal/Kconfig b/drivers/thermal/ti-soc-thermal/Kconfig
index bd4c7be..7a24018 100644
--- a/drivers/thermal/ti-soc-thermal/Kconfig
+++ b/drivers/thermal/ti-soc-thermal/Kconfig
@@ -1,7 +1,7 @@
 config TI_SOC_THERMAL
 	tristate "Texas Instruments SoCs temperature sensor driver"
 	depends on THERMAL
-	depends on ARCH_HAS_BANDGAP
+	depends on ARCH_HAS_BANDGAP || COMPILE_TEST
 	help
 	  If you say yes here you get support for the Texas Instruments
 	  OMAP4460+ on die bandgap temperature sensor support. The register
@@ -24,7 +24,7 @@ config TI_THERMAL
 config OMAP4_THERMAL
 	bool "Texas Instruments OMAP4 thermal support"
 	depends on TI_SOC_THERMAL
-	depends on ARCH_OMAP4
+	depends on ARCH_OMAP4 || COMPILE_TEST
 	help
 	  If you say yes here you get thermal support for the Texas Instruments
 	  OMAP4 SoC family. The current chip supported are:
@@ -38,7 +38,7 @@ config OMAP4_THERMAL
 config OMAP5_THERMAL
 	bool "Texas Instruments OMAP5 thermal support"
 	depends on TI_SOC_THERMAL
-	depends on SOC_OMAP5
+	depends on SOC_OMAP5 || COMPILE_TEST
 	help
 	  If you say yes here you get thermal support for the Texas Instruments
 	  OMAP5 SoC family. The current chip supported are:
@@ -50,7 +50,7 @@ config OMAP5_THERMAL
 config DRA752_THERMAL
 	bool "Texas Instruments DRA752 thermal support"
 	depends on TI_SOC_THERMAL
-	depends on SOC_DRA7XX
+	depends on SOC_DRA7XX || COMPILE_TEST
 	help
 	  If you say yes here you get thermal support for the Texas Instruments
 	  DRA752 SoC family. The current chip supported are:
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH 02/17] thermal: spear: allow compile test
  2015-09-10  4:10 ` [PATCH 02/17] thermal: spear: " Eduardo Valentin
@ 2015-09-10  4:23   ` Viresh Kumar
  0 siblings, 0 replies; 34+ messages in thread
From: Viresh Kumar @ 2015-09-10  4:23 UTC (permalink / raw)
  To: Eduardo Valentin; +Cc: Rui Zhang, Linux PM, LKML

On Thu, Sep 10, 2015 at 9:40 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> Adding COMPILE_TEST flag to spear driver to facilitate
> maintenance.
>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: linux-pm@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
> ---
>  drivers/thermal/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index f2b1d31..0334886 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -182,7 +182,7 @@ config IMX_THERMAL
>
>  config SPEAR_THERMAL
>         bool "SPEAr thermal sensor driver"
> -       depends on PLAT_SPEAR
> +       depends on PLAT_SPEAR || COMPILE_TEST
>         depends on OF
>         help
>           Enable this to plug the SPEAr thermal sensor driver into the Linux

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 00/17] thermal: add COMPILE_TEST (on drivers)
  2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
                   ` (16 preceding siblings ...)
  2015-09-10  4:11 ` [PATCH 17/17] thermal: ti-soc: " Eduardo Valentin
@ 2015-09-10  6:23 ` Geert Uytterhoeven
  2015-09-10 14:56   ` Eduardo Valentin
  17 siblings, 1 reply; 34+ messages in thread
From: Geert Uytterhoeven @ 2015-09-10  6:23 UTC (permalink / raw)
  To: Eduardo Valentin; +Cc: Rui Zhang, Linux PM, LKML

Hi Eduardo,

On Thu, Sep 10, 2015 at 6:10 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> This is a simple series to add the COMPILE_TEST on thermal drivers.
> The idea is to help compiling and maintaining.

Thanks, good idea!

> Split into smaller patches so people can have their own voices on their drivers.
>
> Let me know if this is a problem.

Have you compile-tested this?
I'm seeing too many failures.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 10/17] thermal: intel_soc_dts_iosf: allow compile test
  2015-09-10  4:11 ` [PATCH 10/17] thermal: intel_soc_dts_iosf: " Eduardo Valentin
@ 2015-09-10  6:25   ` Geert Uytterhoeven
  0 siblings, 0 replies; 34+ messages in thread
From: Geert Uytterhoeven @ 2015-09-10  6:25 UTC (permalink / raw)
  To: Eduardo Valentin; +Cc: Rui Zhang, Linux PM, LKML

On Thu, Sep 10, 2015 at 6:11 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -284,7 +284,7 @@ config X86_PKG_TEMP_THERMAL
>
>  config INTEL_SOC_DTS_IOSF_CORE
>         tristate
> -       depends on X86
> +       depends on X86 || COMPILE_TEST

drivers/thermal/intel_soc_dts_iosf.c:21:26: error: asm/iosf_mbi.h: No
such file or directory

x86-only header file

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 11/17] thermal: intel_soc_dts: allow compile test
  2015-09-10  4:11 ` [PATCH 11/17] thermal: intel_soc_dts: " Eduardo Valentin
@ 2015-09-10  6:25   ` Geert Uytterhoeven
  0 siblings, 0 replies; 34+ messages in thread
From: Geert Uytterhoeven @ 2015-09-10  6:25 UTC (permalink / raw)
  To: Eduardo Valentin; +Cc: Rui Zhang, Linux PM, LKML

On Thu, Sep 10, 2015 at 6:11 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -294,7 +294,7 @@ config INTEL_SOC_DTS_IOSF_CORE
>
>  config INTEL_SOC_DTS_THERMAL
>         tristate "Intel SoCs DTS thermal driver"
> -       depends on X86
> +       depends on X86 || COMPILE_TEST

drivers/thermal/intel_soc_dts_thermal.c:20:31: error:
asm/cpu_device_id.h: No such file or directory

x86-only header file

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 12/17] thermal: intel_quark: allow compile test
  2015-09-10  4:11 ` [PATCH 12/17] thermal: intel_quark: " Eduardo Valentin
@ 2015-09-10  6:25   ` Geert Uytterhoeven
  0 siblings, 0 replies; 34+ messages in thread
From: Geert Uytterhoeven @ 2015-09-10  6:25 UTC (permalink / raw)
  To: Eduardo Valentin; +Cc: Rui Zhang, Linux PM, LKML

On Thu, Sep 10, 2015 at 6:11 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -307,7 +307,7 @@ config INTEL_SOC_DTS_THERMAL
>
>  config INTEL_QUARK_DTS_THERMAL
>         tristate "Intel Quark DTS thermal driver"
> -       depends on X86_INTEL_QUARK
> +       depends on X86_INTEL_QUARK || COMPILE_TEST

drivers/thermal/intel_quark_dts_thermal.c:64:31: error:
asm/cpu_device_id.h: No such file or directory
drivers/thermal/intel_quark_dts_thermal.c:65:26: error:
asm/iosf_mbi.h: No such file or directory

x86-only header files

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 13/17] thermal: int340x: allow compile test
  2015-09-10  4:11 ` [PATCH 13/17] thermal: int340x: " Eduardo Valentin
@ 2015-09-10  6:31   ` Geert Uytterhoeven
  2015-09-17 22:48     ` Srinivas Pandruvada
  0 siblings, 1 reply; 34+ messages in thread
From: Geert Uytterhoeven @ 2015-09-10  6:31 UTC (permalink / raw)
  To: Eduardo Valentin
  Cc: Rui Zhang, Linux PM, LKML, Srinivas Pandruvada, Jacob Pan,
	lan,Tianyu, Ilkka Koskinen

On Thu, Sep 10, 2015 at 6:11 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -317,7 +317,7 @@ config INTEL_QUARK_DTS_THERMAL
>
>  config INT340X_THERMAL
>         tristate "ACPI INT340X thermal drivers"
> -       depends on X86 && ACPI
> +       depends on (X86 && ACPI) || COMPILE_TEST
>         select THERMAL_GOV_USER_SPACE
>         select ACPI_THERMAL_REL
>         select ACPI_FAN

This driver selects INTEL_SOC_DTS_IOSF_CORE, which uses an x86-specific
header file.

drivers/thermal/int340x_thermal/int3400_thermal.c: In function
‘int3400_thermal_get_uuids’:
drivers/thermal/int340x_thermal/int3400_thermal.c:120: error:
dereferencing pointer to incomplete type

Missing #include <linux/fs.h>

drivers/thermal/int340x_thermal/acpi_thermal_rel.c: In function
‘acpi_parse_trt’:
drivers/thermal/int340x_thermal/acpi_thermal_rel.c:85: error: implicit
declaration of function ‘acpi_has_method’

Missing #include <acpi/acpi_bus.h>

drivers/thermal/int340x_thermal/acpi_thermal_rel.c: In function
‘acpi_parse_trt’:
drivers/thermal/int340x_thermal/acpi_thermal_rel.c:125: error:
implicit declaration of function ‘acpi_bus_get_device’

Definition depends on CONFIG_ACPI

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 06/17] thermal: u8500: allow compile test
  2015-09-10  4:11 ` [PATCH 06/17] thermal: u8500: " Eduardo Valentin
@ 2015-09-10  6:35   ` Geert Uytterhoeven
  0 siblings, 0 replies; 34+ messages in thread
From: Geert Uytterhoeven @ 2015-09-10  6:35 UTC (permalink / raw)
  To: Eduardo Valentin; +Cc: Rui Zhang, Linux PM, LKML

On Thu, Sep 10, 2015 at 6:11 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -224,7 +224,7 @@ config DOVE_THERMAL
>
>  config DB8500_THERMAL
>         bool "DB8500 thermal management"
> -       depends on ARCH_U8500
> +       depends on ARCH_U8500 || COMPILE_TEST

In file included from drivers/thermal/db8500_thermal.c:23:
include/linux/mfd/dbx500-prcmu.h:437: error: redefinition of ‘prcmu_abb_read’
include/linux/mfd/db8500-prcmu.h:648: error: previous definition of
‘prcmu_abb_read’ was here
include/linux/mfd/dbx500-prcmu.h:442: error: redefinition of ‘prcmu_abb_write’
include/linux/mfd/db8500-prcmu.h:653: error: previous definition of
‘prcmu_abb_write’ was here
include/linux/mfd/dbx500-prcmu.h:453: error: redefinition of
‘prcmu_config_clkout’
include/linux/mfd/db8500-prcmu.h:618: error: previous definition of
‘prcmu_config_clkout’ was here
include/linux/mfd/dbx500-prcmu.h:520: error: redefinition of ‘prcmu_ac_wake_req’
include/linux/mfd/db8500-prcmu.h:658: error: previous definition of
‘prcmu_ac_wake_req’ was here
include/linux/mfd/dbx500-prcmu.h:524: error: redefinition of
‘prcmu_ac_sleep_req’
include/linux/mfd/db8500-prcmu.h:662: error: previous definition of
‘prcmu_ac_sleep_req’ was here

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 08/17] thermal: tegra: allow compile test
  2015-09-10  4:11 ` [PATCH 08/17] thermal: tegra: " Eduardo Valentin
@ 2015-09-10  6:38       ` Geert Uytterhoeven
  0 siblings, 0 replies; 34+ messages in thread
From: Geert Uytterhoeven @ 2015-09-10  6:38 UTC (permalink / raw)
  To: Eduardo Valentin
  Cc: Rui Zhang, Linux PM, LKML, Stephen Warren, Thierry Reding,
	Alexandre Courbot, linux-tegra-u79uwXL29TY76Z2rM5mHXA

On Thu, Sep 10, 2015 at 6:11 AM, Eduardo Valentin <edubezval-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -242,7 +242,7 @@ config ARMADA_THERMAL
>
>  config TEGRA_SOCTHERM
>         tristate "Tegra SOCTHERM thermal management"
> -       depends on ARCH_TEGRA
> +       depends on ARCH_TEGRA || COMPILE_TEST

ERROR: "tegra_fuse_readl" [drivers/thermal/tegra_soctherm.ko] undefined!

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 08/17] thermal: tegra: allow compile test
@ 2015-09-10  6:38       ` Geert Uytterhoeven
  0 siblings, 0 replies; 34+ messages in thread
From: Geert Uytterhoeven @ 2015-09-10  6:38 UTC (permalink / raw)
  To: Eduardo Valentin
  Cc: Rui Zhang, Linux PM, LKML, Stephen Warren, Thierry Reding,
	Alexandre Courbot, linux-tegra

On Thu, Sep 10, 2015 at 6:11 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -242,7 +242,7 @@ config ARMADA_THERMAL
>
>  config TEGRA_SOCTHERM
>         tristate "Tegra SOCTHERM thermal management"
> -       depends on ARCH_TEGRA
> +       depends on ARCH_TEGRA || COMPILE_TEST

ERROR: "tegra_fuse_readl" [drivers/thermal/tegra_soctherm.ko] undefined!

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 00/17] thermal: add COMPILE_TEST (on drivers)
  2015-09-10  6:23 ` [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Geert Uytterhoeven
@ 2015-09-10 14:56   ` Eduardo Valentin
  2015-09-10 15:34     ` Geert Uytterhoeven
  0 siblings, 1 reply; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-10 14:56 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Rui Zhang, Linux PM, LKML

[-- Attachment #1: Type: text/plain, Size: 1278 bytes --]

On Thu, Sep 10, 2015 at 08:23:45AM +0200, Geert Uytterhoeven wrote:
> Hi Eduardo,
> 
> On Thu, Sep 10, 2015 at 6:10 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> > This is a simple series to add the COMPILE_TEST on thermal drivers.
> > The idea is to help compiling and maintaining.
> 
> Thanks, good idea!
> 
> > Split into smaller patches so people can have their own voices on their drivers.
> >
> > Let me know if this is a problem.
> 
> Have you compile-tested this?
> I'm seeing too many failures.

Yes I did, but apparently I missed some drivers in my test. I had x86
target while compile arm drivers, thats probably why I did not see the
cpuid problems. I am trying again, as I really did not see all errors
you mentioned on arm drivers.

That generating errors will need to be left out of the series and should
have its own set of changes to allow compile test.

> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 490 bytes --]

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 00/17] thermal: add COMPILE_TEST (on drivers)
  2015-09-10 14:56   ` Eduardo Valentin
@ 2015-09-10 15:34     ` Geert Uytterhoeven
  0 siblings, 0 replies; 34+ messages in thread
From: Geert Uytterhoeven @ 2015-09-10 15:34 UTC (permalink / raw)
  To: Eduardo Valentin; +Cc: Rui Zhang, Linux PM, LKML

Hi Eduardo,

On Thu, Sep 10, 2015 at 4:56 PM, Eduardo Valentin <edubezval@gmail.com> wrote:
> On Thu, Sep 10, 2015 at 08:23:45AM +0200, Geert Uytterhoeven wrote:
>> On Thu, Sep 10, 2015 at 6:10 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
>> > This is a simple series to add the COMPILE_TEST on thermal drivers.
>> > The idea is to help compiling and maintaining.
>>
>> Thanks, good idea!
>>
>> > Split into smaller patches so people can have their own voices on their drivers.
>> >
>> > Let me know if this is a problem.
>>
>> Have you compile-tested this?
>> I'm seeing too many failures.
>
> Yes I did, but apparently I missed some drivers in my test. I had x86
> target while compile arm drivers, thats probably why I did not see the
> cpuid problems. I am trying again, as I really did not see all errors
> you mentioned on arm drivers.

I compiled on m68k, to make sure none of the drivers are native ;-)

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 15/17] thermal: st: allow compile test
  2015-09-10  4:11 ` [PATCH 15/17] thermal: st: " Eduardo Valentin
@ 2015-09-11  9:23   ` Lee Jones
  2015-09-14  2:11     ` Eduardo Valentin
  0 siblings, 1 reply; 34+ messages in thread
From: Lee Jones @ 2015-09-11  9:23 UTC (permalink / raw)
  To: Eduardo Valentin
  Cc: Rui Zhang, Linux PM, LKML, Nicolas Boichat, Mark Brown,
	Fabian Frederick, Wolfram Sang

On Wed, 09 Sep 2015, Eduardo Valentin wrote:

> Adding COMPILE_TEST flag to st driver to facilitate
> maintenance.
> 
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Nicolas Boichat <drinkcat@chromium.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Fabian Frederick <fabf@skynet.be>
> Cc: Wolfram Sang <wsa@the-dreams.de>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: linux-pm@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
> ---
>  drivers/thermal/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Sounds reasonable.

So long as it's been tested and nothing breaks:

Acked-by: Lee Jones <lee.jones@linaro.org>

> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index 9069cc7..d94b4e9 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -350,7 +350,7 @@ source "drivers/thermal/samsung/Kconfig"
>  endmenu
>  
>  menu "STMicroelectronics thermal drivers"
> -depends on ARCH_STI && OF
> +depends on (ARCH_STI || COMPILE_TEST) && OF
>  source "drivers/thermal/st/Kconfig"
>  endmenu
>  

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 15/17] thermal: st: allow compile test
  2015-09-11  9:23   ` Lee Jones
@ 2015-09-14  2:11     ` Eduardo Valentin
  0 siblings, 0 replies; 34+ messages in thread
From: Eduardo Valentin @ 2015-09-14  2:11 UTC (permalink / raw)
  To: Lee Jones
  Cc: Rui Zhang, Linux PM, LKML, Nicolas Boichat, Mark Brown,
	Fabian Frederick, Wolfram Sang

[-- Attachment #1: Type: text/plain, Size: 1636 bytes --]

On Fri, Sep 11, 2015 at 10:23:35AM +0100, Lee Jones wrote:
> On Wed, 09 Sep 2015, Eduardo Valentin wrote:
> 
> > Adding COMPILE_TEST flag to st driver to facilitate
> > maintenance.
> > 
> > Cc: Zhang Rui <rui.zhang@intel.com>
> > Cc: Nicolas Boichat <drinkcat@chromium.org>
> > Cc: Mark Brown <broonie@kernel.org>
> > Cc: Fabian Frederick <fabf@skynet.be>
> > Cc: Wolfram Sang <wsa@the-dreams.de>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: linux-pm@vger.kernel.org
> > Cc: linux-kernel@vger.kernel.org
> > Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
> > ---
> >  drivers/thermal/Kconfig | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Sounds reasonable.
> 
> So long as it's been tested and nothing breaks:
> 
> Acked-by: Lee Jones <lee.jones@linaro.org>

Thanks for the review. But now checking the compilation test using bfin
as target, I see several compilation issues.

So, I am leaving this one out for now. It requires further changes.

BR,

> 
> > diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> > index 9069cc7..d94b4e9 100644
> > --- a/drivers/thermal/Kconfig
> > +++ b/drivers/thermal/Kconfig
> > @@ -350,7 +350,7 @@ source "drivers/thermal/samsung/Kconfig"
> >  endmenu
> >  
> >  menu "STMicroelectronics thermal drivers"
> > -depends on ARCH_STI && OF
> > +depends on (ARCH_STI || COMPILE_TEST) && OF
> >  source "drivers/thermal/st/Kconfig"
> >  endmenu
> >  
> 
> -- 
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 490 bytes --]

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH 13/17] thermal: int340x: allow compile test
  2015-09-10  6:31   ` Geert Uytterhoeven
@ 2015-09-17 22:48     ` Srinivas Pandruvada
  0 siblings, 0 replies; 34+ messages in thread
From: Srinivas Pandruvada @ 2015-09-17 22:48 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Eduardo Valentin, Rui Zhang, Linux PM, LKML, Jacob Pan,
	lan,Tianyu, Ilkka Koskinen

On Thu, 2015-09-10 at 08:31 +0200, Geert Uytterhoeven wrote:
> On Thu, Sep 10, 2015 at 6:11 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> > --- a/drivers/thermal/Kconfig
> > +++ b/drivers/thermal/Kconfig
> > @@ -317,7 +317,7 @@ config INTEL_QUARK_DTS_THERMAL
> >
> >  config INT340X_THERMAL
> >         tristate "ACPI INT340X thermal drivers"
> > -       depends on X86 && ACPI
> > +       depends on (X86 && ACPI) || COMPILE_TEST
> >         select THERMAL_GOV_USER_SPACE
> >         select ACPI_THERMAL_REL
> >         select ACPI_FAN
> 
> This driver selects INTEL_SOC_DTS_IOSF_CORE, which uses an x86-specific
> header file.
> 
> drivers/thermal/int340x_thermal/int3400_thermal.c: In function
> ‘int3400_thermal_get_uuids’:
> drivers/thermal/int340x_thermal/int3400_thermal.c:120: error:
> dereferencing pointer to incomplete type
> 
> Missing #include <linux/fs.h>
> 
> drivers/thermal/int340x_thermal/acpi_thermal_rel.c: In function
> ‘acpi_parse_trt’:
> drivers/thermal/int340x_thermal/acpi_thermal_rel.c:85: error: implicit
> declaration of function ‘acpi_has_method’
> 
> Missing #include <acpi/acpi_bus.h>
> 
> drivers/thermal/int340x_thermal/acpi_thermal_rel.c: In function
> ‘acpi_parse_trt’:
> drivers/thermal/int340x_thermal/acpi_thermal_rel.c:125: error:
> implicit declaration of function ‘acpi_bus_get_device’
> 
> Definition depends on CONFIG_ACPI
> 
Yes. This has dependency on ACPI. We need to also include ACPI modules
to build. For IOSF we need to have stub functions for core calls.

Thanks,
Srinivas

> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds
> --
> 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



^ permalink raw reply	[flat|nested] 34+ messages in thread

end of thread, other threads:[~2015-09-17 22:48 UTC | newest]

Thread overview: 34+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-10  4:10 [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin
2015-09-10  4:10 ` [PATCH 01/17] thermal: hisi: allow compile test Eduardo Valentin
2015-09-10  4:10 ` [PATCH 02/17] thermal: spear: " Eduardo Valentin
2015-09-10  4:23   ` Viresh Kumar
2015-09-10  4:10 ` [PATCH 03/17] thermal: rockchip: " Eduardo Valentin
2015-09-10  4:10   ` Eduardo Valentin
2015-09-10  4:10   ` Eduardo Valentin
2015-09-10  4:11 ` [PATCH 04/17] thermal: kirkwood: " Eduardo Valentin
2015-09-10  4:11 ` [PATCH 05/17] thermal: dove: " Eduardo Valentin
2015-09-10  4:11 ` [PATCH 06/17] thermal: u8500: " Eduardo Valentin
2015-09-10  6:35   ` Geert Uytterhoeven
2015-09-10  4:11 ` [PATCH 07/17] thermal: armada: " Eduardo Valentin
2015-09-10  4:11 ` [PATCH 08/17] thermal: tegra: " Eduardo Valentin
     [not found]   ` <1441858273-4017-9-git-send-email-edubezval-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-09-10  6:38     ` Geert Uytterhoeven
2015-09-10  6:38       ` Geert Uytterhoeven
2015-09-10  4:11 ` [PATCH 09/17] thermal: powerclamp: " Eduardo Valentin
2015-09-10  4:11 ` [PATCH 10/17] thermal: intel_soc_dts_iosf: " Eduardo Valentin
2015-09-10  6:25   ` Geert Uytterhoeven
2015-09-10  4:11 ` [PATCH 11/17] thermal: intel_soc_dts: " Eduardo Valentin
2015-09-10  6:25   ` Geert Uytterhoeven
2015-09-10  4:11 ` [PATCH 12/17] thermal: intel_quark: " Eduardo Valentin
2015-09-10  6:25   ` Geert Uytterhoeven
2015-09-10  4:11 ` [PATCH 13/17] thermal: int340x: " Eduardo Valentin
2015-09-10  6:31   ` Geert Uytterhoeven
2015-09-17 22:48     ` Srinivas Pandruvada
2015-09-10  4:11 ` [PATCH 14/17] thermal: exynos: " Eduardo Valentin
2015-09-10  4:11 ` [PATCH 15/17] thermal: st: " Eduardo Valentin
2015-09-11  9:23   ` Lee Jones
2015-09-14  2:11     ` Eduardo Valentin
2015-09-10  4:11 ` [PATCH 16/17] thermal: qcom_spmi: " Eduardo Valentin
2015-09-10  4:11 ` [PATCH 17/17] thermal: ti-soc: " Eduardo Valentin
2015-09-10  6:23 ` [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Geert Uytterhoeven
2015-09-10 14:56   ` Eduardo Valentin
2015-09-10 15:34     ` Geert Uytterhoeven

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.