Messages in this thread | | | Subject | Re: [PATCH v6 3/7] powercap/drivers/dtpm: Simplify the dtpm table | From | Daniel Lezcano <> | Date | Tue, 30 Nov 2021 17:46:24 +0100 |
| |
On 26/11/2021 20:29, Rafael J. Wysocki wrote: > On Fri, Nov 26, 2021 at 8:10 PM Doug Smythies <dsmythies@telus.net> wrote: >> >> On Fri, Nov 26, 2021 at 9:22 AM Rafael J. Wysocki <rafael@kernel.org> wrote: >>> >>> Hi Doug, >>> >>> On Fri, Nov 26, 2021 at 6:08 PM Doug Smythies <dsmythies@telus.net> wrote: >>>> >>>> Hi Daniel, >>>> >>>> This patch introduces a regression, at least on my test system. >>>> I can no longer change CPU frequency scaling drivers, for example >>>> from intel_cpufreq (A.K.A intel_pstate in passive mode) to intel_pstate >>>> (A.K.A. active mode). The task just hangs forever. >>>> >>>> I bisected the kernel and got this commit as the result. >>>> As a double check, I reverted this commit: >>>> 7a89d7eacf8e84f2afb94db5ae9d9f9faa93f01c >>>> on kernel 5.16-rc2 and the issue was resolved.
[ ... ]
>> -static int __init dtpm_cpu_init(void) >> +static int dtpm_cpu_init(void) >> { >> int ret; > > OK > > This needs to be fixed or we'll need to revert commit > 7a89d7eacf8e84f2afb94db5ae9d9f9faa93f01c for 5.16.
I've send a fix for that and Doug tested it (thanks) [1]
-- Daniel
[1] https://lore.kernel.org/all/CAAYoRsWEXoe_LjuHuQUL3Tdov0JVW887T4ciUTVOC410mZjgvA@mail.gmail.com/
-- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
| |