| Message ID | 20240620020015.4021696-3-jacobe.zang@wesion.com |
|---|---|
| State | New |
| Headers | show |
| Series | Add AP6275P wireless support | expand |
On June 21, 2024 3:45:51 AM Jacobe Zang <jacobe.zang@wesion.com> wrote: > Hello, > >> Where is the binding for this? > I seperate the binding in this patch " [PATCH v1 1/3] arm64: dts: rockchip: > Add AP6275P wireless support to Khadas Edge 2 ", the specific code is > + clocks = <&hym8563>; > + clock-names = "32k"; > > Should I combine these two patch to one? That's not the binding. That is the device tree specification which must adhere to the binding specification under Documentation/device tree/bindings/net/wireless. Regards, Arend
On 21/06/2024 03:45, Jacobe Zang wrote: > Hello, > >> Where is the binding for this? > I seperate the binding in this patch " [PATCH v1 1/3] arm64: dts: rockchip: Add AP6275P wireless support to Khadas Edge 2 ", the specific code is > + clocks = <&hym8563>; > + clock-names = "32k"; > > Should I combine these two patch to one? That's DTS, not binding. I ask about Devicetree binding. Best regards, Krzysztof
On 21/06/2024 09:45, Jacobe Zang wrote: >> That's DTS, not binding. I ask about Devicetree binding. > > Ok... I have grep in all dts files and can't find wifi node which is under pcie node has clock. So should I add an example in the yaml file? No, that's example, not binding. Validate your DTS with dtbs_check. Best regards, Krzysztof
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c index 06698a714b523..f241e1757d7e3 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c @@ -3,6 +3,7 @@ * Copyright (c) 2014 Broadcom Corporation */ +#include <linux/clk.h> #include <linux/kernel.h> #include <linux/module.h> #include <linux/firmware.h> @@ -2411,6 +2412,7 @@ brcmf_pcie_probe(struct pci_dev *pdev, const struct pci_device_id *id) struct brcmf_pciedev *pcie_bus_dev; struct brcmf_core *core; struct brcmf_bus *bus; + struct clk *clk; if (!id) { id = pci_match_id(brcmf_pcie_devid_table, pdev); @@ -2422,6 +2424,14 @@ brcmf_pcie_probe(struct pci_dev *pdev, const struct pci_device_id *id) brcmf_dbg(PCIE, "Enter %x:%x\n", pdev->vendor, pdev->device); + clk = devm_clk_get_optional_enabled(&pdev->dev, "32k"); + if (IS_ERR(clk)) + return PTR_ERR(clk); + if (clk) { + dev_info(&pdev->dev, "enabling 32kHz clock\n"); + clk_set_rate(clk, 32768); + } + ret = -ENOMEM; devinfo = kzalloc(sizeof(*devinfo), GFP_KERNEL); if (devinfo == NULL)
WiFi modules often require 32kHz clock to function. Add support to enable the clock to PCIe driver. Signed-off-by: Jacobe Zang <jacobe.zang@wesion.com> --- .../net/wireless/broadcom/brcm80211/brcmfmac/pcie.c | 10 ++++++++++ 1 file changed, 10 insertions(+)