Hello Ondrej, On 2024-07-29 10:44, Ondřej Jirman wrote:
On Mon, Jul 29, 2024 at 09:12:20AM GMT, Dragan Simic wrote:Hello Jacobe, [...] > > + clk = devm_clk_get_optional_enabled(dev, "lpo"); > + if (IS_ERR(clk)) > + if (clk) { These two lines looks really confusing. Shouldn't it be just a single "if (!IS_ERR(clk)) {" line instead?It should be `!IS_ERR(clk) && clk` otherwise the debug message will be incorrect.
Ah, I see now, thanks. There's also IS_ERR_OR_NULL, so the condition can actually be "!IS_ERR_OR_NULL(clk)".
> + brcmf_dbg(INFO, "enabling 32kHz clock\n"); > + clk_set_rate(clk, 32768); > + } > + > if (!np || !of_device_is_compatible(np, "brcm,bcm4329-fmac")) > return;
