diff mbox series

[v2,17/17] dt-bindings: net: wireless: cc33xx: Add ti,cc33xx.yaml

Message ID 20240609182102.2950457-18-michael.nemanov@ti.com
State New
Headers show
Series wifi: cc33xx: Add driver for new TI CC33xx wireless device family | expand

Commit Message

Nemanov, Michael June 9, 2024, 6:21 p.m. UTC 
From: Michael Nemanov <michael.nemanov@ti.com>

---
 .../bindings/net/wireless/ti,cc33xx.yaml      | 60 +++++++++++++++++++
 1 file changed, 60 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/net/wireless/ti,cc33xx.yaml

Comments

Krzysztof Kozlowski June 10, 2024, 8:17 a.m. UTC | #1 
On 10/06/2024 10:16, Krzysztof Kozlowski wrote:
> On 09/06/2024 20:21, michael.nemanov@ti.com wrote:
>> From: Michael Nemanov <michael.nemanov@ti.com>
>>
> 
> Missing commit msg (explain the hardware), missing SoB.

And now I found that you actually received such feedback and you just
ignored it.

Go back to v1 and respond to each feedback you got.

Best regards,
Krzysztof
Nemanov, Michael June 13, 2024, 7:39 a.m. UTC | #2 
On 10/6/2024 11:17 AM, Krzysztof Kozlowski wrote:
>> On 09/06/2024 20:21, michael.nemanov@ti.com wrote:
>>> From: Michael Nemanov <michael.nemanov@ti.com>
>>>
>> 
>> Missing commit msg (explain the hardware), missing SoB.
> 
> And now I found that you actually received such feedback and you just
> ignored it.
> 
> Go back to v1 and respond to each feedback you got.
> 
> Best regards,
> Krzysztof
> 

Oversight on my part, will fix.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/net/wireless/ti,cc33xx.yaml b/Documentation/devicetree/bindings/net/wireless/ti,cc33xx.yaml
new file mode 100644
index 000000000000..08ab2ed93dba
--- /dev/null
+++ b/Documentation/devicetree/bindings/net/wireless/ti,cc33xx.yaml
@@ -0,0 +1,60 @@ 
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/net/wireless/ti,cc33xx.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments CC33xx Wireless LAN Controller
+
+maintainers:
+  - Michael Nemanov <michael.nemanov@ti.com>
+
+description:
+  These are dt entries for the IEEE 802.11ax chips CC33xx from Texas Instruments.
+  Currently, these chips must be connected via SDIO.
+
+properties:
+  compatible:
+    enum:
+      - ti,cc3300
+      - ti,cc3301
+      - ti,cc3350
+      - ti,cc3351
+
+  reg:
+    description:
+      For WLAN communication, <reg> must be set to 2.
+    maxItems: 1
+
+  interrupts:
+    description: The interrupt line. Can be IRQ_TYPE_EDGE_RISING or IRQ_TYPE_LEVEL_HIGH.
+      When SDIO is used, the "in-band" interrupt provided by the SDIO bus is used
+      unless an interrupt is defined in the Device Tree.
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/irq.h>
+
+    // SDIO example:
+    mmc3 {
+        vmmc-supply = <&wlan_en_reg>;
+        bus-width = <4>;
+        cap-power-off-card;
+        keep-power-in-suspend;
+
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        cc33xx: cc33xx@0 {
+            compatible = "ti,cc3300";
+            reg = <2>;
+            interrupts = <19 IRQ_TYPE_EDGE_RISING>;
+        };
+    };