Re: [PATCH v2 2/6] dt-bindings: memory-controllers: ti,gpmc: Convert to yaml

From: Rob Herring
Date: Thu Sep 02 2021 - 15:57:10 EST


On Thu, Sep 02, 2021 at 12:56:05PM +0300, Roger Quadros wrote:
> Convert omap-gpmc.txt to ti,gpmc.yaml.
>
> Signed-off-by: Roger Quadros <rogerq@xxxxxxxxxx>
> ---
> .../bindings/memory-controllers/omap-gpmc.txt | 157 --------
> .../bindings/memory-controllers/ti,gpmc.yaml | 364 ++++++++++++++++++
> .../devicetree/bindings/mtd/gpmc-nand.txt | 2 +-
> .../devicetree/bindings/mtd/gpmc-nor.txt | 4 +-
> .../devicetree/bindings/mtd/gpmc-onenand.txt | 2 +-
> .../devicetree/bindings/net/gpmc-eth.txt | 4 +-
> 6 files changed, 370 insertions(+), 163 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/memory-controllers/omap-gpmc.txt
> create mode 100644 Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml

> diff --git a/Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml b/Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml
> new file mode 100644
> index 000000000000..b7d43370a95d
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml
> @@ -0,0 +1,364 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/memory-controllers/ti,gpmc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Texas Instruments GPMC Memory Controller device-tree bindings
> +
> +maintainers:
> + - Tony Lindgren <tony@xxxxxxxxxxx>
> + - Roger Quadros <rogerq@xxxxxxxxxx>
> +
> +description:
> + The GPMC is a unified memory controller dedicated for interfacing
> + with external memory devices like
> + - Asynchronous SRAM-like memories and ASICs
> + - Asynchronous, synchronous, and page mode burst NOR flash
> + - NAND flash
> + - Pseudo-SRAM devices
> +
> +properties:
> + compatible:
> + items:
> + - enum:
> + - ti,omap2420-gpmc
> + - ti,omap2430-gpmc
> + - ti,omap3430-gpmc
> + - ti,omap4430-gpmc
> + - ti,am3352-gpmc
> +
> + reg:
> + items:
> + - description:
> + Configuration registers for the controller.

Just 'maxItems: 1' is sufficient.

> +
> + interrupts: true

Need to define how many.

> +
> + clocks:
> + maxItems: 1
> + description: |
> + Functional clock. Used for bus timing calculations and
> + GPMC configuration.
> +
> + clock-names:
> + items:
> + - const: fck
> +
> + dmas:
> + items:
> + - description: DMA channel for GPMC NAND prefetch
> +
> + dma-names:
> + items:
> + - const: rxtx
> +
> + "#address-cells":
> + const: 2
> +
> + "#size-cells":
> + const: 1
> +
> + gpmc,num-cs:
> + description: maximum number of supported chip-select lines.
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> + gpmc,num-waitpins:
> + description: maximum number of supported wait pins.
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> + ranges:
> + minItems: 1
> + description: |
> + 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>
> +
> + items:
> + - description: NAND bank 0
> + - description: NOR/SRAM bank 0
> + - description: NOR/SRAM bank 1
> +
> + '#interrupt-cells':
> + const: 2
> +
> + interrupt-controller:
> + description: |
> + The GPMC driver implements and interrupt controller for
> + the NAND events "fifoevent" and "termcount" plus the
> + rising/falling edges on the GPMC_WAIT pins.
> + The interrupt number mapping is as follows
> + 0 - NAND_fifoevent
> + 1 - NAND_termcount
> + 2 - GPMC_WAIT0 pin edge
> + 3 - GPMC_WAIT1 pin edge, and so on.
> +
> + '#gpio-cells':
> + const: 2
> +
> + gpio-controller:
> + description: |
> + The GPMC driver implements a GPIO controller for the
> + GPMC WAIT pins that can be used as general purpose inputs.
> + 0 maps to GPMC_WAIT0 pin.
> +
> + ti,hwmods:
> + description:
> + Name of the HWMOD associated with GPMC. This is for legacy
> + omap2/3 platforms only.
> + $ref: /schemas/types.yaml#/definitions/string
> + deprecated: true
> +
> + ti,no-idle-on-init:
> + description:
> + Prevent idling the module at init. This is for legacy omap2/3
> + platforms only.
> + type: boolean
> + deprecated: true
> +
> +patternProperties:
> +# "@[0-3],[a-f0-9]+$":
> + "^[a-zA-Z][a-zA-Z0-9,+\\-._]{0,63}@[0-9a-fA-F]+(,[0-9a-fA-F]+)*$":

Why the commented regex. There's no need for a full regex as we already
do that elsewhere. You only need to define the unit-address format.

> + type: object
> + description: |
> + The child device node represents the device connected to the GPMC
> + bus. The device can be a NAND controller, SRAM device, NOR device
> + or an ASIC.
> +
> + properties:
> + compatible:
> + description:
> + Compatible of attached device.

Duh. Drop.

> +
> + reg:
> + items:
> + - description: Register access space for the device

A device with 2 register ranges isn't allowed?

> +
> +# GPMC Timing properties for child nodes. All are optional and default to 0.
> +
> + gpmc,sync-clk-ps:
> + description: Minimum clock period for synchronous mode, in picoseconds
> + $ref: /schemas/types.yaml#/definitions/uint32

default: 0

And elsewhere...

> +
> +# Chip-select signal timings (in nanoseconds) corresponding to GPMC_CONFIG2:
> + gpmc,cs-on-ns:
> + description: Assertion time
> + $ref: /schemas/types.yaml#/definitions/uint32

Blank line between DT properties.

Don't need a type ref with a standard unit suffix.

Is there a maximum value for all these?

> + gpmc,cs-rd-off-ns:
> + description: Read deassertion time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,cs-wr-off-ns:
> + description: Write deassertion time
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> +# ADV signal timings (in nanoseconds) corresponding to GPMC_CONFIG3:
> + gpmc,adv-on-ns:
> + description: Assertion time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,adv-rd-off-ns:
> + description: Read deassertion time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,adv-wr-off-ns:
> + description: Write deassertion time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,adv-aad-mux-on-ns:
> + description: Assertion time for AAD
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,adv-aad-mux-rd-off-ns:
> + description: Read deassertion time for AAD
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,adv-aad-mux-wr-off-ns:
> + description: Write deassertion time for AAD
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> +# WE signals timings (in nanoseconds) corresponding to GPMC_CONFIG4:
> + gpmc,we-on-ns:
> + description: Assertion time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,we-off-ns:
> + description: Deassertion time
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> +# OE signals timings (in nanoseconds) corresponding to GPMC_CONFIG4:
> + gpmc,oe-on-ns:
> + description: Assertion time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,oe-off-ns:
> + description: Deassertion time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,oe-aad-mux-on-ns:
> + description: Assertion time for AAD
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,oe-aad-mux-off-ns:
> + description: Deassertion time for AAD
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> +# Access time and cycle time timings (in nanoseconds) corresponding to
> +# GPMC_CONFIG5:
> + gpmc,page-burst-access-ns:
> + description: Multiple access word delay
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,access-ns:
> + description: Start-cycle to first data valid delay
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,rd-cycle-ns:
> + description: Total read cycle time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,wr-cycle-ns:
> + description: Total write cycle time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,bus-turnaround-ns:
> + description: Turn-around time between successive accesses
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,cycle2cycle-delay-ns:
> + description: Delay between chip-select pulses
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,clk-activation-ns:
> + description: GPMC clock activation time
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,wait-monitoring-ns:
> + description: Start of wait monitoring with regard to valid data
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> +# Boolean timing parameters. If property is present, parameter is enabled
> +# otherwise disabled.
> + gpmc,adv-extra-delay:
> + description: ADV signal is delayed by half GPMC clock
> + type: boolean
> + gpmc,cs-extra-delay:
> + description: CS signal is delayed by half GPMC clock
> + type: boolean
> + gpmc,cycle2cycle-diffcsen:
> + description: |
> + Add "cycle2cycle-delay" between successive accesses
> + to a different CS
> + type: boolean
> + gpmc,cycle2cycle-samecsen:
> + description: |
> + Add "cycle2cycle-delay" between successive accesses
> + to the same CS
> + type: boolean
> + gpmc,oe-extra-delay:
> + description: OE signal is delayed by half GPMC clock
> + type: boolean
> + gpmc,we-extra-delay:
> + description: WE signal is delayed by half GPMC clock
> + type: boolean
> + gpmc,time-para-granularity:
> + description: Multiply all access times by 2
> + type: boolean
> +
> +# The following two properties are applicable only to OMAP3+ and AM335x:
> + gpmc,wr-access-ns:
> + description: |
> + In synchronous write mode, for single or
> + burst accesses, defines the number of
> + GPMC_FCLK cycles from start access time
> + to the GPMC_CLK rising edge used by the
> + memory device for the first data capture.
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,wr-data-mux-bus-ns:
> + description: |
> + In address-data multiplex mode, specifies
> + the time when the first data is driven on
> + the address-data bus.
> + $ref: /schemas/types.yaml#/definitions/uint32
> +
> +# GPMC chip-select settings properties for child nodes. All are optional.
> +
> + gpmc,burst-length:
> + description: Page/burst length. Must be 4, 8 or 16.
> + $ref: /schemas/types.yaml#/definitions/uint32
> + enum: [4, 8, 16]
> + gpmc,burst-wrap:
> + description: Enables wrap bursting
> + type: boolean
> + gpmc,burst-read:
> + description: Enables read page/burst mode
> + type: boolean
> + gpmc,burst-write:
> + description: Enables write page/burst mode
> + type: boolean
> + gpmc,device-width:
> + description: |
> + Total width of device(s) connected to a GPMC
> + chip-select in bytes. The GPMC supports 8-bit
> + and 16-bit devices and so this property must be
> + 1 or 2.
> + $ref: /schemas/types.yaml#/definitions/uint32
> + enum: [1, 2]
> + gpmc,mux-add-data:
> + description: |
> + Address and data multiplexing configuration.
> + Valid values are:
> + 0 for Non multiplexed mode
> + 1 for address-address-data multiplexing mode and
> + 2 for address-data multiplexing mode.
> + $ref: /schemas/types.yaml#/definitions/uint32
> + enum: [0, 1, 2]
> + gpmc,sync-read:
> + description: |
> + Enables synchronous read. Defaults to asynchronous
> + is this is not set.
> + type: boolean
> + gpmc,sync-write:
> + description: |
> + Enables synchronous writes. Defaults to asynchronous
> + is this is not set.
> + type: boolean
> + gpmc,wait-pin:
> + description: |
> + Wait-pin used by client. Must be less than "gpmc,num-waitpins".
> + $ref: /schemas/types.yaml#/definitions/uint32
> + gpmc,wait-on-read:
> + description: Enables wait monitoring on reads.
> + type: boolean
> + gpmc,wait-on-write:
> + description: Enables wait monitoring on writes.
> + type: boolean
> +
> + required:
> + - reg
> +
> +required:
> + - compatible
> + - reg
> + - gpmc,num-cs
> + - gpmc,num-waitpins
> + - "#address-cells"
> + - "#size-cells"
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/interrupt-controller/arm-gic.h>
> + #include <dt-bindings/gpio/gpio.h>
> +
> + gpmc: memory-controller@50000000 {
> + compatible = "ti,am3352-gpmc";
> + reg = <0x50000000 0x2000>;
> + interrupts = <100>;
> + clocks = <&l3s_clkctrl>;
> + clock-names = "fck";
> + dmas = <&edma 52 0>;
> + dma-names = "rxtx";
> + gpmc,num-cs = <8>;
> + gpmc,num-waitpins = <2>;
> + #address-cells = <2>;
> + #size-cells = <1>;
> + ranges = <0 0 0x08000000 0x10000000>; /* CS0 @addr 0x8000000, size 0x10000000 */
> + interrupt-controller;
> + #interrupt-cells = <2>;
> + gpio-controller;
> + #gpio-cells = <2>;
> +
> + nand@0,0 {
> + compatible = "ti,omap2-nand";
> + reg = <0 0 4>;
> + interrupt-parent = <&gpmc>;
> + interrupts = <0 IRQ_TYPE_NONE>, /* fifoevent */
> + <1 IRQ_TYPE_NONE>; /* termcount */
> + rb-gpios = <&gpmc 0 GPIO_ACTIVE_HIGH>; /* gpmc_wait0 pin */
> + };
> + };
> diff --git a/Documentation/devicetree/bindings/mtd/gpmc-nand.txt b/Documentation/devicetree/bindings/mtd/gpmc-nand.txt
> index 44919d48d241..e439949d49e6 100644
> --- a/Documentation/devicetree/bindings/mtd/gpmc-nand.txt
> +++ b/Documentation/devicetree/bindings/mtd/gpmc-nand.txt
> @@ -5,7 +5,7 @@ the GPMC controller with a name of "nand".
>
> All timing relevant properties as well as generic gpmc child properties are
> explained in a separate documents - please refer to
> -Documentation/devicetree/bindings/memory-controllers/omap-gpmc.txt
> +Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml
>
> For NAND specific properties such as ECC modes or bus width, please refer to
> Documentation/devicetree/bindings/mtd/nand-controller.yaml
> diff --git a/Documentation/devicetree/bindings/mtd/gpmc-nor.txt b/Documentation/devicetree/bindings/mtd/gpmc-nor.txt
> index c8567b40fe13..c9bea106ea65 100644
> --- a/Documentation/devicetree/bindings/mtd/gpmc-nor.txt
> +++ b/Documentation/devicetree/bindings/mtd/gpmc-nor.txt
> @@ -5,7 +5,7 @@ child nodes of the GPMC controller with a name of "nor".
>
> All timing relevant properties as well as generic GPMC child properties are
> explained in a separate documents. Please refer to
> -Documentation/devicetree/bindings/memory-controllers/omap-gpmc.txt
> +Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml
>
> Required properties:
> - bank-width: Width of NOR flash in bytes. GPMC supports 8-bit and
> @@ -28,7 +28,7 @@ Required properties:
>
> Optional properties:
> - gpmc,XXX Additional GPMC timings and settings parameters. See
> - Documentation/devicetree/bindings/memory-controllers/omap-gpmc.txt
> + Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml
>
> Optional properties for partition table parsing:
> - #address-cells: should be set to 1
> diff --git a/Documentation/devicetree/bindings/mtd/gpmc-onenand.txt b/Documentation/devicetree/bindings/mtd/gpmc-onenand.txt
> index e9f01a963a0a..0da78cc4ccca 100644
> --- a/Documentation/devicetree/bindings/mtd/gpmc-onenand.txt
> +++ b/Documentation/devicetree/bindings/mtd/gpmc-onenand.txt
> @@ -5,7 +5,7 @@ the GPMC controller with a name of "onenand".
>
> All timing relevant properties as well as generic gpmc child properties are
> explained in a separate documents - please refer to
> -Documentation/devicetree/bindings/memory-controllers/omap-gpmc.txt
> +Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml
>
> Required properties:
>
> diff --git a/Documentation/devicetree/bindings/net/gpmc-eth.txt b/Documentation/devicetree/bindings/net/gpmc-eth.txt
> index 32821066a85b..5e2f610455fa 100644
> --- a/Documentation/devicetree/bindings/net/gpmc-eth.txt
> +++ b/Documentation/devicetree/bindings/net/gpmc-eth.txt
> @@ -9,7 +9,7 @@ the GPMC controller with an "ethernet" name.
>
> All timing relevant properties as well as generic GPMC child properties are
> explained in a separate documents. Please refer to
> -Documentation/devicetree/bindings/memory-controllers/omap-gpmc.txt
> +Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml
>
> For the properties relevant to the ethernet controller connected to the GPMC
> refer to the binding documentation of the device. For example, the documentation
> @@ -43,7 +43,7 @@ Required properties:
>
> Optional properties:
> - gpmc,XXX Additional GPMC timings and settings parameters. See
> - Documentation/devicetree/bindings/memory-controllers/omap-gpmc.txt
> + Documentation/devicetree/bindings/memory-controllers/ti,gpmc.yaml

Perhaps delete this file first in the series so you don't have to
update it.

Rob