Re: [PATCH v2] clk: keystone: don't cache clock rate

From: Nishanth Menon

Date: Tue May 12 2026 - 06:51:02 EST


On 10:36-20260511, Brian Masney wrote:
> On Thu, May 07, 2026 at 11:09:34AM -0500, a-christidis@xxxxxx wrote:
> > From: Michael Walle <mwalle@xxxxxxxxxx>
> >
> > The TISCI firmware will return 0 if the clock or consumer is not
> > enabled although there is a stored value in the firmware. IOW a call to
> > set rate will work but at get rate will always return 0 if the clock is
> > disabled.
> > The clk framework will try to cache the clock rate when it's requested
> > by a consumer. If the clock or consumer is not enabled at that point,
> > the cached value is 0, which is wrong. Thus, disable the cache
> > altogether.
> >
> > Signed-off-by: Michael Walle <mwalle@xxxxxxxxxx>
> > Reviewed-by: Kevin Hilman <khilman@xxxxxxxxxxxx>
> > Reviewed-by: Randolph Sapp <rs@xxxxxx>
> > Reviewed-by: Nishanth Menon <nm@xxxxxx>
> > Signed-off-by: Antonios Christidis <a-christidis@xxxxxx>
>
> Reviewed-by: Brian Masney <bmasney@xxxxxxxxxx>
>

Brian,

Could you clarify if I need to take it via my tree to arnd or if this
patch will go via the clk tree?

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource