Re: [PATCH v2] clk: keystone: don't cache clock rate
From: Brian Masney
Date: Mon May 11 2026 - 10:46:48 EST
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>