[PATCH v3 22/65] clk: davinci: da8xx-cfgchip: Add a determine_rate hook
David Lechner
david at lechnology.com
Wed Apr 5 08:04:04 PDT 2023
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.
More information about the Linux-mediatek
mailing list