Quoting Daniel Palmer (2021-02-10 18:28:40)

Hi Stephen,

On Wed, 10 Feb 2021 at 11:29, Stephen Boyd [off-list ref] wrote:

quoted

The child clks should be using clk_parent_data to point to the parent
clks through DT. That way the name of the clk doesn't matter except for
debug purposes.

I think I get it now. I was using of_clk_parent_fill() to get the
parent clocks sourced
from the mpll but I seems like I should be filling out an array of
struct clk_parent_data
with the indices of the parents and using
clk_register_composite_pdata() etc instead.

Yes that's the idea. Thanks!