Re: [RFC PATCH 3/9] dt-bindings: clock: Add apple,cluster-clk binding
From: Hector Martin
Date: Tue Oct 12 2021 - 09:48:23 EST
On 12/10/2021 18.57, Viresh Kumar wrote:
I didn't realize earlier that we have moved out of lists :)
Whoops, sorry, I was on mobile and must've hit the wrong reply button!
My apologies.
On 12-10-21, 18:54, Hector Martin "marcan" wrote:
Typically cpufreq-dt is used with clock drivers that directly take
the clock frequency and do whatever voodoo is necessary to set it
for the CPU. But here, the hardware just wants to know the index,
and does everything itself. So we need to encode that somewhere, to
avoid hardcoding it in the clock driver.
In general, based on how these SoCs are designed, we're trying to
avoid having tables of volatile information in the drivers, and
instead keep everything in the DT. This means we have a good chance
that these drivers will continue to work with future SoC
generations, since Apple doesn't change register definitions
randomly most of the time.
Yeah I get that and it is actually better this way. I just wanted to
point out that we didn't think of it this way earlier :)
Yeah, makes sense. Seems to work fine :)
--
Hector Martin (marcan@xxxxxxxxx)
Public Key: https://mrcn.st/pub