[v1,02/14] dt-bindings: bus: imx-weim: convert to YAML
Commit Message
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
Comments
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
new file mode 100644
@@ -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>;
+ };
+ };
deleted file mode 100644
@@ -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>;
- };
- };