Received-SPF: pass (google.com: domain of linux-kernel+bounces-154948-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Tue, 23 Apr 2024 19:07:09 +0800
From: Huang Rui <ray.huang@amd.com>
To: "Yuan, Perry" <Perry.Yuan@amd.com>
Cc: "rafael.j.wysocki@intel.com" <rafael.j.wysocki@intel.com>,
	"Limonciello, Mario" <Mario.Limonciello@amd.com>,
	"viresh.kumar@linaro.org" <viresh.kumar@linaro.org>,
	"Shenoy, Gautham Ranjal" <gautham.shenoy@amd.com>,
	"Petkov, Borislav" <Borislav.Petkov@amd.com>,
	"Deucher, Alexander" <Alexander.Deucher@amd.com>,
	"Huang, Shimmer" <Shimmer.Huang@amd.com>,
	"oleksandr@natalenko.name" <oleksandr@natalenko.name>,
	"Du, Xiaojian" <Xiaojian.Du@amd.com>,
	"Meng, Li (Jassmine)" <Li.Meng@amd.com>,
	"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v7 6/6] cpufreq: amd-pstate-ut: support new cpb boost
 control interface
Message-ID: <ZieWXSG4MY5QJSl7@amd.com>
References: <cover.1713861200.git.perry.yuan@amd.com>
 <fff9a75f745ef10a4e7e9b0a9b383655ebb0cfd1.1713861200.git.perry.yuan@amd.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <fff9a75f745ef10a4e7e9b0a9b383655ebb0cfd1.1713861200.git.perry.yuan@amd.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?6tzPRY8h+Wn2+aED+bDjm2z5EaVTSWDb/RnSjuXN2R+/to0WcInuy2mPFwBc?=
 =?us-ascii?Q?GubLQIOuG++YwA2tApWgJRyDJVAp0bDZudcL72MooZLuZVrzkWyb3/gPWCVA?=
 =?us-ascii?Q?gKtQJTzunyMT9pB30GrvBAnU7JP04GBv8ndKn2WPOQemU0nmj5XuJEWuT3w6?=
 =?us-ascii?Q?9PZ20uej4QP+XRpGX8QZJ+7xL3ym0remhQ/uMW/r2dRNoB7oDew2YTmDit1O?=
 =?us-ascii?Q?RTUXo26JvKdx08/cBCOOREjZ35hDOIS8Q9DL1qzfhe5x2f0ecnSaqamYICbs?=
 =?us-ascii?Q?JgciGc8RDS9tSWYRTd96O8mb5TjsKBZiYSAKu8Bw4s+EalZEfctLEE6IH+82?=
 =?us-ascii?Q?FlatfI+c6Vopuxed4fjy5YU5pj98ie8MN4ab4qZjDS/8sSaETfaqIlUEg7ko?=
 =?us-ascii?Q?DkHkvASm6vbnv/mzHwTBtig6rGSx1Y9EUcXxsooatj+wNb2+6KRAEEO/Zle9?=
 =?us-ascii?Q?06/IOCaZ4U/dEEoRmle/k/nV6s1+SN2npmemFNjXk9g2pbc7+WXVzxh6ViMP?=
 =?us-ascii?Q?T+9el2hr28S6mymMYZhpqsjjuodoUrItSLO8uk7PAP25ylzPRDaQMkKNXqfr?=
 =?us-ascii?Q?JwDDp+fB2l1F/A+6AumXThmY1LdMbDL0Em2cqiUMmSI8RaHG98aBVaOmWzXS?=
 =?us-ascii?Q?I3BoFM9j29s8hvs+G7TkxveO/8YtWZaZT5yqs+IzlqqdbppaN5hfxQzy8c2f?=
 =?us-ascii?Q?K7v9zlrN9bEeXywBygpwl821ogKLhcikdofWbzyJvDiPo0L7NCTm8Ar3pjjo?=
 =?us-ascii?Q?JvfW6VSl2H1NyX/ICV8vZI73+TWE6gMCfrhiQfT+RztFiIoCrXaPu4fTGC9W?=
 =?us-ascii?Q?MkvbsGllO00uykSH82nn2ZHX0X/hoheOw2Nh3uMXLiIvH5J5PQlHGFwWOn32?=
 =?us-ascii?Q?xA4J+pYeC29q/ZY29Q+aiYNym6BBTpLnCVqDcMcFLFvIq3f/ruNl5SNAK4fp?=
 =?us-ascii?Q?SpXLVZbPssSpOO/yA+k3GxWwhs08iwGB9EiltEaHZASTdrhn3CQPuLM6+k+g?=
 =?us-ascii?Q?YPtZhFeFsXBCNHw+AhJvZWtZ5k5BXdfi0XN7OoNefCFG0WeIqUMTsFXUGj2F?=
 =?us-ascii?Q?uMEe0W2LOOY4t/JRWAVQLpLdCpTTMaBEZgAlgEsTnQy0cKz/dfyyk1zHOzap?=
 =?us-ascii?Q?0HoI4lMQZLiBR9MQ4MI7LAdLrM7gR15P0rnQA9BsOaRocfeVvDBpV3W8ddNQ?=
 =?us-ascii?Q?4D4eHBMcbQ4UwcTokuxYwBPsxS7j8oDCUj5wNtN5o1QEc51HME+F4obqcJxL?=
 =?us-ascii?Q?9B6h1FCA/qOxF0J5hy7h0TAR2HOKWC8OtaRT/OihAFT6eP/idfCMhzSw33aQ?=
 =?us-ascii?Q?+0DTnR2MxtCTq3Q20ILfOvk4/6LNTMuDvNPdjsm5g1jF1HQWzi8A6tGvUxQS?=
 =?us-ascii?Q?k76Aw/Se+o0i/2TdV8ujBfy1WGJLIZDn2OuAbuw+Up4u0LrRFlSYYlf4qaV9?=
 =?us-ascii?Q?uohVTzO53yOyglayX8M7DQA279uvJfeaUFkIWkyVoDYSUfK2E6u6HX0ZTMjB?=
 =?us-ascii?Q?qvTNgQoXiPf4emdRk4w0PDotyU00xpSFrJXq9LSsw9ipA2z4F6ErAnXLTVcp?=
 =?us-ascii?Q?cyN4vBZjLQpjpdwbmdmz3+ScpfZ/I/yeQMiVF3id?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0699af97-9bf7-4883-5692-08dc63859165
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR12MB8690.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Apr 2024 11:07:30.2409
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: JqvZRJ6QBEUkZkvigsrMBGEyUUiiU0b/JcmKVhrOVhRMiJolg2ygckaAq0bfEGyi/hANxSAqtGAK1zGfRhC5pg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR12MB8129

On Tue, Apr 23, 2024 at 04:40:59PM +0800, Yuan, Perry wrote:
> As the new CPB boost control is enabled, pstate unit test needs to remove
> legacy `boost_supported` check and start to use new CPB boost control
> interface `amd_pstate_global_params.cpb_boost`.
> 
> Signed-off-by: Perry Yuan <perry.yuan@amd.com>

Acked-by: Huang Rui <ray.huang@amd.com>

> ---
>  drivers/cpufreq/amd-pstate-ut.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/cpufreq/amd-pstate-ut.c b/drivers/cpufreq/amd-pstate-ut.c
> index f04ae67dda37..b3601b0e6dd3 100644
> --- a/drivers/cpufreq/amd-pstate-ut.c
> +++ b/drivers/cpufreq/amd-pstate-ut.c
> @@ -226,7 +226,7 @@ static void amd_pstate_ut_check_freq(u32 index)
>  			goto skip_test;
>  		}
>  
> -		if (cpudata->boost_supported) {
> +		if (amd_pstate_global_params.cpb_boost) {
>  			if ((policy->max == cpudata->max_freq) ||
>  					(policy->max == cpudata->nominal_freq))
>  				amd_pstate_ut_cases[index].result = AMD_PSTATE_UT_RESULT_PASS;
> -- 
> 2.34.1
>