| Date | Wed, 5 Apr 2023 10:04:04 -0500 | Subject | Re: [PATCH v3 22/65] clk: davinci: da8xx-cfgchip: Add a determine_rate hook | From | David Lechner <> |
| |
On 4/4/23 5:11 AM, Maxime Ripard wrote: > The Davinci DA8xxx cfgchip "clk48" clock implements a mux with a > set_parent hook, but doesn't provide a determine_rate implementation. > > This is a bit odd, since set_parent() is there to, as its name implies, > change the parent of a clock. However, the most likely candidate to > trigger that parent change is a call to clk_set_rate(), with > determine_rate() figuring out which parent is the best suited for a > given rate. >
As mentioned in my previous review, parent is selected by device tree and should never be changed after init.
|