Re: [PATCH v3 1/2] drivers: thermal: Move various drivers for intel platforms into a subdir

From: Zhang Rui
Date: Wed Dec 05 2018 - 10:16:17 EST


On ä, 2018-11-28 at 00:28 +0530, Amit Kucheria wrote:
> This cleans up the directory a bit, now that we have several other
> platforms using platform-specific sub-directories. Compile-tested
> with
> ARCH=x86 defconfig and the drivers explicitly enabled with
> menuconfig.
>
> Signed-off-by: Amit Kucheria <amit.kucheria@xxxxxxxxxx>
> Acked-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
> ---
> Âdrivers/thermal/KconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 83 ++---------------
> --
> Âdrivers/thermal/MakefileÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 11 +--
> Âdrivers/thermal/intel/KconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 77 +++++++++++++++++
> Âdrivers/thermal/intel/MakefileÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 12 +++
> Â.../{ => intel}/int340x_thermal/KconfigÂÂÂÂÂÂÂ|ÂÂ0
> Â.../{ => intel}/int340x_thermal/MakefileÂÂÂÂÂÂ|ÂÂ0
> Â.../int340x_thermal/acpi_thermal_rel.cÂÂÂÂÂÂÂÂ|ÂÂ0
> Â.../int340x_thermal/acpi_thermal_rel.hÂÂÂÂÂÂÂÂ|ÂÂ0
> Â.../int340x_thermal/int3400_thermal.cÂÂÂÂÂÂÂÂÂ|ÂÂ0
> Â.../int340x_thermal/int3402_thermal.cÂÂÂÂÂÂÂÂÂ|ÂÂ0
> Â.../int340x_thermal/int3403_thermal.cÂÂÂÂÂÂÂÂÂ|ÂÂ0
> Â.../int340x_thermal/int3406_thermal.cÂÂÂÂÂÂÂÂÂ|ÂÂ0
> Â.../int340x_thermal/int340x_thermal_zone.cÂÂÂÂ|ÂÂ0
> Â.../int340x_thermal/int340x_thermal_zone.hÂÂÂÂ|ÂÂ0
> Â.../processor_thermal_device.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂ0
> Â.../{ => intel}/intel_bxt_pmic_thermal.cÂÂÂÂÂÂ|ÂÂ0
> Â.../thermal/{ => intel}/intel_pch_thermal.cÂÂÂ|ÂÂ0
> Â.../thermal/{ => intel}/intel_powerclamp.cÂÂÂÂ|ÂÂ0
> Â.../{ => intel}/intel_quark_dts_thermal.cÂÂÂÂÂ|ÂÂ0
> Â.../thermal/{ => intel}/intel_soc_dts_iosf.cÂÂ|ÂÂ0
> Â.../thermal/{ => intel}/intel_soc_dts_iosf.hÂÂ|ÂÂ0
> Â.../{ => intel}/intel_soc_dts_thermal.cÂÂÂÂÂÂÂ|ÂÂ0
> Â.../{ => intel}/x86_pkg_temp_thermal.cÂÂÂÂÂÂÂÂ|ÂÂ0
> Â23 files changed, 96 insertions(+), 87 deletions(-)
> Âcreate mode 100644 drivers/thermal/intel/Kconfig
> Âcreate mode 100644 drivers/thermal/intel/Makefile
> Ârename drivers/thermal/{ => intel}/int340x_thermal/Kconfig (100%)
> Ârename drivers/thermal/{ => intel}/int340x_thermal/Makefile (100%)
> Ârename drivers/thermal/{ =>
> intel}/int340x_thermal/acpi_thermal_rel.c (100%)
> Ârename drivers/thermal/{ =>
> intel}/int340x_thermal/acpi_thermal_rel.h (100%)
> Ârename drivers/thermal/{ => intel}/int340x_thermal/int3400_thermal.c
> (100%)
> Ârename drivers/thermal/{ => intel}/int340x_thermal/int3402_thermal.c
> (100%)
> Ârename drivers/thermal/{ => intel}/int340x_thermal/int3403_thermal.c
> (100%)
> Ârename drivers/thermal/{ => intel}/int340x_thermal/int3406_thermal.c
> (100%)
> Ârename drivers/thermal/{ =>
> intel}/int340x_thermal/int340x_thermal_zone.c (100%)
> Ârename drivers/thermal/{ =>
> intel}/int340x_thermal/int340x_thermal_zone.h (100%)
> Ârename drivers/thermal/{ =>
> intel}/int340x_thermal/processor_thermal_device.c (100%)
> Ârename drivers/thermal/{ => intel}/intel_bxt_pmic_thermal.c (100%)
> Ârename drivers/thermal/{ => intel}/intel_pch_thermal.c (100%)
> Ârename drivers/thermal/{ => intel}/intel_powerclamp.c (100%)
> Ârename drivers/thermal/{ => intel}/intel_quark_dts_thermal.c (100%)
> Ârename drivers/thermal/{ => intel}/intel_soc_dts_iosf.c (100%)
> Ârename drivers/thermal/{ => intel}/intel_soc_dts_iosf.h (100%)
> Ârename drivers/thermal/{ => intel}/intel_soc_dts_thermal.c (100%)
> Ârename drivers/thermal/{ => intel}/x86_pkg_temp_thermal.c (100%)
>
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index 5422523c03f8..772ab9dadda7 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -326,84 +326,6 @@ config DA9062_THERMAL
> Â ÂÂzone.
> Â ÂÂCompatible with the DA9062 and DA9061 PMICs.
> Â
> -config INTEL_POWERCLAMP
> - tristate "Intel PowerClamp idle injection driver"
> - depends on THERMAL
> - depends on X86
> - depends on CPU_SUP_INTEL
> - help
> - ÂÂEnable this to enable Intel PowerClamp idle injection
> driver. This
> - ÂÂenforce idle time which results in more package C-state
> residency. The
> - ÂÂuser interface is exposed via generic thermal framework.
> -
> -config X86_PKG_TEMP_THERMAL
> - tristate "X86 package temperature thermal driver"
> - depends on X86_THERMAL_VECTOR
> - select THERMAL_GOV_USER_SPACE
> - select THERMAL_WRITABLE_TRIPS
> - default m
> - help
> - ÂÂEnable this to register CPU digital sensor for package
> temperature as
> - ÂÂthermal zone. Each package will have its own thermal zone.
> There are
> - ÂÂtwo trip points which can be set by user to get
> notifications via thermal
> - ÂÂnotification methods.
> -
> -config INTEL_SOC_DTS_IOSF_CORE
> - tristate
> - depends on X86 && PCI
> - select IOSF_MBI
> - help
> - ÂÂThis is becoming a common feature for Intel SoCs to expose
> the additional
> - ÂÂdigital temperature sensors (DTSs) using side band
> interface (IOSF). This
> - ÂÂimplements the common set of helper functions to register,
> get temperature
> - ÂÂand get/set thresholds on DTSs.
> -
> -config INTEL_SOC_DTS_THERMAL
> - tristate "Intel SoCs DTS thermal driver"
> - depends on X86 && PCI && ACPI
> - select INTEL_SOC_DTS_IOSF_CORE
> - select THERMAL_WRITABLE_TRIPS
> - help
> - ÂÂEnable this to register Intel SoCs (e.g. Bay Trail)
> platform digital
> - ÂÂtemperature sensor (DTS). These SoCs have two additional
> DTSs in
> - ÂÂaddition to DTSs on CPU cores. Each DTS will be registered
> as a
> - ÂÂthermal zone. There are two trip points. One of the trip
> point can
> - ÂÂbe set by user mode programs to get notifications via
> Linux thermal
> - ÂÂnotification methods.The other trip is a critical trip
> point, which
> - ÂÂwas set by the driver based on the TJ MAX temperature.
> -
> -config INTEL_QUARK_DTS_THERMAL
> - tristate "Intel Quark DTS thermal driver"
> - depends on X86_INTEL_QUARK
> - 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.
> - ÂÂThe DTS will be registered as a thermal zone. There are
> two trip points:
> - ÂÂhot & critical. The critical trip point default value is
> set by
> - ÂÂunderlying BIOS/Firmware.
> -
> -menu "ACPI INT340X thermal drivers"
> -source drivers/thermal/int340x_thermal/Kconfig
> -endmenu
> -
> -config INTEL_BXT_PMIC_THERMAL
> - tristate "Intel Broxton PMIC thermal driver"
> - depends on X86 && INTEL_SOC_PMIC_BXTWC && REGMAP
> - help
> - ÂÂSelect this driver for Intel Broxton PMIC with ADC
> channels monitoring
> - ÂÂsystem temperature measurements and alerts.
> - ÂÂThis driver is used for monitoring the ADC channels of
> PMIC and handles
> - ÂÂthe alert trip point interrupts and notifies the thermal
> framework with
> - ÂÂthe trip point and temperature details of the zone.
> -
> -config INTEL_PCH_THERMAL
> - tristate "Intel PCH Thermal Reporting Driver"
> - depends on X86 && PCI
> - help
> - ÂÂEnable this to support thermal reporting on certain intel
> PCHs.
> - ÂÂThermal reporting device will provide temperature reading,
> - ÂÂprogrammable trip points and other information.
> -
> Âconfig MTK_THERMAL
> Â tristate "Temperature sensor driver for mediatek SoCs"
> Â depends on ARCH_MEDIATEK || COMPILE_TEST
> @@ -415,6 +337,11 @@ config MTK_THERMAL
> Â ÂÂEnable this option if you want to have support for thermal
> management
> Â ÂÂcontroller present in Mediatek SoCs
> Â
> +menu "Intel thermal drivers"
> +depends on X86 || X86_INTEL_QUARK || COMPILE_TEST
> +source "drivers/thermal/intel/Kconfig"
> +endmenu
> +
> Âmenu "Broadcom thermal drivers"
> Âdepends on ARCH_BCM || ARCH_BRCMSTB || ARCH_BCM2835 || COMPILE_TEST
> Âsource "drivers/thermal/broadcom/Kconfig"
> diff --git a/drivers/thermal/Makefile b/drivers/thermal/Makefile
> index 82bb50dc6423..031f61e7dfe8 100644
> --- a/drivers/thermal/Makefile
> +++ b/drivers/thermal/Makefile
> @@ -44,16 +44,9 @@ obj-$(CONFIG_IMX_THERMAL) += imx_thermal.o
> Âobj-$(CONFIG_MAX77620_THERMAL) += max77620_thermal.o
> Âobj-$(CONFIG_QORIQ_THERMAL) += qoriq_thermal.o
> Âobj-$(CONFIG_DA9062_THERMAL) += da9062-thermal.o
> -obj-$(CONFIG_INTEL_POWERCLAMP) += intel_powerclamp.o
> -obj-$(CONFIG_X86_PKG_TEMP_THERMAL) += x86_pkg_temp_thermal.o
> -obj-$(CONFIG_INTEL_SOC_DTS_IOSF_CORE) += intel_soc_dts_iosf.o
> -obj-$(CONFIG_INTEL_SOC_DTS_THERMAL) +=
> intel_soc_dts_thermal.o
> -obj-$(CONFIG_INTEL_QUARK_DTS_THERMAL) +=
> intel_quark_dts_thermal.o
> +obj-y += intel/
> Âobj-$(CONFIG_TI_SOC_THERMAL) += ti-soc-thermal/
> -obj-$(CONFIG_INT340X_THERMAL)ÂÂ+= int340x_thermal/
> -obj-$(CONFIG_INTEL_BXT_PMIC_THERMAL) += intel_bxt_pmic_thermal.o
> -obj-$(CONFIG_INTEL_PCH_THERMAL) += intel_pch_thermal.o

> -obj-y += st/
> +obj-$(CONFIG_ST_THERMAL) += st/

it's better to do one thing in one patch. I will apply the patch, but
without this line change.

thanks,
rui

> Âobj-$(CONFIG_QCOM_TSENS) += qcom/
> Âobj-y += tegra/
> Âobj-$(CONFIG_HISI_THERMAL)ÂÂÂÂÂ+= hisi_thermal.o
> diff --git a/drivers/thermal/intel/Kconfig
> b/drivers/thermal/intel/Kconfig
> new file mode 100644
> index 000000000000..9c06d4ad7c97
> --- /dev/null
> +++ b/drivers/thermal/intel/Kconfig
> @@ -0,0 +1,77 @@
> +config INTEL_POWERCLAMP
> + tristate "Intel PowerClamp idle injection driver"
> + depends on THERMAL
> + depends on X86
> + depends on CPU_SUP_INTEL
> + help
> + ÂÂEnable this to enable Intel PowerClamp idle injection
> driver. This
> + ÂÂenforce idle time which results in more package C-state
> residency. The
> + ÂÂuser interface is exposed via generic thermal framework.
> +
> +config X86_PKG_TEMP_THERMAL
> + tristate "X86 package temperature thermal driver"
> + depends on X86_THERMAL_VECTOR
> + select THERMAL_GOV_USER_SPACE
> + select THERMAL_WRITABLE_TRIPS
> + default m
> + help
> + ÂÂEnable this to register CPU digital sensor for package
> temperature as
> + ÂÂthermal zone. Each package will have its own thermal zone.
> There are
> + ÂÂtwo trip points which can be set by user to get
> notifications via thermal
> + ÂÂnotification methods.
> +
> +config INTEL_SOC_DTS_IOSF_CORE
> + tristate
> + depends on X86 && PCI
> + select IOSF_MBI
> + help
> + ÂÂThis is becoming a common feature for Intel SoCs to expose
> the additional
> + ÂÂdigital temperature sensors (DTSs) using side band
> interface (IOSF). This
> + ÂÂimplements the common set of helper functions to register,
> get temperature
> + ÂÂand get/set thresholds on DTSs.
> +
> +config INTEL_SOC_DTS_THERMAL
> + tristate "Intel SoCs DTS thermal driver"
> + depends on X86 && PCI && ACPI
> + select INTEL_SOC_DTS_IOSF_CORE
> + select THERMAL_WRITABLE_TRIPS
> + help
> + ÂÂEnable this to register Intel SoCs (e.g. Bay Trail)
> platform digital
> + ÂÂtemperature sensor (DTS). These SoCs have two additional
> DTSs in
> + ÂÂaddition to DTSs on CPU cores. Each DTS will be registered
> as a
> + ÂÂthermal zone. There are two trip points. One of the trip
> point can
> + ÂÂbe set by user mode programs to get notifications via
> Linux thermal
> + ÂÂnotification methods.The other trip is a critical trip
> point, which
> + ÂÂwas set by the driver based on the TJ MAX temperature.
> +
> +config INTEL_QUARK_DTS_THERMAL
> + tristate "Intel Quark DTS thermal driver"
> + depends on X86_INTEL_QUARK
> + 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.
> + ÂÂThe DTS will be registered as a thermal zone. There are
> two trip points:
> + ÂÂhot & critical. The critical trip point default value is
> set by
> + ÂÂunderlying BIOS/Firmware.
> +
> +menu "ACPI INT340X thermal drivers"
> +source drivers/thermal/intel/int340x_thermal/Kconfig
> +endmenu
> +
> +config INTEL_BXT_PMIC_THERMAL
> + tristate "Intel Broxton PMIC thermal driver"
> + depends on X86 && INTEL_SOC_PMIC_BXTWC && REGMAP
> + help
> + ÂÂSelect this driver for Intel Broxton PMIC with ADC
> channels monitoring
> + ÂÂsystem temperature measurements and alerts.
> + ÂÂThis driver is used for monitoring the ADC channels of
> PMIC and handles
> + ÂÂthe alert trip point interrupts and notifies the thermal
> framework with
> + ÂÂthe trip point and temperature details of the zone.
> +
> +config INTEL_PCH_THERMAL
> + tristate "Intel PCH Thermal Reporting Driver"
> + depends on X86 && PCI
> + help
> + ÂÂEnable this to support thermal reporting on certain intel
> PCHs.
> + ÂÂThermal reporting device will provide temperature reading,
> + ÂÂprogrammable trip points and other information.
> diff --git a/drivers/thermal/intel/Makefile
> b/drivers/thermal/intel/Makefile
> new file mode 100644
> index 000000000000..0d9736ced5d4
> --- /dev/null
> +++ b/drivers/thermal/intel/Makefile
> @@ -0,0 +1,12 @@
> +# SPDX-License-Identifier: GPL-2.0
> +#
> +# Makefile for various Intel thermal drivers.
> +
> +obj-$(CONFIG_INTEL_POWERCLAMP) += intel_powerclamp.o
> +obj-$(CONFIG_X86_PKG_TEMP_THERMAL) += x86_pkg_temp_thermal.o
> +obj-$(CONFIG_INTEL_SOC_DTS_IOSF_CORE) += intel_soc_dts_iosf.o
> +obj-$(CONFIG_INTEL_SOC_DTS_THERMAL) +=
> intel_soc_dts_thermal.o
> +obj-$(CONFIG_INTEL_QUARK_DTS_THERMAL) +=
> intel_quark_dts_thermal.o
> +obj-$(CONFIG_INT340X_THERMAL)ÂÂ+= int340x_thermal/
> +obj-$(CONFIG_INTEL_BXT_PMIC_THERMAL) += intel_bxt_pmic_thermal.o
> +obj-$(CONFIG_INTEL_PCH_THERMAL) += intel_pch_thermal.o
> diff --git a/drivers/thermal/int340x_thermal/Kconfig
> b/drivers/thermal/intel/int340x_thermal/Kconfig
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/Kconfig
> rename to drivers/thermal/intel/int340x_thermal/Kconfig
> diff --git a/drivers/thermal/int340x_thermal/Makefile
> b/drivers/thermal/intel/int340x_thermal/Makefile
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/Makefile
> rename to drivers/thermal/intel/int340x_thermal/Makefile
> diff --git a/drivers/thermal/int340x_thermal/acpi_thermal_rel.c
> b/drivers/thermal/intel/int340x_thermal/acpi_thermal_rel.c
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/acpi_thermal_rel.c
> rename to drivers/thermal/intel/int340x_thermal/acpi_thermal_rel.c
> diff --git a/drivers/thermal/int340x_thermal/acpi_thermal_rel.h
> b/drivers/thermal/intel/int340x_thermal/acpi_thermal_rel.h
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/acpi_thermal_rel.h
> rename to drivers/thermal/intel/int340x_thermal/acpi_thermal_rel.h
> diff --git a/drivers/thermal/int340x_thermal/int3400_thermal.c
> b/drivers/thermal/intel/int340x_thermal/int3400_thermal.c
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/int3400_thermal.c
> rename to drivers/thermal/intel/int340x_thermal/int3400_thermal.c
> diff --git a/drivers/thermal/int340x_thermal/int3402_thermal.c
> b/drivers/thermal/intel/int340x_thermal/int3402_thermal.c
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/int3402_thermal.c
> rename to drivers/thermal/intel/int340x_thermal/int3402_thermal.c
> diff --git a/drivers/thermal/int340x_thermal/int3403_thermal.c
> b/drivers/thermal/intel/int340x_thermal/int3403_thermal.c
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/int3403_thermal.c
> rename to drivers/thermal/intel/int340x_thermal/int3403_thermal.c
> diff --git a/drivers/thermal/int340x_thermal/int3406_thermal.c
> b/drivers/thermal/intel/int340x_thermal/int3406_thermal.c
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/int3406_thermal.c
> rename to drivers/thermal/intel/int340x_thermal/int3406_thermal.c
> diff --git a/drivers/thermal/int340x_thermal/int340x_thermal_zone.c
> b/drivers/thermal/intel/int340x_thermal/int340x_thermal_zone.c
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/int340x_thermal_zone.c
> rename to
> drivers/thermal/intel/int340x_thermal/int340x_thermal_zone.c
> diff --git a/drivers/thermal/int340x_thermal/int340x_thermal_zone.h
> b/drivers/thermal/intel/int340x_thermal/int340x_thermal_zone.h
> similarity index 100%
> rename from drivers/thermal/int340x_thermal/int340x_thermal_zone.h
> rename to
> drivers/thermal/intel/int340x_thermal/int340x_thermal_zone.h
> diff --git
> a/drivers/thermal/int340x_thermal/processor_thermal_device.c
> b/drivers/thermal/intel/int340x_thermal/processor_thermal_device.c
> similarity index 100%
> rename from
> drivers/thermal/int340x_thermal/processor_thermal_device.c
> rename to
> drivers/thermal/intel/int340x_thermal/processor_thermal_device.c
> diff --git a/drivers/thermal/intel_bxt_pmic_thermal.c
> b/drivers/thermal/intel/intel_bxt_pmic_thermal.c
> similarity index 100%
> rename from drivers/thermal/intel_bxt_pmic_thermal.c
> rename to drivers/thermal/intel/intel_bxt_pmic_thermal.c
> diff --git a/drivers/thermal/intel_pch_thermal.c
> b/drivers/thermal/intel/intel_pch_thermal.c
> similarity index 100%
> rename from drivers/thermal/intel_pch_thermal.c
> rename to drivers/thermal/intel/intel_pch_thermal.c
> diff --git a/drivers/thermal/intel_powerclamp.c
> b/drivers/thermal/intel/intel_powerclamp.c
> similarity index 100%
> rename from drivers/thermal/intel_powerclamp.c
> rename to drivers/thermal/intel/intel_powerclamp.c
> diff --git a/drivers/thermal/intel_quark_dts_thermal.c
> b/drivers/thermal/intel/intel_quark_dts_thermal.c
> similarity index 100%
> rename from drivers/thermal/intel_quark_dts_thermal.c
> rename to drivers/thermal/intel/intel_quark_dts_thermal.c
> diff --git a/drivers/thermal/intel_soc_dts_iosf.c
> b/drivers/thermal/intel/intel_soc_dts_iosf.c
> similarity index 100%
> rename from drivers/thermal/intel_soc_dts_iosf.c
> rename to drivers/thermal/intel/intel_soc_dts_iosf.c
> diff --git a/drivers/thermal/intel_soc_dts_iosf.h
> b/drivers/thermal/intel/intel_soc_dts_iosf.h
> similarity index 100%
> rename from drivers/thermal/intel_soc_dts_iosf.h
> rename to drivers/thermal/intel/intel_soc_dts_iosf.h
> diff --git a/drivers/thermal/intel_soc_dts_thermal.c
> b/drivers/thermal/intel/intel_soc_dts_thermal.c
> similarity index 100%
> rename from drivers/thermal/intel_soc_dts_thermal.c
> rename to drivers/thermal/intel/intel_soc_dts_thermal.c
> diff --git a/drivers/thermal/x86_pkg_temp_thermal.c
> b/drivers/thermal/intel/x86_pkg_temp_thermal.c
> similarity index 100%
> rename from drivers/thermal/x86_pkg_temp_thermal.c
> rename to drivers/thermal/intel/x86_pkg_temp_thermal.c