amd-pstate passive mode driver is hyphenated. So make amd-pstate active mode driver consistent with that rename "amd_pstate_epp" to "amd-pstate-epp".
Cc: stable@vger.kernel.org Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors") Reviewed-by: Gautham R. Shenoy gautham.shenoy@amd.com Signed-off-by: Wyes Karny wyes.karny@amd.com --- drivers/cpufreq/amd-pstate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpufreq/amd-pstate.c b/drivers/cpufreq/amd-pstate.c index ddd346a239e0..a5764946434c 100644 --- a/drivers/cpufreq/amd-pstate.c +++ b/drivers/cpufreq/amd-pstate.c @@ -1356,7 +1356,7 @@ static struct cpufreq_driver amd_pstate_epp_driver = { .online = amd_pstate_epp_cpu_online, .suspend = amd_pstate_epp_suspend, .resume = amd_pstate_epp_resume, - .name = "amd_pstate_epp", + .name = "amd-pstate-epp", .attr = amd_pstate_epp_attr, };
On Mon, Jun 12, 2023 at 1:37 PM Wyes Karny wyes.karny@amd.com wrote:
amd-pstate passive mode driver is hyphenated. So make amd-pstate active mode driver consistent with that rename "amd_pstate_epp" to "amd-pstate-epp".
Cc: stable@vger.kernel.org Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors") Reviewed-by: Gautham R. Shenoy gautham.shenoy@amd.com Signed-off-by: Wyes Karny wyes.karny@amd.com
How much does the rest of the series depend on this patch?
I can apply it right away and push it out to a forward-only branch if that helps.
drivers/cpufreq/amd-pstate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpufreq/amd-pstate.c b/drivers/cpufreq/amd-pstate.c index ddd346a239e0..a5764946434c 100644 --- a/drivers/cpufreq/amd-pstate.c +++ b/drivers/cpufreq/amd-pstate.c @@ -1356,7 +1356,7 @@ static struct cpufreq_driver amd_pstate_epp_driver = { .online = amd_pstate_epp_cpu_online, .suspend = amd_pstate_epp_suspend, .resume = amd_pstate_epp_resume,
.name = "amd_pstate_epp",
.name = "amd-pstate-epp", .attr = amd_pstate_epp_attr,};
-- 2.34.1
Hi Rafael,
On 15 Jun 19:31, Rafael J. Wysocki wrote:
On Mon, Jun 12, 2023 at 1:37 PM Wyes Karny wyes.karny@amd.com wrote:
amd-pstate passive mode driver is hyphenated. So make amd-pstate active mode driver consistent with that rename "amd_pstate_epp" to "amd-pstate-epp".
Cc: stable@vger.kernel.org Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors") Reviewed-by: Gautham R. Shenoy gautham.shenoy@amd.com Signed-off-by: Wyes Karny wyes.karny@amd.com
How much does the rest of the series depend on this patch?
The rest of the series is independent of this patch.
I can apply it right away and push it out to a forward-only branch if that helps.
Sure, that helps!
Thanks & Regards, Wyes
drivers/cpufreq/amd-pstate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpufreq/amd-pstate.c b/drivers/cpufreq/amd-pstate.c index ddd346a239e0..a5764946434c 100644 --- a/drivers/cpufreq/amd-pstate.c +++ b/drivers/cpufreq/amd-pstate.c @@ -1356,7 +1356,7 @@ static struct cpufreq_driver amd_pstate_epp_driver = { .online = amd_pstate_epp_cpu_online, .suspend = amd_pstate_epp_suspend, .resume = amd_pstate_epp_resume,
.name = "amd_pstate_epp",
.name = "amd-pstate-epp", .attr = amd_pstate_epp_attr,};
-- 2.34.1
On Thu, Jun 15, 2023 at 7:55 PM Wyes Karny wyes.karny@amd.com wrote:
Hi Rafael,
On 15 Jun 19:31, Rafael J. Wysocki wrote:
On Mon, Jun 12, 2023 at 1:37 PM Wyes Karny wyes.karny@amd.com wrote:
amd-pstate passive mode driver is hyphenated. So make amd-pstate active mode driver consistent with that rename "amd_pstate_epp" to "amd-pstate-epp".
Cc: stable@vger.kernel.org Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors") Reviewed-by: Gautham R. Shenoy gautham.shenoy@amd.com Signed-off-by: Wyes Karny wyes.karny@amd.com
How much does the rest of the series depend on this patch?
The rest of the series is independent of this patch.
So it should have been posted separately as an individual fix.
Please resend the rest of the series without it to avoid confusion and I'll apply it for 6.5 tomorrow.
On 15 Jun 20:30, Rafael J. Wysocki wrote:
On Thu, Jun 15, 2023 at 7:55 PM Wyes Karny wyes.karny@amd.com wrote:
Hi Rafael,
On 15 Jun 19:31, Rafael J. Wysocki wrote:
On Mon, Jun 12, 2023 at 1:37 PM Wyes Karny wyes.karny@amd.com wrote:
amd-pstate passive mode driver is hyphenated. So make amd-pstate active mode driver consistent with that rename "amd_pstate_epp" to "amd-pstate-epp".
Cc: stable@vger.kernel.org Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors") Reviewed-by: Gautham R. Shenoy gautham.shenoy@amd.com Signed-off-by: Wyes Karny wyes.karny@amd.com
How much does the rest of the series depend on this patch?
The rest of the series is independent of this patch.
So it should have been posted separately as an individual fix.
Please resend the rest of the series without it to avoid confusion and I'll apply it for 6.5 tomorrow.
Sure, I'll send rest of the series separately.
Thanks, Wyes
[AMD Official Use Only - General]
Hi Wyse,
-----Original Message----- From: Karny, Wyes Wyes.Karny@amd.com Sent: Friday, June 16, 2023 3:05 AM To: Rafael J. Wysocki rafael@kernel.org Cc: Huang, Ray Ray.Huang@amd.com; viresh.kumar@linaro.org; trenn@suse.com; shuah@kernel.org; Shenoy, Gautham Ranjal gautham.shenoy@amd.com; Limonciello, Mario Mario.Limonciello@amd.com; Yuan, Perry Perry.Yuan@amd.com; linux- pm@vger.kernel.org; linux-kernel@vger.kernel.org; stable@vger.kernel.org Subject: Re: [PATCH 1/6] amd-pstate: Make amd-pstate epp driver name hyphenated
On 15 Jun 20:30, Rafael J. Wysocki wrote:
On Thu, Jun 15, 2023 at 7:55 PM Wyes Karny wyes.karny@amd.com
wrote:
Hi Rafael,
On 15 Jun 19:31, Rafael J. Wysocki wrote:
On Mon, Jun 12, 2023 at 1:37 PM Wyes Karny wyes.karny@amd.com
wrote:
amd-pstate passive mode driver is hyphenated. So make amd-pstate active mode driver consistent with that rename "amd_pstate_epp" to "amd-pstate-epp".
Cc: stable@vger.kernel.org Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors") Reviewed-by: Gautham R. Shenoy gautham.shenoy@amd.com Signed-off-by: Wyes Karny wyes.karny@amd.com
How much does the rest of the series depend on this patch?
The rest of the series is independent of this patch.
So it should have been posted separately as an individual fix.
Please resend the rest of the series without it to avoid confusion and I'll apply it for 6.5 tomorrow.
Sure, I'll send rest of the series separately.
Thanks, Wyes
LGTM, Reviewed-by: Perry Yuan Perry.Yuan@amd.com
On Mon, Jun 12, 2023 at 07:36:10PM +0800, Karny, Wyes wrote:
amd-pstate passive mode driver is hyphenated. So make amd-pstate active mode driver consistent with that rename "amd_pstate_epp" to "amd-pstate-epp".
Cc: stable@vger.kernel.org Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors") Reviewed-by: Gautham R. Shenoy gautham.shenoy@amd.com Signed-off-by: Wyes Karny wyes.karny@amd.com
Acked-by: Huang Rui ray.huang@amd.com
And yes, we should seprate it from cpupower as Rafael mentioned. cpupower tool may go to another repo.
Thanks, Ray
drivers/cpufreq/amd-pstate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpufreq/amd-pstate.c b/drivers/cpufreq/amd-pstate.c index ddd346a239e0..a5764946434c 100644 --- a/drivers/cpufreq/amd-pstate.c +++ b/drivers/cpufreq/amd-pstate.c @@ -1356,7 +1356,7 @@ static struct cpufreq_driver amd_pstate_epp_driver = { .online = amd_pstate_epp_cpu_online, .suspend = amd_pstate_epp_suspend, .resume = amd_pstate_epp_resume,
- .name = "amd_pstate_epp",
- .name = "amd-pstate-epp", .attr = amd_pstate_epp_attr,
}; -- 2.34.1
On Fri, Jun 16, 2023 at 9:08 AM Huang Rui ray.huang@amd.com wrote:
On Mon, Jun 12, 2023 at 07:36:10PM +0800, Karny, Wyes wrote:
amd-pstate passive mode driver is hyphenated. So make amd-pstate active mode driver consistent with that rename "amd_pstate_epp" to "amd-pstate-epp".
Cc: stable@vger.kernel.org Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors") Reviewed-by: Gautham R. Shenoy gautham.shenoy@amd.com Signed-off-by: Wyes Karny wyes.karny@amd.com
Acked-by: Huang Rui ray.huang@amd.com
And yes, we should seprate it from cpupower as Rafael mentioned. cpupower tool may go to another repo.
Not only that.
It is generally better to send individual fixes that don't depend on anything else as separate patches, because this allows them to be picked up and fast-tracked at multiple levels (mainline, stable, distro kernels etc.).
On Fri, Jun 16, 2023 at 08:33:30PM +0800, Rafael J. Wysocki wrote:
On Fri, Jun 16, 2023 at 9:08 AM Huang Rui ray.huang@amd.com wrote:
On Mon, Jun 12, 2023 at 07:36:10PM +0800, Karny, Wyes wrote:
amd-pstate passive mode driver is hyphenated. So make amd-pstate active mode driver consistent with that rename "amd_pstate_epp" to "amd-pstate-epp".
Cc: stable@vger.kernel.org Fixes: ffa5096a7c33 ("cpufreq: amd-pstate: implement Pstate EPP support for the AMD processors") Reviewed-by: Gautham R. Shenoy gautham.shenoy@amd.com Signed-off-by: Wyes Karny wyes.karny@amd.com
Acked-by: Huang Rui ray.huang@amd.com
And yes, we should seprate it from cpupower as Rafael mentioned. cpupower tool may go to another repo.
Not only that.
It is generally better to send individual fixes that don't depend on anything else as separate patches, because this allows them to be picked up and fast-tracked at multiple levels (mainline, stable, distro kernels etc.).
Get it. Thanks for the clarification.
Best Regards, Ray
linux-stable-mirror@lists.linaro.org
-
Huang Rui -
Rafael J. Wysocki -
Wyes Karny -
Yuan, Perry