| Message ID | 20230612113615.205353-2-wyes.karny@amd.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | cpupower: Add various feature control support for amd_pstate | expand |
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
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 >
[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 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
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, };
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(-)