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

2018-12-05 Thread Zhang Rui
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 
> Acked-by: Daniel Lezcano 
> ---
>  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
> - 

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

2018-12-05 Thread Zhang Rui
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 
> Acked-by: Daniel Lezcano 
> ---
>  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
> - 

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

2018-11-27 Thread Amit Kucheria
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 
Acked-by: Daniel Lezcano 
---
 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 

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

2018-11-27 Thread Amit Kucheria
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 
Acked-by: Daniel Lezcano 
---
 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