On 7/29/2024 11:12 AM, Dragan Simic wrote:
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)".
++ best suggestion
> + brcmf_dbg(INFO, "enabling 32kHz clock\n");
> + clk_set_rate(clk, 32768);
> + }
> +
> if (!np || !of_device_is_compatible(np, "brcm,bcm4329-fmac"))
> return;