Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: s+hi2SqqKEQU4IkuYooKx2VJGN/dum//EDdeBLAqPsdBC3bYgkyT8EquYG/yguTPjmI1CP4sn+
 veFZod+TLVbA==
IronPort-SDR: 1FNf/2Jt9eaIreCkupbrPpcxokjRoPlJPhb1fitGpc8/1ucG/gImZjH+wr1Y18zoyTbG3BelDh
 6QqovO61rssg==
Message-ID: <3e4ca631b8fb85de570bf490e71dd71219c5a2c2.camel@linux.intel.com>
Subject: Re: [PATCH 2/2] ACPI: DPTF: Add ACPI_DPTF Kconfig menu
From:   Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
To:     "Rafael J. Wysocki" <rjw@rjwysocki.net>,
        Linux ACPI <linux-acpi@vger.kernel.org>
Cc:     LKML <linux-kernel@vger.kernel.org>, Borislav Petkov <bp@suse.de>
Date:   Thu, 15 Oct 2020 12:46:10 -0700
In-Reply-To: <35637045.pmxlVluP8t@kreacher>
References: <2206290.MayQypTng0@kreacher> <35637045.pmxlVluP8t@kreacher>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.34.4 (3.34.4-1.fc31) 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Thu, 2020-10-15 at 18:59 +0200, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> Add a Kconfig menu for Intel DPTF (Dynamic Platform and Thermal
> Framework), put both the existing participant drivers in it and set
> them to be built as modules by default.
> 
> While at it, do a few assorted cleanups for a good measure.
> 
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>

> ---
>  drivers/acpi/dptf/Kconfig |   29 ++++++++++++++++++++++++-----
>  1 file changed, 24 insertions(+), 5 deletions(-)
> 
> Index: linux-pm/drivers/acpi/dptf/Kconfig
> ===================================================================
> --- linux-pm.orig/drivers/acpi/dptf/Kconfig
> +++ linux-pm/drivers/acpi/dptf/Kconfig
> @@ -1,8 +1,25 @@
>  # SPDX-License-Identifier: GPL-2.0
> -config DPTF_POWER
> -	tristate "DPTF Platform Power Participant"
> +
> +menuconfig ACPI_DPTF
> +	bool "Intel DPTF (Dynamic Platform and Thermal Framework)
> Support"
>  	depends on X86
>  	help
> +	  Intel Dynamic Platform and Thermal Framework (DPTF) is a
> platform
> +	  level hardware/software solution for power and thermal
> management.
> +
> +	  As a container for multiple power/thermal technologies, DPTF
> provides
> +	  a coordinated approach for different policies to effect the
> hardware
> +	  state of a system.
> +
> +	  For more information see:
> +	  <
> https://01.org/intel%C2%AE-dynamic-platform-and-thermal-framework-dptf-chromium-os/overview
> >
> +
> +if ACPI_DPTF
> +
> +config DPTF_POWER
> +	tristate "Platform Power DPTF Participant"
> +	default m
> +	help
>  	  This driver adds support for Dynamic Platform and Thermal
> Framework
>  	  (DPTF) Platform Power Participant device (INT3407) support.
>  	  This participant is responsible for exposing platform
> telemetry:
> @@ -16,15 +33,17 @@ config DPTF_POWER
>  	  the module will be called dptf_power.
>  
>  config DPTF_PCH_FIVR
> -	tristate "DPTF PCH FIVR Participant"
> -	depends on X86
> +	tristate "PCH FIVR DPTF Participant"
> +	default m
>  	help
>  	  This driver adds support for Dynamic Platform and Thermal
> Framework
>  	  (DPTF) PCH FIVR Participant device support. This driver
> allows to
> -	  switch PCH FIVR (Fully Integrated Voltage Regulator)
> frequency.
> +	  switch the PCH FIVR (Fully Integrated Voltage Regulator)
> frequency.
>  	  This participant is responsible for exposing:
>  		freq_mhz_low_clock
>  		freq_mhz_high_clock
>  
>  	  To compile this driver as a module, choose M here:
>  	  the module will be called dptf_pch_fivr.
> +
> +endif
> 
> 
>