Re: [PATCH v3 03/42] clk: ep93xx: add DT support for Cirrus EP93xx

From: Nikita Shubin
Date: Fri Jul 28 2023 - 04:39:38 EST


Hello Stephen!

On Thu, 2023-07-20 at 16:27 -0700, Stephen Boyd wrote:
> Quoting Nikita Shubin via B4 Relay (2023-07-20 04:29:03)
>
> > +static int clk_hw_register_div(struct ep93xx_clk *clk,
> > +                                         const char *name,
> > +                                         const char *parent_name,
>
> Please try to pass a direct pointer to the parent clk_hw instead or
> if
> the clk exists outside the controller pass a DT index.

I'll use parent_data everywhere, however i currently can't pass "xtali"
as index, as: 

- moved all clk's that are not used via device tree to SoC driver 
https://lore.kernel.org/b4-sent/20230605-ep93xx-v3-7-3d63a5f1103e@xxxxxxxxxxx/

and SoC driver isn't a OF clock provider, should i make it clock
provider as well ?