| Message ID | 20240210012114.489102-3-sre@kernel.org |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | UNI-T UTi260B support | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 0 errors, 1 warnings, 225 lines checked |
| robh/patch-applied | success | |
| robh/dt-meta-schema | fail | build log |
On Sat, 10 Feb 2024 02:18:06 +0100, Sebastian Reichel wrote: > Convert the i.MX Wireless External Interface Module binding to YAML. > > Signed-off-by: Sebastian Reichel <sre@kernel.org> > --- > .../devicetree/bindings/bus/fsl,imx-weim.yaml | 225 ++++++++++++++++++ > .../devicetree/bindings/bus/imx-weim.txt | 117 --------- > 2 files changed, 225 insertions(+), 117 deletions(-) > create mode 100644 Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml > delete mode 100644 Documentation/devicetree/bindings/bus/imx-weim.txt > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13): yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/bus/fsl,imx-weim.example.dtb: nor@0,0: $nodename:0: 'nor@0,0' does not match '^(flash|.*sram|nand)(@.*)?$' from schema $id: http://devicetree.org/schemas/mtd/mtd-physmap.yaml# Documentation/devicetree/bindings/bus/fsl,imx-weim.example.dtb: /example-1/weim@21b8000/acme@0,0: failed to match any schema with compatible: ['acme,whatever'] doc reference errors (make refcheckdocs): Warning: drivers/staging/fieldbus/Documentation/devicetree/bindings/fieldbus/arcx,anybus-controller.txt references a file that doesn't exist: Documentation/devicetree/bindings/bus/imx-weim.txt drivers/staging/fieldbus/Documentation/devicetree/bindings/fieldbus/arcx,anybus-controller.txt: Documentation/devicetree/bindings/bus/imx-weim.txt See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240210012114.489102-3-sre@kernel.org The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.
On 10/02/2024 02:18, Sebastian Reichel wrote: > Convert the i.MX Wireless External Interface Module binding to YAML. > > Signed-off-by: Sebastian Reichel <sre@kernel.org> > --- > .../devicetree/bindings/bus/fsl,imx-weim.yaml | 225 ++++++++++++++++++ > .../devicetree/bindings/bus/imx-weim.txt | 117 --------- > 2 files changed, 225 insertions(+), 117 deletions(-) > create mode 100644 Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml > delete mode 100644 Documentation/devicetree/bindings/bus/imx-weim.txt > > diff --git a/Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml b/Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml > new file mode 100644 > index 000000000000..3d27bdaef304 > --- /dev/null > +++ b/Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml > @@ -0,0 +1,225 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/bus/fsl,imx-weim.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: i.MX Wireless External Interface Module (WEIM) This patchset does not do what I asked to, but maybe that's just coincidence that you send conversion now. To recall: https://lore.kernel.org/all/cee1ca11-03bf-4a0b-9ff3-490457f9fbe8@linaro.org/ > + > +maintainers: > + - Shawn Guo <shawnguo@kernel.org> > + - Sascha Hauer <s.hauer@pengutronix.de> > + > +description: > + The term "wireless" does not imply that the WEIM is literally an interface > + without wires. It simply means that this module was originally designed for > + wireless and mobile applications that use low-power technology. The actual > + devices are instantiated from the child nodes of a WEIM node. > + > +properties: > + compatible: > + oneOf: > + - enum: > + - fsl,imx1-weim > + - fsl,imx27-weim > + - fsl,imx50-weim > + - fsl,imx51-weim > + - fsl,imx6q-weim > + - items: > + - enum: > + - fsl,imx31-weim > + - fsl,imx35-weim > + - const: fsl,imx27-weim > + - items: > + - enum: > + - fsl,imx6sx-weim > + - fsl,imx6ul-weim > + - const: fsl,imx6q-weim > + > + reg: > + maxItems: 1 > + > + clocks: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + '#address-cells': > + const: 2 > + > + '#size-cells': > + const: 1 > + > + ranges: true > + > + fsl,weim-cs-gpr: > + $ref: /schemas/types.yaml#/definitions/phandle > + description: | > + Phandle to the system General Purpose Register controller that contains > + WEIM CS GPR register, e.g. IOMUXC_GPR1 on i.MX6Q. IOMUXC_GPR1[11:0] > + should be set up as one of the following 4 possible values depending on > + the CS space configuration. > + > + IOMUXC_GPR1[11:0] CS0 CS1 CS2 CS3 > + --------------------------------------------- > + 05 128M 0M 0M 0M > + 033 64M 64M 0M 0M > + 0113 64M 32M 32M 0M > + 01111 32M 32M 32M 32M > + > + In case that the property is absent, the reset value or what bootloader > + sets up in IOMUXC_GPR1[11:0] will be used. > + > + fsl,burst-clk-enable: > + type: boolean > + description: > + The presence of this property indicates that the weim bus should operate > + in Burst Clock Mode. > + > + fsl,continuous-burst-clk: > + type: boolean > + description: > + Make Burst Clock to output continuous clock. Without this option Burst > + Clock will output clock only when necessary. > + > +patternProperties: > + "^.*@[0-7],[0-9a-f]+$": > + description: Devices attached to chip selects are represented as subnodes. > + type: object > + additionalProperties: true > + properties: > + fsl,weim-cs-timing: > + $ref: /schemas/types.yaml#/definitions/uint32-array > + description: > + Timing values for the child node. > + required: > + - fsl,weim-cs-timing You need to integrate it into mc-peripheral-props.yaml. Also check how other bus controllers define children schema. required: block goes here. > + > +allOf: > + - if: > + properties: > + compatible: > + not: > + contains: > + enum: > + - fsl,imx50-weim Best regards, Krzysztof
On Sat, Feb 10, 2024 at 02:18:06AM +0100, Sebastian Reichel wrote: > Convert the i.MX Wireless External Interface Module binding to YAML. > > Signed-off-by: Sebastian Reichel <sre@kernel.org> > --- > .../devicetree/bindings/bus/fsl,imx-weim.yaml | 225 ++++++++++++++++++ > .../devicetree/bindings/bus/imx-weim.txt | 117 --------- > 2 files changed, 225 insertions(+), 117 deletions(-) > create mode 100644 Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml > delete mode 100644 Documentation/devicetree/bindings/bus/imx-weim.txt > > diff --git a/Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml b/Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml > new file mode 100644 > index 000000000000..3d27bdaef304 > --- /dev/null > +++ b/Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml > @@ -0,0 +1,225 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/bus/fsl,imx-weim.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: i.MX Wireless External Interface Module (WEIM) > + > +maintainers: > + - Shawn Guo <shawnguo@kernel.org> > + - Sascha Hauer <s.hauer@pengutronix.de> > + > +description: > + The term "wireless" does not imply that the WEIM is literally an interface > + without wires. It simply means that this module was originally designed for > + wireless and mobile applications that use low-power technology. The actual > + devices are instantiated from the child nodes of a WEIM node. > + > +properties: > + compatible: > + oneOf: > + - enum: > + - fsl,imx1-weim > + - fsl,imx27-weim > + - fsl,imx50-weim > + - fsl,imx51-weim > + - fsl,imx6q-weim > + - items: > + - enum: > + - fsl,imx31-weim > + - fsl,imx35-weim > + - const: fsl,imx27-weim > + - items: > + - enum: > + - fsl,imx6sx-weim > + - fsl,imx6ul-weim > + - const: fsl,imx6q-weim > + > + reg: > + maxItems: 1 > + > + clocks: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + '#address-cells': > + const: 2 > + > + '#size-cells': > + const: 1 > + > + ranges: true > + > + fsl,weim-cs-gpr: > + $ref: /schemas/types.yaml#/definitions/phandle > + description: | > + Phandle to the system General Purpose Register controller that contains > + WEIM CS GPR register, e.g. IOMUXC_GPR1 on i.MX6Q. IOMUXC_GPR1[11:0] > + should be set up as one of the following 4 possible values depending on > + the CS space configuration. > + > + IOMUXC_GPR1[11:0] CS0 CS1 CS2 CS3 > + --------------------------------------------- > + 05 128M 0M 0M 0M > + 033 64M 64M 0M 0M > + 0113 64M 32M 32M 0M > + 01111 32M 32M 32M 32M > + > + In case that the property is absent, the reset value or what bootloader > + sets up in IOMUXC_GPR1[11:0] will be used. > + > + fsl,burst-clk-enable: > + type: boolean > + description: > + The presence of this property indicates that the weim bus should operate > + in Burst Clock Mode. > + > + fsl,continuous-burst-clk: > + type: boolean > + description: > + Make Burst Clock to output continuous clock. Without this option Burst > + Clock will output clock only when necessary. > + > +patternProperties: > + "^.*@[0-7],[0-9a-f]+$": > + description: Devices attached to chip selects are represented as subnodes. > + type: object > + additionalProperties: true > + properties: > + fsl,weim-cs-timing: > + $ref: /schemas/types.yaml#/definitions/uint32-array > + description: > + Timing values for the child node. > + required: > + - fsl,weim-cs-timing This needs to go in its own schema doc and then added to memory-controllers/mc-peripheral-props.yaml We should probably also move this binding to memory-controllers/ > + > +allOf: > + - if: > + properties: > + compatible: > + not: > + contains: > + enum: > + - fsl,imx50-weim > + - fsl,imx6q-weim > + then: > + properties: > + fsl,weim-cs-gpr: false > + fsl,burst-clk-enable: false > + - if: > + properties: > + fsl,burst-clk-enable: false > + then: > + properties: > + fsl,continuous-burst-clk: false > + - if: > + properties: > + compatible: > + contains: > + const: fsl,imx1-weim > + then: > + patternProperties: > + "^.*@[0-7],[0-9a-f]+$": > + properties: > + fsl,weim-cs-timing: > + items: > + items: > + - description: CSxU > + - description: CSxL > + - if: > + properties: > + compatible: > + contains: > + enum: > + - fsl,imx27-weim > + - fsl,imx31-weim > + - fsl,imx35-weim > + then: > + patternProperties: > + "^.*@[0-7],[0-9a-f]+$": > + properties: > + fsl,weim-cs-timing: > + items: > + items: > + - description: CSCRxU > + - description: CSCRxL > + - description: CSCRxA > + - if: > + properties: > + compatible: > + contains: > + enum: > + - fsl,imx50-weim > + - fsl,imx51-weim > + - fsl,imx6q-weim > + - fsl,imx6sx-weim > + - fsl,imx6ul-weim > + then: > + patternProperties: > + "^.*@[0-7],[0-9a-f]+$": > + properties: > + fsl,weim-cs-timing: > + items: > + items: > + - description: CSxGCR1 > + - description: CSxGCR2 > + - description: CSxRCR1 > + - description: CSxRCR2 > + - description: CSxWCR1 > + - description: CSxWCR2 > + > +required: > + - compatible > + - reg > + - clocks > + - '#address-cells' > + - '#size-cells' > + - ranges > + > +additionalProperties: false > + > +examples: > + - | > + weim@21b8000 { bus or external-bus > + compatible = "fsl,imx6q-weim"; > + reg = <0x021b8000 0x4000>; > + clocks = <&clks 196>; > + #address-cells = <2>; > + #size-cells = <1>; > + ranges = <0 0 0x08000000 0x08000000>; > + fsl,weim-cs-gpr = <&gpr>; > + > + nor@0,0 { > + compatible = "cfi-flash"; > + reg = <0 0 0x02000000>; > + #address-cells = <1>; > + #size-cells = <1>; > + bank-width = <2>; > + fsl,weim-cs-timing = <0x00620081 0x00000001 0x1c022000 > + 0x0000c000 0x1404a38e 0x00000000>; > + }; > + }; > + - | > + weim@21b8000 { > + compatible = "fsl,imx6q-weim"; > + reg = <0x021b8000 0x4000>; > + clocks = <&clks 196>; > + #address-cells = <2>; > + #size-cells = <1>; > + ranges = <0 0 0x08000000 0x02000000 > + 1 0 0x0a000000 0x02000000 > + 2 0 0x0c000000 0x02000000 > + 3 0 0x0e000000 0x02000000>; > + fsl,weim-cs-gpr = <&gpr>; > + > + acme@0,0 { > + compatible = "acme,whatever"; Real bindings only please. > + reg = <0 0 0x100>, <0 0x400000 0x800>, > + <1 0x400000 0x800>; > + fsl,weim-cs-timing = <0x024400b1 0x00001010 0x20081100 > + 0x00000000 0xa0000240 0x00000000>; > + }; > + };
Hi Rob, On Mon, Feb 12, 2024 at 07:16:31AM -0600, Rob Herring wrote: > [...] > > +examples: > > + - | > > + weim@21b8000 { > > bus or external-bus I used "memory-controller", which is what the other bindings in memory-controller use. -- Sebastian
diff --git a/Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml b/Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml new file mode 100644 index 000000000000..3d27bdaef304 --- /dev/null +++ b/Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml @@ -0,0 +1,225 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/bus/fsl,imx-weim.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: i.MX Wireless External Interface Module (WEIM) + +maintainers: + - Shawn Guo <shawnguo@kernel.org> + - Sascha Hauer <s.hauer@pengutronix.de> + +description: + The term "wireless" does not imply that the WEIM is literally an interface + without wires. It simply means that this module was originally designed for + wireless and mobile applications that use low-power technology. The actual + devices are instantiated from the child nodes of a WEIM node. + +properties: + compatible: + oneOf: + - enum: + - fsl,imx1-weim + - fsl,imx27-weim + - fsl,imx50-weim + - fsl,imx51-weim + - fsl,imx6q-weim + - items: + - enum: + - fsl,imx31-weim + - fsl,imx35-weim + - const: fsl,imx27-weim + - items: + - enum: + - fsl,imx6sx-weim + - fsl,imx6ul-weim + - const: fsl,imx6q-weim + + reg: + maxItems: 1 + + clocks: + maxItems: 1 + + interrupts: + maxItems: 1 + + '#address-cells': + const: 2 + + '#size-cells': + const: 1 + + ranges: true + + fsl,weim-cs-gpr: + $ref: /schemas/types.yaml#/definitions/phandle + description: | + Phandle to the system General Purpose Register controller that contains + WEIM CS GPR register, e.g. IOMUXC_GPR1 on i.MX6Q. IOMUXC_GPR1[11:0] + should be set up as one of the following 4 possible values depending on + the CS space configuration. + + IOMUXC_GPR1[11:0] CS0 CS1 CS2 CS3 + --------------------------------------------- + 05 128M 0M 0M 0M + 033 64M 64M 0M 0M + 0113 64M 32M 32M 0M + 01111 32M 32M 32M 32M + + In case that the property is absent, the reset value or what bootloader + sets up in IOMUXC_GPR1[11:0] will be used. + + fsl,burst-clk-enable: + type: boolean + description: + The presence of this property indicates that the weim bus should operate + in Burst Clock Mode. + + fsl,continuous-burst-clk: + type: boolean + description: + Make Burst Clock to output continuous clock. Without this option Burst + Clock will output clock only when necessary. + +patternProperties: + "^.*@[0-7],[0-9a-f]+$": + description: Devices attached to chip selects are represented as subnodes. + type: object + additionalProperties: true + properties: + fsl,weim-cs-timing: + $ref: /schemas/types.yaml#/definitions/uint32-array + description: + Timing values for the child node. + required: + - fsl,weim-cs-timing + +allOf: + - if: + properties: + compatible: + not: + contains: + enum: + - fsl,imx50-weim + - fsl,imx6q-weim + then: + properties: + fsl,weim-cs-gpr: false + fsl,burst-clk-enable: false + - if: + properties: + fsl,burst-clk-enable: false + then: + properties: + fsl,continuous-burst-clk: false + - if: + properties: + compatible: + contains: + const: fsl,imx1-weim + then: + patternProperties: + "^.*@[0-7],[0-9a-f]+$": + properties: + fsl,weim-cs-timing: + items: + items: + - description: CSxU + - description: CSxL + - if: + properties: + compatible: + contains: + enum: + - fsl,imx27-weim + - fsl,imx31-weim + - fsl,imx35-weim + then: + patternProperties: + "^.*@[0-7],[0-9a-f]+$": + properties: + fsl,weim-cs-timing: + items: + items: + - description: CSCRxU + - description: CSCRxL + - description: CSCRxA + - if: + properties: + compatible: + contains: + enum: + - fsl,imx50-weim + - fsl,imx51-weim + - fsl,imx6q-weim + - fsl,imx6sx-weim + - fsl,imx6ul-weim + then: + patternProperties: + "^.*@[0-7],[0-9a-f]+$": + properties: + fsl,weim-cs-timing: + items: + items: + - description: CSxGCR1 + - description: CSxGCR2 + - description: CSxRCR1 + - description: CSxRCR2 + - description: CSxWCR1 + - description: CSxWCR2 + +required: + - compatible + - reg + - clocks + - '#address-cells' + - '#size-cells' + - ranges + +additionalProperties: false + +examples: + - | + weim@21b8000 { + compatible = "fsl,imx6q-weim"; + reg = <0x021b8000 0x4000>; + clocks = <&clks 196>; + #address-cells = <2>; + #size-cells = <1>; + ranges = <0 0 0x08000000 0x08000000>; + fsl,weim-cs-gpr = <&gpr>; + + nor@0,0 { + compatible = "cfi-flash"; + reg = <0 0 0x02000000>; + #address-cells = <1>; + #size-cells = <1>; + bank-width = <2>; + fsl,weim-cs-timing = <0x00620081 0x00000001 0x1c022000 + 0x0000c000 0x1404a38e 0x00000000>; + }; + }; + - | + weim@21b8000 { + compatible = "fsl,imx6q-weim"; + reg = <0x021b8000 0x4000>; + clocks = <&clks 196>; + #address-cells = <2>; + #size-cells = <1>; + ranges = <0 0 0x08000000 0x02000000 + 1 0 0x0a000000 0x02000000 + 2 0 0x0c000000 0x02000000 + 3 0 0x0e000000 0x02000000>; + fsl,weim-cs-gpr = <&gpr>; + + acme@0,0 { + compatible = "acme,whatever"; + reg = <0 0 0x100>, <0 0x400000 0x800>, + <1 0x400000 0x800>; + fsl,weim-cs-timing = <0x024400b1 0x00001010 0x20081100 + 0x00000000 0xa0000240 0x00000000>; + }; + }; diff --git a/Documentation/devicetree/bindings/bus/imx-weim.txt b/Documentation/devicetree/bindings/bus/imx-weim.txt deleted file mode 100644 index e7f502070d77..000000000000 --- a/Documentation/devicetree/bindings/bus/imx-weim.txt +++ /dev/null @@ -1,117 +0,0 @@ -Device tree bindings for i.MX Wireless External Interface Module (WEIM) - -The term "wireless" does not imply that the WEIM is literally an interface -without wires. It simply means that this module was originally designed for -wireless and mobile applications that use low-power technology. - -The actual devices are instantiated from the child nodes of a WEIM node. - -Required properties: - - - compatible: Should contain one of the following: - "fsl,imx1-weim" - "fsl,imx27-weim" - "fsl,imx51-weim" - "fsl,imx50-weim" - "fsl,imx6q-weim" - - reg: A resource specifier for the register space - (see the example below) - - clocks: the clock, see the example below. - - #address-cells: Must be set to 2 to allow memory address translation - - #size-cells: Must be set to 1 to allow CS address passing - - ranges: Must be set up to reflect the memory layout with four - integer values for each chip-select line in use: - - <cs-number> 0 <physical address of mapping> <size> - -Optional properties: - - - fsl,weim-cs-gpr: For "fsl,imx50-weim" and "fsl,imx6q-weim" type of - devices, it should be the phandle to the system General - Purpose Register controller that contains WEIM CS GPR - register, e.g. IOMUXC_GPR1 on i.MX6Q. IOMUXC_GPR1[11:0] - should be set up as one of the following 4 possible - values depending on the CS space configuration. - - IOMUXC_GPR1[11:0] CS0 CS1 CS2 CS3 - --------------------------------------------- - 05 128M 0M 0M 0M - 033 64M 64M 0M 0M - 0113 64M 32M 32M 0M - 01111 32M 32M 32M 32M - - In case that the property is absent, the reset value or - what bootloader sets up in IOMUXC_GPR1[11:0] will be - used. - - - fsl,burst-clk-enable For "fsl,imx50-weim" and "fsl,imx6q-weim" type of - devices, the presence of this property indicates that - the weim bus should operate in Burst Clock Mode. - - - fsl,continuous-burst-clk Make Burst Clock to output continuous clock. - Without this option Burst Clock will output clock - only when necessary. This takes effect only if - "fsl,burst-clk-enable" is set. - -Timing property for child nodes. It is mandatory, not optional. - - - fsl,weim-cs-timing: The timing array, contains timing values for the - child node. We get the CS indexes from the address - ranges in the child node's "reg" property. - The number of registers depends on the selected chip: - For i.MX1, i.MX21 ("fsl,imx1-weim") there are two - registers: CSxU, CSxL. - For i.MX25, i.MX27, i.MX31 and i.MX35 ("fsl,imx27-weim") - there are three registers: CSCRxU, CSCRxL, CSCRxA. - For i.MX50, i.MX53 ("fsl,imx50-weim"), - i.MX51 ("fsl,imx51-weim") and i.MX6Q ("fsl,imx6q-weim") - there are six registers: CSxGCR1, CSxGCR2, CSxRCR1, - CSxRCR2, CSxWCR1, CSxWCR2. - -Example for an imx6q-sabreauto board, the NOR flash connected to the WEIM: - - weim: weim@21b8000 { - compatible = "fsl,imx6q-weim"; - reg = <0x021b8000 0x4000>; - clocks = <&clks 196>; - #address-cells = <2>; - #size-cells = <1>; - ranges = <0 0 0x08000000 0x08000000>; - fsl,weim-cs-gpr = <&gpr>; - - nor@0,0 { - compatible = "cfi-flash"; - reg = <0 0 0x02000000>; - #address-cells = <1>; - #size-cells = <1>; - bank-width = <2>; - fsl,weim-cs-timing = <0x00620081 0x00000001 0x1c022000 - 0x0000c000 0x1404a38e 0x00000000>; - }; - }; - -Example for an imx6q-based board, a multi-chipselect device connected to WEIM: - -In this case, both chip select 0 and 1 will be configured with the same timing -array values. - - weim: weim@21b8000 { - compatible = "fsl,imx6q-weim"; - reg = <0x021b8000 0x4000>; - clocks = <&clks 196>; - #address-cells = <2>; - #size-cells = <1>; - ranges = <0 0 0x08000000 0x02000000 - 1 0 0x0a000000 0x02000000 - 2 0 0x0c000000 0x02000000 - 3 0 0x0e000000 0x02000000>; - fsl,weim-cs-gpr = <&gpr>; - - acme@0 { - compatible = "acme,whatever"; - reg = <0 0 0x100>, <0 0x400000 0x800>, - <1 0x400000 0x800>; - fsl,weim-cs-timing = <0x024400b1 0x00001010 0x20081100 - 0x00000000 0xa0000240 0x00000000>; - }; - };
Convert the i.MX Wireless External Interface Module binding to YAML. Signed-off-by: Sebastian Reichel <sre@kernel.org> --- .../devicetree/bindings/bus/fsl,imx-weim.yaml | 225 ++++++++++++++++++ .../devicetree/bindings/bus/imx-weim.txt | 117 --------- 2 files changed, 225 insertions(+), 117 deletions(-) create mode 100644 Documentation/devicetree/bindings/bus/fsl,imx-weim.yaml delete mode 100644 Documentation/devicetree/bindings/bus/imx-weim.txt