Re: [PATCH] clk: fixed: handle failed clk setup

[Emil Abildgaard Svendsen <EMAS@bang-olufsen.dk>]

On Mon, Apr 17, 2023 at 05:34:45PM -0700, Stephen Boyd wrote:
> Quoting Emil Abildgaard Svendsen (2023-04-14 05:52:16)
> > When initializing clock providers "of_clk_init" will try and init
> > parents first. But if parent clock is provided by a platform driver it
> > can't. Then clocks will be forced on and OF_POPULATED flag will be set
> > blindly. So if setup failes e.g. with -EPROBE_DEFER the clock will not
> > be probed later on.
> 
> Did something break this recently? For example, commit c28cd1f3433c
> ("clk: Mark a fwnode as initialized when using CLK_OF_DECLARE() macro")?

Sorry for missing this mail. But no nothing broke I just added support
for some HW in a DT where the clock parent has a lot of dependencies and
has a likelihood of getting deferred until all the dependencies has been
probed. 

> > 
> > This patch will clear the OF_POPULATED falg if clock setup failes.
> 
> s/falg/flag/

Thank you! I will update the commit message. I'm still new to the
mailing list. What is the preferred way? Sending PATCH v2 or just resend
the updated patch to the mailing list?