Re: [PATCH v2] arch_topology: Fix putting invalid cpu clk

From: Sudeep Holla
Date: Tue Mar 17 2020 - 02:42:01 EST


On Tue, Mar 17, 2020 at 02:33:08PM +0800, Jeffy Chen wrote:
> Add a sanity check before putting the cpu clk.
>
> Fixes: b8fe128dad8f (âarch_topology: Adjust initial CPU capacities with current freq")


Interesting, I had NACKed the original patch in v1 IIRC.
I was against for adding clk into arch topology as not all platform
control CPUFreq via clk APIs. Anyways, as fix this makes sense, but
I need to take a look and find ways to remove it if possible, but that's
for some other time.

--
Regards,
Sudeep