Re: [PATCH 4/6] dt-bindings: net: bluetooth: Document Qualcomm IPQ5018 Bluetooth controller
From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2026-06-26 10:53:50
Also in:
linux-arm-msm, linux-block, linux-bluetooth, linux-devicetree, linux-mmc, linux-remoteproc, linux-wireless, lkml
On Thu, Jun 25, 2026 at 06:10:08PM +0400, George Moussalem wrote:
quoted hunk ↗ jump to hunk
Document the Qualcomm IPQ5018 Bluetooth controller. Signed-off-by: George Moussalem <redacted> --- .../bindings/net/bluetooth/qcom,ipq5018-bt.yaml | 63 ++++++++++++++++++++++ 1 file changed, 63 insertions(+)diff --git a/Documentation/devicetree/bindings/net/bluetooth/qcom,ipq5018-bt.yaml b/Documentation/devicetree/bindings/net/bluetooth/qcom,ipq5018-bt.yaml new file mode 100644 index 000000000000..afd33f851858 --- /dev/null +++ b/Documentation/devicetree/bindings/net/bluetooth/qcom,ipq5018-bt.yaml@@ -0,0 +1,63 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/net/bluetooth/qcom,ipq5018-bt.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Qualcomm IPQ5018 Bluetooth + +maintainers: + - George Moussalem <george.moussalem@outlook.com> + +properties: + compatible: + enum: + - qcom,ipq5018-bt + + interrupts: + items: + - description: + Interrupt line from the M0 Bluetooth Subsystem to the host processor
What is M0? Anyway, this part feels completely redundant. Can "interrupts" property be anything else than an interrupt line from the device to the host processor?
+ to notify it of events such as re
This feels useful, but cut/incomplete.
+
+ qcom,ipc:
+ $ref: /schemas/types.yaml#/definitions/phandle-array
+ items:
+ - items:
+ - description: phandle to a syscon node representing the APCS registers
+ - description: u32 representing offset to the register within the syscon
+ - description: u32 representing the ipc bit within the register
+ description: |
+ These entries specify the outgoing IPC bit used for signaling the remote
+ M0 BTSS core of a host event or for sending an ACK if the remote processor
+ expects it.
+
+ qcom,rproc:
+ $ref: /schemas/types.yaml#/definitions/phandle
+ description:
+ Phandle to the remote processor node representing the M0 BTSS core.
+
+required:
+ - compatible
+ - interrupts
+ - qcom,ipc
+ - qcom,rproc
+
+allOf:
+ - $ref: bluetooth-controller.yaml#
+ - $ref: qcom,bluetooth-common.yaml
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
+
+ bluetooth: bluetooth {Drop unused label
+ compatible = "qcom,ipq5018-bt"; + + qcom,ipc = <&apcs_glb 8 23>; + interrupts = <GIC_SPI 162 IRQ_TYPE_EDGE_RISING>;
No firmware to load? It feels like remoteproc node split is fake. The property qcom,rproc is even more supporting that case. Shouldn't this be simply one device - bluetooth? What sort of two devices do you have exactly? How can I identify them in the hardware?
+ + qcom,rproc = <&m0_btss>;
Best regards, Krzysztof