Re: [PATCH v4 4/5] wifi: brcmfmac: Add optional lpo clock enable support
From: Dragan Simic <hidden>
Date: 2024-07-29 09:12:25
Also in:
linux-arm-kernel, linux-devicetree, linux-rockchip, linux-wireless, lkml
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:quoted
Hello Jacobe, [...]quoted
+ 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)".
quoted
quoted
+ brcmf_dbg(INFO, "enabling 32kHz clock\n"); + clk_set_rate(clk, 32768); + } + if (!np || !of_device_is_compatible(np, "brcm,bcm4329-fmac")) return;