Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1308134 > unrolled thread
| Started by | Wei Ni <wni@nvidia.com> |
|---|---|
| First post | 2016-01-13 09:00 +0100 |
| Last post | 2016-01-14 06:40 +0100 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH V1 01/10] thermal: tegra: move tegra thermal files into tegra directory Wei Ni <wni@nvidia.com> - 2016-01-13 09:00 +0100
Re: [PATCH V1 01/10] thermal: tegra: move tegra thermal files into tegra directory Thierry Reding <thierry.reding@gmail.com> - 2016-01-13 15:30 +0100
Re: [PATCH V1 01/10] thermal: tegra: move tegra thermal files into tegra directory Wei Ni <wni@nvidia.com> - 2016-01-14 06:40 +0100
| From | Wei Ni <wni@nvidia.com> |
|---|---|
| Date | 2016-01-13 09:00 +0100 |
| Subject | [PATCH V1 01/10] thermal: tegra: move tegra thermal files into tegra directory |
| Message-ID | <qQovv-4Vo-5@gated-at.bofh.it> |
Move tegra soctherm driver to tegra directory, it's easy to maintain
and add more new function support for tegra platforms.
This will also help to split soctherm driver into common parts and
chip specific data related parts.
Signed-off-by: Wei Ni <wni@nvidia.com>
---
drivers/thermal/Kconfig | 15 +++++----------
drivers/thermal/Makefile | 2 +-
drivers/thermal/tegra/Kconfig | 9 +++++++++
drivers/thermal/tegra/Makefile | 6 ++++++
drivers/thermal/{ => tegra}/tegra_soctherm.c | 0
5 files changed, 21 insertions(+), 11 deletions(-)
create mode 100644 drivers/thermal/tegra/Kconfig
create mode 100644 drivers/thermal/tegra/Makefile
rename drivers/thermal/{ => tegra}/tegra_soctherm.c (100%)
diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index c463c89b90ef..7984cba6b340 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -254,16 +254,6 @@ config ARMADA_THERMAL
Enable this option if you want to have support for thermal management
controller present in Armada 370 and Armada XP SoC.
-config TEGRA_SOCTHERM
- tristate "Tegra SOCTHERM thermal management"
- depends on ARCH_TEGRA
- help
- Enable this option for integrated thermal management support on NVIDIA
- Tegra124 systems-on-chip. The driver supports four thermal zones
- (CPU, GPU, MEM, PLLX). Cooling devices can be bound to the thermal
- zones to manage temperatures. This option is also required for the
- emergency thermal reset (thermtrip) feature to function.
-
config DB8500_CPUFREQ_COOLING
tristate "DB8500 cpufreq cooling"
depends on ARCH_U8500
@@ -380,6 +370,11 @@ depends on ARCH_STI && OF
source "drivers/thermal/st/Kconfig"
endmenu
+menu "Tegra thermal drivers"
+depends on ARCH_TEGRA
+source "drivers/thermal/tegra/Kconfig"
+endmenu
+
config QCOM_SPMI_TEMP_ALARM
tristate "Qualcomm SPMI PMIC Temperature Alarm"
depends on OF && (SPMI || COMPILE_TEST) && IIO
diff --git a/drivers/thermal/Makefile b/drivers/thermal/Makefile
index cfae6a654793..119e25cdcc66 100644
--- a/drivers/thermal/Makefile
+++ b/drivers/thermal/Makefile
@@ -46,5 +46,5 @@ obj-$(CONFIG_TI_SOC_THERMAL) += ti-soc-thermal/
obj-$(CONFIG_INT340X_THERMAL) += int340x_thermal/
obj-$(CONFIG_INTEL_PCH_THERMAL) += intel_pch_thermal.o
obj-$(CONFIG_ST_THERMAL) += st/
-obj-$(CONFIG_TEGRA_SOCTHERM) += tegra_soctherm.o
+obj-$(CONFIG_TEGRA_SOCTHERM) += tegra/
obj-$(CONFIG_HISI_THERMAL) += hisi_thermal.o
diff --git a/drivers/thermal/tegra/Kconfig b/drivers/thermal/tegra/Kconfig
new file mode 100644
index 000000000000..a6e6cd4528dc
--- /dev/null
+++ b/drivers/thermal/tegra/Kconfig
@@ -0,0 +1,9 @@
+config TEGRA_SOCTHERM
+ tristate "Tegra SOCTHERM thermal management"
+ depends on ARCH_TEGRA
+ help
+ Enable this option for integrated thermal management support on NVIDIA
+ Tegra124 systems-on-chip. The driver supports four thermal zones
+ (CPU, GPU, MEM, PLLX). Cooling devices can be bound to the thermal
+ zones to manage temperatures. This option is also required for the
+ emergency thermal reset (thermtrip) feature to function.
diff --git a/drivers/thermal/tegra/Makefile b/drivers/thermal/tegra/Makefile
new file mode 100644
index 000000000000..8c51076e4b1e
--- /dev/null
+++ b/drivers/thermal/tegra/Makefile
@@ -0,0 +1,6 @@
+#
+# Tegra thermal specific Makefile
+#
+
+# Tegra soc thermal drivers
+obj-$(CONFIG_TEGRA_SOCTHERM) += tegra_soctherm.o
diff --git a/drivers/thermal/tegra_soctherm.c b/drivers/thermal/tegra/tegra_soctherm.c
similarity index 100%
rename from drivers/thermal/tegra_soctherm.c
rename to drivers/thermal/tegra/tegra_soctherm.c
--
1.9.1
[toc] | [next] | [standalone]
| From | Thierry Reding <thierry.reding@gmail.com> |
|---|---|
| Date | 2016-01-13 15:30 +0100 |
| Subject | Re: [PATCH V1 01/10] thermal: tegra: move tegra thermal files into tegra directory |
| Message-ID | <qQuAW-X1-21@gated-at.bofh.it> |
| In reply to | #1308134 |
[Multipart message — attachments visible in raw view] — view raw
On Wed, Jan 13, 2016 at 03:58:40PM +0800, Wei Ni wrote:
> Move tegra soctherm driver to tegra directory, it's easy to maintain
> and add more new function support for tegra platforms.
Please use the proper spelling "Tegra", except where you refer to
directory or file names.
> This will also help to split soctherm driver into common parts and
> chip specific data related parts.
>
> Signed-off-by: Wei Ni <wni@nvidia.com>
> ---
> drivers/thermal/Kconfig | 15 +++++----------
> drivers/thermal/Makefile | 2 +-
> drivers/thermal/tegra/Kconfig | 9 +++++++++
> drivers/thermal/tegra/Makefile | 6 ++++++
> drivers/thermal/{ => tegra}/tegra_soctherm.c | 0
> 5 files changed, 21 insertions(+), 11 deletions(-)
> create mode 100644 drivers/thermal/tegra/Kconfig
> create mode 100644 drivers/thermal/tegra/Makefile
> rename drivers/thermal/{ => tegra}/tegra_soctherm.c (100%)
>
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index c463c89b90ef..7984cba6b340 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -254,16 +254,6 @@ config ARMADA_THERMAL
> Enable this option if you want to have support for thermal management
> controller present in Armada 370 and Armada XP SoC.
>
> -config TEGRA_SOCTHERM
> - tristate "Tegra SOCTHERM thermal management"
> - depends on ARCH_TEGRA
> - help
> - Enable this option for integrated thermal management support on NVIDIA
> - Tegra124 systems-on-chip. The driver supports four thermal zones
> - (CPU, GPU, MEM, PLLX). Cooling devices can be bound to the thermal
> - zones to manage temperatures. This option is also required for the
> - emergency thermal reset (thermtrip) feature to function.
> -
> config DB8500_CPUFREQ_COOLING
> tristate "DB8500 cpufreq cooling"
> depends on ARCH_U8500
> @@ -380,6 +370,11 @@ depends on ARCH_STI && OF
> source "drivers/thermal/st/Kconfig"
> endmenu
>
> +menu "Tegra thermal drivers"
> +depends on ARCH_TEGRA
> +source "drivers/thermal/tegra/Kconfig"
> +endmenu
> +
I think it'd be more idiomatic to only source the file here and move the
menu declaration and the dependencies into that file. Perhaps also make
the menu description "NVIDIA Tegra thermal drivers", I think we've been
using that consistently elsewhere.
> diff --git a/drivers/thermal/tegra/Kconfig b/drivers/thermal/tegra/Kconfig
> new file mode 100644
> index 000000000000..a6e6cd4528dc
> --- /dev/null
> +++ b/drivers/thermal/tegra/Kconfig
> @@ -0,0 +1,9 @@
> +config TEGRA_SOCTHERM
> + tristate "Tegra SOCTHERM thermal management"
> + depends on ARCH_TEGRA
> + help
> + Enable this option for integrated thermal management support on NVIDIA
> + Tegra124 systems-on-chip. The driver supports four thermal zones
> + (CPU, GPU, MEM, PLLX). Cooling devices can be bound to the thermal
> + zones to manage temperatures. This option is also required for the
> + emergency thermal reset (thermtrip) feature to function.
> diff --git a/drivers/thermal/tegra/Makefile b/drivers/thermal/tegra/Makefile
> new file mode 100644
> index 000000000000..8c51076e4b1e
> --- /dev/null
> +++ b/drivers/thermal/tegra/Makefile
> @@ -0,0 +1,6 @@
> +#
> +# Tegra thermal specific Makefile
> +#
> +
> +# Tegra soc thermal drivers
> +obj-$(CONFIG_TEGRA_SOCTHERM) += tegra_soctherm.o
I personally don't think these comments are helpful. They're really
redundant given that they're in a Makefile within a subdirectory of
drivers/thermal.
Thierry
[toc] | [prev] | [next] | [standalone]
| From | Wei Ni <wni@nvidia.com> |
|---|---|
| Date | 2016-01-14 06:40 +0100 |
| Subject | Re: [PATCH V1 01/10] thermal: tegra: move tegra thermal files into tegra directory |
| Message-ID | <qQINA-2u2-3@gated-at.bofh.it> |
| In reply to | #1308452 |
Thierry, thanks for you comments, will fix them.
On 2016年01月13日 22:24, Thierry Reding wrote:
> * PGP Signed by an unknown key
>
> On Wed, Jan 13, 2016 at 03:58:40PM +0800, Wei Ni wrote:
>> Move tegra soctherm driver to tegra directory, it's easy to maintain
>> and add more new function support for tegra platforms.
>
> Please use the proper spelling "Tegra", except where you refer to
> directory or file names.
>
>> This will also help to split soctherm driver into common parts and
>> chip specific data related parts.
>>
>> Signed-off-by: Wei Ni <wni@nvidia.com>
>> ---
>> drivers/thermal/Kconfig | 15 +++++----------
>> drivers/thermal/Makefile | 2 +-
>> drivers/thermal/tegra/Kconfig | 9 +++++++++
>> drivers/thermal/tegra/Makefile | 6 ++++++
>> drivers/thermal/{ => tegra}/tegra_soctherm.c | 0
>> 5 files changed, 21 insertions(+), 11 deletions(-)
>> create mode 100644 drivers/thermal/tegra/Kconfig
>> create mode 100644 drivers/thermal/tegra/Makefile
>> rename drivers/thermal/{ => tegra}/tegra_soctherm.c (100%)
>>
>> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
>> index c463c89b90ef..7984cba6b340 100644
>> --- a/drivers/thermal/Kconfig
>> +++ b/drivers/thermal/Kconfig
>> @@ -254,16 +254,6 @@ config ARMADA_THERMAL
>> Enable this option if you want to have support for thermal management
>> controller present in Armada 370 and Armada XP SoC.
>>
>> -config TEGRA_SOCTHERM
>> - tristate "Tegra SOCTHERM thermal management"
>> - depends on ARCH_TEGRA
>> - help
>> - Enable this option for integrated thermal management support on NVIDIA
>> - Tegra124 systems-on-chip. The driver supports four thermal zones
>> - (CPU, GPU, MEM, PLLX). Cooling devices can be bound to the thermal
>> - zones to manage temperatures. This option is also required for the
>> - emergency thermal reset (thermtrip) feature to function.
>> -
>> config DB8500_CPUFREQ_COOLING
>> tristate "DB8500 cpufreq cooling"
>> depends on ARCH_U8500
>> @@ -380,6 +370,11 @@ depends on ARCH_STI && OF
>> source "drivers/thermal/st/Kconfig"
>> endmenu
>>
>> +menu "Tegra thermal drivers"
>> +depends on ARCH_TEGRA
>> +source "drivers/thermal/tegra/Kconfig"
>> +endmenu
>> +
>
> I think it'd be more idiomatic to only source the file here and move the
> menu declaration and the dependencies into that file. Perhaps also make
> the menu description "NVIDIA Tegra thermal drivers", I think we've been
> using that consistently elsewhere.
>> diff --git a/drivers/thermal/tegra/Kconfig b/drivers/thermal/tegra/Kconfig
>> new file mode 100644
>> index 000000000000..a6e6cd4528dc
>> --- /dev/null
>> +++ b/drivers/thermal/tegra/Kconfig
>> @@ -0,0 +1,9 @@
>> +config TEGRA_SOCTHERM
>> + tristate "Tegra SOCTHERM thermal management"
>> + depends on ARCH_TEGRA
>> + help
>> + Enable this option for integrated thermal management support on NVIDIA
>> + Tegra124 systems-on-chip. The driver supports four thermal zones
>> + (CPU, GPU, MEM, PLLX). Cooling devices can be bound to the thermal
>> + zones to manage temperatures. This option is also required for the
>> + emergency thermal reset (thermtrip) feature to function.
>> diff --git a/drivers/thermal/tegra/Makefile b/drivers/thermal/tegra/Makefile
>> new file mode 100644
>> index 000000000000..8c51076e4b1e
>> --- /dev/null
>> +++ b/drivers/thermal/tegra/Makefile
>> @@ -0,0 +1,6 @@
>> +#
>> +# Tegra thermal specific Makefile
>> +#
>> +
>> +# Tegra soc thermal drivers
>> +obj-$(CONFIG_TEGRA_SOCTHERM) += tegra_soctherm.o
>
> I personally don't think these comments are helpful. They're really
> redundant given that they're in a Makefile within a subdirectory of
> drivers/thermal.
>
> Thierry
>
> * Unknown Key
> * 0x7F3EB3A1
>
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web