Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1221882 > unrolled thread

[PATCH 00/17] thermal: add COMPILE_TEST (on drivers)

Started byEduardo Valentin <edubezval@gmail.com>
First post2015-09-10 06:20 +0200
Last post2015-09-10 17:40 +0200
Articles 20 on this page of 24 — 5 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    [PATCH 02/17] thermal: spear: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
      Re: [PATCH 02/17] thermal: spear: allow compile test Viresh Kumar <viresh.kumar@linaro.org> - 2015-09-10 06:30 +0200
    [PATCH 08/17] thermal: tegra: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
      Re: [PATCH 08/17] thermal: tegra: allow compile test Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:40 +0200
    [PATCH 12/17] thermal: intel_quark: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
      Re: [PATCH 12/17] thermal: intel_quark: allow compile test Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:30 +0200
    [PATCH 11/17] thermal: intel_soc_dts: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
      Re: [PATCH 11/17] thermal: intel_soc_dts: allow compile test Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:30 +0200
    [PATCH 09/17] thermal: powerclamp: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    [PATCH 01/17] thermal: hisi: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    [PATCH 13/17] thermal: int340x: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
      Re: [PATCH 13/17] thermal: int340x: allow compile test Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:40 +0200
        Re: [PATCH 13/17] thermal: int340x: allow compile test Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> - 2015-09-18 00:50 +0200
    [PATCH 17/17] thermal: ti-soc: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    [PATCH 16/17] thermal: qcom_spmi: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    [PATCH 14/17] thermal: exynos: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    [PATCH 15/17] thermal: st: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
      Re: [PATCH 15/17] thermal: st: allow compile test Lee Jones <lee.jones@linaro.org> - 2015-09-11 11:30 +0200
        Re: [PATCH 15/17] thermal: st: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-14 04:20 +0200
    [PATCH 05/17] thermal: dove: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    Re: [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:30 +0200
      Re: [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 17:00 +0200
        Re: [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 17:40 +0200

Page 1 of 2  [1] 2  Next page →

#1221882 — [PATCH 00/17] thermal: add COMPILE_TEST (on drivers)

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 00/17] thermal: add COMPILE_TEST (on drivers)
Message-ID<q71v3-7rK-7@gated-at.bofh.it>
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

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

[toc] | [next] | [standalone]

#1221883 — [PATCH 02/17] thermal: spear: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 02/17] thermal: spear: allow compile test
Message-ID<q71v3-7rK-17@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221900 — Re: [PATCH 02/17] thermal: spear: allow compile test

FromViresh Kumar <viresh.kumar@linaro.org>
Date2015-09-10 06:30 +0200
SubjectRe: [PATCH 02/17] thermal: spear: allow compile test
Message-ID<q71EK-7Fd-3@gated-at.bofh.it>
In reply to#1221883
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>
--
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/

[toc] | [prev] | [next] | [standalone]

#1221884 — [PATCH 08/17] thermal: tegra: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 08/17] thermal: tegra: allow compile test
Message-ID<q71v3-7rK-19@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221950 — Re: [PATCH 08/17] thermal: tegra: allow compile test

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2015-09-10 08:40 +0200
SubjectRe: [PATCH 08/17] thermal: tegra: allow compile test
Message-ID<q73Gy-24B-7@gated-at.bofh.it>
In reply to#1221884
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
--
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/

[toc] | [prev] | [next] | [standalone]

#1221885 — [PATCH 12/17] thermal: intel_quark: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 12/17] thermal: intel_quark: allow compile test
Message-ID<q71v4-7rK-23@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221945 — Re: [PATCH 12/17] thermal: intel_quark: allow compile test

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2015-09-10 08:30 +0200
SubjectRe: [PATCH 12/17] thermal: intel_quark: allow compile test
Message-ID<q73wS-1Tm-7@gated-at.bofh.it>
In reply to#1221885
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
--
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/

[toc] | [prev] | [next] | [standalone]

#1221888 — [PATCH 11/17] thermal: intel_soc_dts: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 11/17] thermal: intel_soc_dts: allow compile test
Message-ID<q71v4-7rK-29@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221946 — Re: [PATCH 11/17] thermal: intel_soc_dts: allow compile test

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2015-09-10 08:30 +0200
SubjectRe: [PATCH 11/17] thermal: intel_soc_dts: allow compile test
Message-ID<q73wS-1Tm-11@gated-at.bofh.it>
In reply to#1221888
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
--
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/

[toc] | [prev] | [next] | [standalone]

#1221889 — [PATCH 09/17] thermal: powerclamp: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 09/17] thermal: powerclamp: allow compile test
Message-ID<q71v4-7rK-35@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221890 — [PATCH 01/17] thermal: hisi: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 01/17] thermal: hisi: allow compile test
Message-ID<q71v4-7rK-39@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221892 — [PATCH 13/17] thermal: int340x: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 13/17] thermal: int340x: allow compile test
Message-ID<q71v4-7rK-43@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221952 — Re: [PATCH 13/17] thermal: int340x: allow compile test

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2015-09-10 08:40 +0200
SubjectRe: [PATCH 13/17] thermal: int340x: allow compile test
Message-ID<q73Gy-24B-11@gated-at.bofh.it>
In reply to#1221892
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
--
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/

[toc] | [prev] | [next] | [standalone]

#1227470 — Re: [PATCH 13/17] thermal: int340x: allow compile test

FromSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Date2015-09-18 00:50 +0200
SubjectRe: [PATCH 13/17] thermal: int340x: allow compile test
Message-ID<q9Qa6-OT-19@gated-at.bofh.it>
In reply to#1221952
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


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

[toc] | [prev] | [next] | [standalone]

#1221894 — [PATCH 17/17] thermal: ti-soc: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 17/17] thermal: ti-soc: allow compile test
Message-ID<q71v5-7rK-45@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221895 — [PATCH 16/17] thermal: qcom_spmi: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 16/17] thermal: qcom_spmi: allow compile test
Message-ID<q71v5-7rK-47@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221896 — [PATCH 14/17] thermal: exynos: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 14/17] thermal: exynos: allow compile test
Message-ID<q71v6-7rK-55@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1221897 — [PATCH 15/17] thermal: st: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-10 06:20 +0200
Subject[PATCH 15/17] thermal: st: allow compile test
Message-ID<q71v5-7rK-51@gated-at.bofh.it>
In reply to#1221882
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

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

[toc] | [prev] | [next] | [standalone]

#1222646 — Re: [PATCH 15/17] thermal: st: allow compile test

FromLee Jones <lee.jones@linaro.org>
Date2015-09-11 11:30 +0200
SubjectRe: [PATCH 15/17] thermal: st: allow compile test
Message-ID<q7sOD-5Sc-39@gated-at.bofh.it>
In reply to#1221897
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
--
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/

[toc] | [prev] | [next] | [standalone]

#1223794 — Re: [PATCH 15/17] thermal: st: allow compile test

FromEduardo Valentin <edubezval@gmail.com>
Date2015-09-14 04:20 +0200
SubjectRe: [PATCH 15/17] thermal: st: allow compile test
Message-ID<q8rx8-UX-5@gated-at.bofh.it>
In reply to#1222646

[Multipart message — attachments visible in raw view] — view raw

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

[toc] | [prev] | [next] | [standalone]

Page 1 of 2  [1] 2  Next page →

Back to top | Article view | linux.kernel

csiph-web