All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zhang Rui <rui.zhang@intel.com>
To: Matthew Garrett <matthewgarrett@google.com>,
	linux-kernel@vger.kernel.org
Cc: linux-pm@vger.kernel.org, Matthew Garrett <mjg59@google.com>,
	Nisha Aram <nisha.aram@intel.com>
Subject: Re: [PATCH 1/2] thermal/int340x_thermal: Add additional UUIDs
Date: Mon, 03 Dec 2018 23:08:27 +0800	[thread overview]
Message-ID: <1543849707.2841.9.camel@intel.com> (raw)
In-Reply-To: <20181010083007.239938-2-matthewgarrett@google.com>

Hi, Matthew,

On 三, 2018-10-10 at 01:30 -0700, Matthew Garrett wrote:
> Platforms support more DPTF policies than the driver currently
> exposes.
> Add them. This effectively reverts
> 31908f45a583e8f21db37f402b6e8d5739945afd which removed several UUIDs
> without explaining why.
> 
I'm going to apply this patch series, just with two minor changes,
1. 31908f45a583e8f21db37f402b6e8d5739945afd does not follow the git
commit description style 'commit <12+ chars of sha1> ("<title line>")'
2. the UUIDs were removed previously because these policies were not
used.

thanks,
rui

> Signed-off-by: Matthew Garrett <mjg59@google.com>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Nisha Aram <nisha.aram@intel.com>
> ---
>  drivers/thermal/int340x_thermal/int3400_thermal.c | 14
> ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/drivers/thermal/int340x_thermal/int3400_thermal.c
> b/drivers/thermal/int340x_thermal/int3400_thermal.c
> index e26b01c05e82..51c9097eaf7a 100644
> --- a/drivers/thermal/int340x_thermal/int3400_thermal.c
> +++ b/drivers/thermal/int340x_thermal/int3400_thermal.c
> @@ -22,6 +22,13 @@ enum int3400_thermal_uuid {
>  	INT3400_THERMAL_PASSIVE_1,
>  	INT3400_THERMAL_ACTIVE,
>  	INT3400_THERMAL_CRITICAL,
> +	INT3400_THERMAL_ADAPTIVE_PERFORMANCE,
> +	INT3400_THERMAL_EMERGENCY_CALL_MODE,
> +	INT3400_THERMAL_PASSIVE_2,
> +	INT3400_THERMAL_POWER_BOSS,
> +	INT3400_THERMAL_VIRTUAL_SENSOR,
> +	INT3400_THERMAL_COOLING_MODE,
> +	INT3400_THERMAL_HARDWARE_DUTY_CYCLING,
>  	INT3400_THERMAL_MAXIMUM_UUID,
>  };
>  
> @@ -29,6 +36,13 @@ static char
> *int3400_thermal_uuids[INT3400_THERMAL_MAXIMUM_UUID] = {
>  	"42A441D6-AE6A-462b-A84B-4A8CE79027D3",
>  	"3A95C389-E4B8-4629-A526-C52C88626BAE",
>  	"97C68AE7-15FA-499c-B8C9-5DA81D606E0A",
> +	"63BE270F-1C11-48FD-A6F7-3AF253FF3E2D",
> +	"5349962F-71E6-431D-9AE8-0A635B710AEE",
> +	"9E04115A-AE87-4D1C-9500-0F3E340BFE75",
> +	"F5A35014-C209-46A4-993A-EB56DE7530A1",
> +	"6ED722A7-9240-48A5-B479-31EEF723D7CF",
> +	"16CAF1B7-DD38-40ED-B1C1-1B8A1913D531",
> +	"BE84BABF-C4D4-403D-B495-3128FD44dAC1",
>  };
>  
>  struct int3400_thermal_priv {

  reply	other threads:[~2018-12-03 15:08 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-10  8:30 Expose all known Intel DPTF policies Matthew Garrett
2018-10-10  8:30 ` [PATCH 1/2] thermal/int340x_thermal: Add additional UUIDs Matthew Garrett
2018-12-03 15:08   ` Zhang Rui [this message]
2019-01-23  0:44     ` Joel Stanley
2019-01-23  5:58       ` Zhang Rui
2018-10-10  8:30 ` [PATCH 2/2] thermal/int340x_thermal: fix mode setting Matthew Garrett

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1543849707.2841.9.camel@intel.com \
    --to=rui.zhang@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=matthewgarrett@google.com \
    --cc=mjg59@google.com \
    --cc=nisha.aram@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.