Re: [PATCH v3 21/65] clk: davinci: da8xx-cfgchip: Add a determine_rate hook

On 4/4/23 5:11 AM, Maxime Ripard wrote:

The Davinci DA8xxx cfgchip mux 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.