2021-03-15 11:00:36

by Linus Walleij

[permalink] [raw]
Subject: [PATCH v2] dt-bindings: Convert the BCM4329 bindings to YAML and extend

This converts the BCM4329 family bindings to YAML schema, and
extends and fixes the bindings like this:

- Name the bindings after the first chip (BCM4329) since
wildcards like 43xx are nowadays frowned upon by the DT
binding reviewers. We call this the "BCM4329 family"
- Add compatible strings for all the variants that seem to
exist in the wild. (Derived from firmware listings.)
- Support specific-to-generic compatible strings (as already
in use in DTS files).
- Add required reg property (SDIO function number)
- Add reset-gpios property (some systems wire this to a GPIO
line).
- I have only listed Arend as maintainer for now, volunteers
can be added.

Cc: Arend van Spriel <[email protected]>
Cc: Franky Lin <[email protected]>
Cc: Hante Meuleman <[email protected]>
Cc: Chi-hsien Lin <[email protected]>
Cc: Wright Feng <[email protected]>
Cc: Chung-hsien Hsu <[email protected]>
Cc: Julian Calaby <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Signed-off-by: Linus Walleij <[email protected]>
---
ChangeLog v1->v2:
- Specify specific-to-generic compatible strings such as
"brcm,bcm4334-fmac", "brcm,bcm4329-fmac"
- Also allow just "brcm,bcm4329-fmac" for legacy platforms.
- Use this scheme in the example.
---
.../net/wireless/brcm,bcm4329-fmac.yaml | 101 ++++++++++++++++++
.../net/wireless/brcm,bcm43xx-fmac.txt | 38 -------
2 files changed, 101 insertions(+), 38 deletions(-)
create mode 100644 Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml
delete mode 100644 Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt

diff --git a/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml b/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml
new file mode 100644
index 000000000000..b5fcc73ce6be
--- /dev/null
+++ b/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml
@@ -0,0 +1,101 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/net/wireless/brcm,bcm4329-fmac.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Broadcom BCM4329 family fullmac wireless SDIO devices
+
+maintainers:
+ - Arend van Spriel <[email protected]>
+
+description:
+ The Broadcom Single chip MAC part for the BCM4329 family and
+ later Cypress chips in the same family named CYW4373 and similar.
+ These chips also have a Bluetooth portion described in a separate
+ binding.
+
+properties:
+ compatible:
+ oneOf:
+ - items:
+ - enum:
+ - brcm,bcm43143-fmac
+ - brcm,bcm4341b0-fmac
+ - brcm,bcm4341b4-fmac
+ - brcm,bcm4341b5-fmac
+ - brcm,bcm4329-fmac
+ - brcm,bcm4330-fmac
+ - brcm,bcm4334-fmac
+ - brcm,bcm43340-fmac
+ - brcm,bcm4335-fmac
+ - brcm,bcm43362-fmac
+ - brcm,bcm4339-fmac
+ - brcm,bcm43430a0-fmac
+ - brcm,bcm43430a1-fmac
+ - brcm,bcm43455-fmac
+ - brcm,bcm43456-fmac
+ - brcm,bcm4354-fmac
+ - brcm,bcm4356-fmac
+ - brcm,bcm4359-fmac
+ - cypress,cyw4373-fmac
+ - cypress,cyw43012-fmac
+ - const: brcm,bcm4329-fmac
+ - const: brcm,bcm4329-fmac
+
+ reg:
+ description: SDIO function number for the device, for most cases
+ this will be 1.
+
+ interrupts:
+ maxItems: 1
+ description: Out-of-band (OOB) IRQ line for waking up the host
+ in response to WLAN activity. This corresponds to the HOST_WAKE
+ line into the chip.
+
+ interrupt-names:
+ description: Name for the OOB IRQ, this must be set to "host-wake".
+ const: host-wake
+
+ brcm,drive-strength:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description: Drive strength used for the SDIO pins on the device in mA.
+ minimum: 0
+ maximum: 32
+
+ reset-gpios:
+ maxItems: 1
+ description: A GPIO line connected to the WL_RST line, if present
+ this shall be flagged as active low.
+
+required:
+ - compatible
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/irq.h>
+ #include <dt-bindings/gpio/gpio.h>
+ [email protected] {
+ compatible = "arm,pl18x", "arm,primecell";
+ reg = <0x80118000 0x1000>;
+ clocks = <&clk 0>, <&clk 1>;
+ clock-names = "mclk", "apb_pclk";
+ interrupts = <0 60 IRQ_TYPE_LEVEL_HIGH>;
+ bus-width = <4>;
+ non-removable;
+ vmmc-supply = <&wl_bt_reg>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ [email protected] {
+ compatible = "brcm,bcm4334-fmac", "brcm,bcm4329-fmac";
+ reg = <1>;
+ interrupt-parent = <&gpio>;
+ interrupts = <24 IRQ_TYPE_EDGE_FALLING>;
+ interrupt-names = "host-wake";
+ reset-gpios = <&gpio 23 GPIO_ACTIVE_LOW>;
+ };
+ };
diff --git a/Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt b/Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt
deleted file mode 100644
index cffb2d6876e3..000000000000
--- a/Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt
+++ /dev/null
@@ -1,38 +0,0 @@
-Broadcom BCM43xx Fullmac wireless SDIO devices
-
-This node provides properties for controlling the Broadcom wireless device. The
-node is expected to be specified as a child node to the SDIO controller that
-connects the device to the system.
-
-Required properties:
-
- - compatible : Should be "brcm,bcm4329-fmac".
-
-Optional properties:
- - brcm,drive-strength : drive strength used for SDIO pins on device in mA
- (default = 6).
- - interrupts : specifies attributes for the out-of-band interrupt (host-wake).
- When not specified the device will use in-band SDIO interrupts.
- - interrupt-names : name of the out-of-band interrupt, which must be set
- to "host-wake".
-
-Example:
-
-mmc3: [email protected] {
- #address-cells = <1>;
- #size-cells = <0>;
-
- pinctrl-names = "default";
- pinctrl-0 = <&mmc3_pins_a>;
- vmmc-supply = <&reg_vmmc3>;
- bus-width = <4>;
- non-removable;
-
- brcmf: [email protected] {
- reg = <1>;
- compatible = "brcm,bcm4329-fmac";
- interrupt-parent = <&pio>;
- interrupts = <10 8>; /* PH10 / EINT10 */
- interrupt-names = "host-wake";
- };
-};
--
2.29.2


2021-03-19 13:47:02

by Ulf Hansson

[permalink] [raw]
Subject: Re: [PATCH v2] dt-bindings: Convert the BCM4329 bindings to YAML and extend

On Mon, 15 Mar 2021 at 12:00, Linus Walleij <[email protected]> wrote:
>
> This converts the BCM4329 family bindings to YAML schema, and
> extends and fixes the bindings like this:
>
> - Name the bindings after the first chip (BCM4329) since
> wildcards like 43xx are nowadays frowned upon by the DT
> binding reviewers. We call this the "BCM4329 family"
> - Add compatible strings for all the variants that seem to
> exist in the wild. (Derived from firmware listings.)
> - Support specific-to-generic compatible strings (as already
> in use in DTS files).
> - Add required reg property (SDIO function number)
> - Add reset-gpios property (some systems wire this to a GPIO
> line).
> - I have only listed Arend as maintainer for now, volunteers
> can be added.
>
> Cc: Arend van Spriel <[email protected]>
> Cc: Franky Lin <[email protected]>
> Cc: Hante Meuleman <[email protected]>
> Cc: Chi-hsien Lin <[email protected]>
> Cc: Wright Feng <[email protected]>
> Cc: Chung-hsien Hsu <[email protected]>
> Cc: Julian Calaby <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Linus Walleij <[email protected]>

Reviewed-by: Ulf Hansson <[email protected]>

Kind regards
Uffe

> ---
> ChangeLog v1->v2:
> - Specify specific-to-generic compatible strings such as
> "brcm,bcm4334-fmac", "brcm,bcm4329-fmac"
> - Also allow just "brcm,bcm4329-fmac" for legacy platforms.
> - Use this scheme in the example.
> ---
> .../net/wireless/brcm,bcm4329-fmac.yaml | 101 ++++++++++++++++++
> .../net/wireless/brcm,bcm43xx-fmac.txt | 38 -------
> 2 files changed, 101 insertions(+), 38 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml
> delete mode 100644 Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt
>
> diff --git a/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml b/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml
> new file mode 100644
> index 000000000000..b5fcc73ce6be
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml
> @@ -0,0 +1,101 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/net/wireless/brcm,bcm4329-fmac.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Broadcom BCM4329 family fullmac wireless SDIO devices
> +
> +maintainers:
> + - Arend van Spriel <[email protected]>
> +
> +description:
> + The Broadcom Single chip MAC part for the BCM4329 family and
> + later Cypress chips in the same family named CYW4373 and similar.
> + These chips also have a Bluetooth portion described in a separate
> + binding.
> +
> +properties:
> + compatible:
> + oneOf:
> + - items:
> + - enum:
> + - brcm,bcm43143-fmac
> + - brcm,bcm4341b0-fmac
> + - brcm,bcm4341b4-fmac
> + - brcm,bcm4341b5-fmac
> + - brcm,bcm4329-fmac
> + - brcm,bcm4330-fmac
> + - brcm,bcm4334-fmac
> + - brcm,bcm43340-fmac
> + - brcm,bcm4335-fmac
> + - brcm,bcm43362-fmac
> + - brcm,bcm4339-fmac
> + - brcm,bcm43430a0-fmac
> + - brcm,bcm43430a1-fmac
> + - brcm,bcm43455-fmac
> + - brcm,bcm43456-fmac
> + - brcm,bcm4354-fmac
> + - brcm,bcm4356-fmac
> + - brcm,bcm4359-fmac
> + - cypress,cyw4373-fmac
> + - cypress,cyw43012-fmac
> + - const: brcm,bcm4329-fmac
> + - const: brcm,bcm4329-fmac
> +
> + reg:
> + description: SDIO function number for the device, for most cases
> + this will be 1.
> +
> + interrupts:
> + maxItems: 1
> + description: Out-of-band (OOB) IRQ line for waking up the host
> + in response to WLAN activity. This corresponds to the HOST_WAKE
> + line into the chip.
> +
> + interrupt-names:
> + description: Name for the OOB IRQ, this must be set to "host-wake".
> + const: host-wake
> +
> + brcm,drive-strength:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description: Drive strength used for the SDIO pins on the device in mA.
> + minimum: 0
> + maximum: 32
> +
> + reset-gpios:
> + maxItems: 1
> + description: A GPIO line connected to the WL_RST line, if present
> + this shall be flagged as active low.
> +
> +required:
> + - compatible
> + - reg
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/interrupt-controller/irq.h>
> + #include <dt-bindings/gpio/gpio.h>
> + [email protected] {
> + compatible = "arm,pl18x", "arm,primecell";
> + reg = <0x80118000 0x1000>;
> + clocks = <&clk 0>, <&clk 1>;
> + clock-names = "mclk", "apb_pclk";
> + interrupts = <0 60 IRQ_TYPE_LEVEL_HIGH>;
> + bus-width = <4>;
> + non-removable;
> + vmmc-supply = <&wl_bt_reg>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + [email protected] {
> + compatible = "brcm,bcm4334-fmac", "brcm,bcm4329-fmac";
> + reg = <1>;
> + interrupt-parent = <&gpio>;
> + interrupts = <24 IRQ_TYPE_EDGE_FALLING>;
> + interrupt-names = "host-wake";
> + reset-gpios = <&gpio 23 GPIO_ACTIVE_LOW>;
> + };
> + };
> diff --git a/Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt b/Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt
> deleted file mode 100644
> index cffb2d6876e3..000000000000
> --- a/Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt
> +++ /dev/null
> @@ -1,38 +0,0 @@
> -Broadcom BCM43xx Fullmac wireless SDIO devices
> -
> -This node provides properties for controlling the Broadcom wireless device. The
> -node is expected to be specified as a child node to the SDIO controller that
> -connects the device to the system.
> -
> -Required properties:
> -
> - - compatible : Should be "brcm,bcm4329-fmac".
> -
> -Optional properties:
> - - brcm,drive-strength : drive strength used for SDIO pins on device in mA
> - (default = 6).
> - - interrupts : specifies attributes for the out-of-band interrupt (host-wake).
> - When not specified the device will use in-band SDIO interrupts.
> - - interrupt-names : name of the out-of-band interrupt, which must be set
> - to "host-wake".
> -
> -Example:
> -
> -mmc3: [email protected] {
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - pinctrl-names = "default";
> - pinctrl-0 = <&mmc3_pins_a>;
> - vmmc-supply = <&reg_vmmc3>;
> - bus-width = <4>;
> - non-removable;
> -
> - brcmf: [email protected] {
> - reg = <1>;
> - compatible = "brcm,bcm4329-fmac";
> - interrupt-parent = <&pio>;
> - interrupts = <10 8>; /* PH10 / EINT10 */
> - interrupt-names = "host-wake";
> - };
> -};
> --
> 2.29.2
>

2021-03-24 17:49:01

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH v2] dt-bindings: Convert the BCM4329 bindings to YAML and extend

On Mon, 15 Mar 2021 11:59:11 +0100, Linus Walleij wrote:
> This converts the BCM4329 family bindings to YAML schema, and
> extends and fixes the bindings like this:
>
> - Name the bindings after the first chip (BCM4329) since
> wildcards like 43xx are nowadays frowned upon by the DT
> binding reviewers. We call this the "BCM4329 family"
> - Add compatible strings for all the variants that seem to
> exist in the wild. (Derived from firmware listings.)
> - Support specific-to-generic compatible strings (as already
> in use in DTS files).
> - Add required reg property (SDIO function number)
> - Add reset-gpios property (some systems wire this to a GPIO
> line).
> - I have only listed Arend as maintainer for now, volunteers
> can be added.
>
> Cc: Arend van Spriel <[email protected]>
> Cc: Franky Lin <[email protected]>
> Cc: Hante Meuleman <[email protected]>
> Cc: Chi-hsien Lin <[email protected]>
> Cc: Wright Feng <[email protected]>
> Cc: Chung-hsien Hsu <[email protected]>
> Cc: Julian Calaby <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Linus Walleij <[email protected]>
> ---
> ChangeLog v1->v2:
> - Specify specific-to-generic compatible strings such as
> "brcm,bcm4334-fmac", "brcm,bcm4329-fmac"
> - Also allow just "brcm,bcm4329-fmac" for legacy platforms.
> - Use this scheme in the example.
> ---
> .../net/wireless/brcm,bcm4329-fmac.yaml | 101 ++++++++++++++++++
> .../net/wireless/brcm,bcm43xx-fmac.txt | 38 -------
> 2 files changed, 101 insertions(+), 38 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml
> delete mode 100644 Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt
>

Applied, thanks!