dts/bindings: use const to validate #<FOO>-cells

In most cases #<FOO>-cells should be a constant. For example in spi controller #address-cells should be 1, and #size-cells should be 0. Use the const attribute to specify such single known values. Add const value to missing bindings which have cells. Signed-off-by: Kumar Gala <kumar.gala@linaro.org>
2019-08-09 14:54:26 -05:00 · 2019-08-09 14:54:26 -05:00 · b012034519
commit b012034519
parent 3d14374c5e
55 changed files with 144 additions and 18 deletions
--- a/dts/bindings/arm/nxp,kinetis-mcg.yaml
+++ b/dts/bindings/arm/nxp,kinetis-mcg.yaml
@ -22,5 +22,8 @@ properties:
    label:
      category: required

+    "#clock-cells":
+      const: 1
+
 "#cells":
  - name
--- a/dts/bindings/arm/nxp,kinetis-pcc.yaml
+++ b/dts/bindings/arm/nxp,kinetis-pcc.yaml
@ -23,7 +23,7 @@ properties:
      category: required

    "#clock-cells":
-      description: should be 1.
+      const: 1

 "#cells":
  - name
--- a/dts/bindings/arm/nxp,kinetis-scg.yaml
+++ b/dts/bindings/arm/nxp,kinetis-scg.yaml
@ -123,7 +123,7 @@ properties:
      category: optional

    "#clock-cells":
-      description: should be 1.
+      const: 1

 "#cells":
  - name
--- a/dts/bindings/arm/nxp,kinetis-sim.yaml
+++ b/dts/bindings/arm/nxp,kinetis-sim.yaml
@ -35,7 +35,7 @@ properties:
    "#clock-cells":
      type: int
      category: optional
-      description: should be 3.
+      const: 3

 "#cells":
  - name
--- a/dts/bindings/can/can.yaml
+++ b/dts/bindings/can/can.yaml
@ -13,11 +13,11 @@ properties:
    "#address-cells":
      type: int
      category: required
-      description: should be 1.
+      const: 1
    "#size-cells":
      type: int
      category: required
-      description: should be 0.
+      const: 0
    label:
      category: required
    bus-speed:
--- a/dts/bindings/clock/fixed-clock.yaml
+++ b/dts/bindings/clock/fixed-clock.yaml
@ -35,4 +35,4 @@ properties:
      description: input clock source

    "#clock-cells":
-      description: should be 0.
+      const: 0
--- a/dts/bindings/clock/nxp,imx-ccm.yaml
+++ b/dts/bindings/clock/nxp,imx-ccm.yaml
@ -23,7 +23,7 @@ properties:
      category: required

    "#clock-cells":
-      description: should be 3.
+      const: 3

 "#cells":
  - name
--- a/dts/bindings/clock/st,stm32-rcc.yaml
+++ b/dts/bindings/clock/st,stm32-rcc.yaml
@ -14,7 +14,7 @@ properties:
      category: required

    "#clock-cells":
-      description: should be 2.
+      const: 2

 "#cells":
  - bus
--- a/dts/bindings/gpio/arm,cmsdk-gpio.yaml
+++ b/dts/bindings/gpio/arm,cmsdk-gpio.yaml
@ -16,6 +16,9 @@ properties:
    interrupts:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/atmel,sam-gpio.yaml
+++ b/dts/bindings/gpio/atmel,sam-gpio.yaml
@ -24,6 +24,9 @@ properties:
      description: peripheral ID
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/atmel,sam0-gpio.yaml
+++ b/dts/bindings/gpio/atmel,sam0-gpio.yaml
@ -16,6 +16,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/intel,apl-gpio.yaml
+++ b/dts/bindings/gpio/intel,apl-gpio.yaml
@ -25,6 +25,9 @@ properties:
    interrupts:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/intel,qmsi-gpio.yaml
+++ b/dts/bindings/gpio/intel,qmsi-gpio.yaml
@ -25,6 +25,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/intel,qmsi-ss-gpio.yaml
+++ b/dts/bindings/gpio/intel,qmsi-ss-gpio.yaml
@ -25,6 +25,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/microchip,xec-gpio.yaml
+++ b/dts/bindings/gpio/microchip,xec-gpio.yaml
@ -26,6 +26,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/nordic,nrf-gpio.yaml
+++ b/dts/bindings/gpio/nordic,nrf-gpio.yaml
@ -22,6 +22,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/nxp,imx-gpio.yaml
+++ b/dts/bindings/gpio/nxp,imx-gpio.yaml
@ -30,6 +30,9 @@ properties:
     category: optional
     description: Set the RDC permission for this peripheral

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/nxp,kinetis-gpio.yaml
+++ b/dts/bindings/gpio/nxp,kinetis-gpio.yaml
@ -16,6 +16,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/openisa,rv32m1-gpio.yaml
+++ b/dts/bindings/gpio/openisa,rv32m1-gpio.yaml
@ -19,6 +19,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/semtech,sx1509b-gpio.yaml
+++ b/dts/bindings/gpio/semtech,sx1509b-gpio.yaml
@ -19,6 +19,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/sifive,gpio0.yaml
+++ b/dts/bindings/gpio/sifive,gpio0.yaml
@ -25,6 +25,9 @@ properties:
    interrupts:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/silabs,efm32-gpio-port.yaml
+++ b/dts/bindings/gpio/silabs,efm32-gpio-port.yaml
@ -16,6 +16,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/silabs,efr32mg12-gpio-port.yaml
+++ b/dts/bindings/gpio/silabs,efr32mg12-gpio-port.yaml
@ -16,6 +16,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/silabs,efr32xg1-gpio-port.yaml
+++ b/dts/bindings/gpio/silabs,efr32xg1-gpio-port.yaml
@ -16,6 +16,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/snps,designware-gpio.yaml
+++ b/dts/bindings/gpio/snps,designware-gpio.yaml
@ -30,6 +30,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/st,stm32-gpio.yaml
+++ b/dts/bindings/gpio/st,stm32-gpio.yaml
@ -25,6 +25,9 @@ properties:
    clocks:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/ti,cc13xx-cc26xx-gpio.yaml
+++ b/dts/bindings/gpio/ti,cc13xx-cc26xx-gpio.yaml
@ -25,6 +25,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/ti,cc32xx-gpio.yaml
+++ b/dts/bindings/gpio/ti,cc32xx-gpio.yaml
@ -20,6 +20,9 @@ properties:
    label:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/gpio/ti,stellaris-gpio.yaml
+++ b/dts/bindings/gpio/ti,stellaris-gpio.yaml
@ -20,6 +20,9 @@ properties:
    interrupts:
      category: required

+    "#gpio-cells":
+      const: 2
+
 "#cells":
  - pin
  - flags
--- a/dts/bindings/i2c/i2c.yaml
+++ b/dts/bindings/i2c/i2c.yaml
@ -19,11 +19,11 @@ properties:
    "#address-cells":
      type: int
      category: required
-      description: should be 1.
+      const: 1
    "#size-cells":
      type: int
      category: required
-      description: should be 0.
+      const: 0
    clock-frequency :
      type: int
      category: optional
--- a/dts/bindings/i2s/i2s.yaml
+++ b/dts/bindings/i2s/i2s.yaml
@ -19,10 +19,10 @@ properties:
    "#address-cells":
      type: int
      category: required
-      description: should be 1.
+      const: 1
    "#size-cells":
      type: int
      category: required
-      description: should be 0.
+      const: 0
    label:
      category: required
--- a/dts/bindings/interrupt-controller/arm,v6m-nvic.yaml
+++ b/dts/bindings/interrupt-controller/arm,v6m-nvic.yaml
@ -18,6 +18,9 @@ properties:
      type: int
      description: number of bits of IRQ priorities

+    "#interrupt-cells":
+      const: 2
+
 "#cells":
  - irq
  - priority
--- a/dts/bindings/interrupt-controller/arm,v7m-nvic.yaml
+++ b/dts/bindings/interrupt-controller/arm,v7m-nvic.yaml
@ -18,6 +18,9 @@ properties:
      type: int
      description: number of bits of IRQ priorities

+    "#interrupt-cells":
+      const: 2
+
 "#cells":
  - irq
  - priority
--- a/dts/bindings/interrupt-controller/arm,v8m-nvic.yaml
+++ b/dts/bindings/interrupt-controller/arm,v8m-nvic.yaml
@ -18,6 +18,9 @@ properties:
      type: int
      description: number of bits of IRQ priorities

+    "#interrupt-cells":
+      const: 2
+
 "#cells":
  - irq
  - priority
--- a/dts/bindings/interrupt-controller/intel,cavs-intc.yaml
+++ b/dts/bindings/interrupt-controller/intel,cavs-intc.yaml
@ -16,6 +16,9 @@ properties:
  interrupts:
      category: required

+  "#interrupt-cells":
+      const: 3
+
 "#cells":
  - irq
  - sense
--- a/dts/bindings/interrupt-controller/intel,ioapic.yaml
+++ b/dts/bindings/interrupt-controller/intel,ioapic.yaml
@ -14,6 +14,9 @@ properties:
  reg:
      category: required

+  "#interrupt-cells":
+      const: 3
+
 "#cells":
  - irq
  - sense
--- a/dts/bindings/interrupt-controller/openisa,rv32m1-event-unit.yaml
+++ b/dts/bindings/interrupt-controller/openisa,rv32m1-event-unit.yaml
@ -20,5 +20,8 @@ properties:
  reg:
      category: required

+  "#interrupt-cells":
+      const: 1
+
 "#cells":
  - irq
--- a/dts/bindings/interrupt-controller/openisa,rv32m1-intmux.yaml
+++ b/dts/bindings/interrupt-controller/openisa,rv32m1-intmux.yaml
@ -25,5 +25,8 @@ properties:
  interrupts:
      category: required

+  "#interrupt-cells":
+      const: 1
+
 "#cells":
  - irq
--- a/dts/bindings/interrupt-controller/riscv,cpu-intc.yaml
+++ b/dts/bindings/interrupt-controller/riscv,cpu-intc.yaml
@ -16,5 +16,8 @@ properties:
  compatible:
      constraint: "riscv,cpu-intc"

+  "#interrupt-cells":
+      const: 1
+
 "#cells":
  - irq
--- a/dts/bindings/interrupt-controller/riscv,plic0.yaml
+++ b/dts/bindings/interrupt-controller/riscv,plic0.yaml
@ -21,5 +21,8 @@ properties:
      description: maximum interrupt priority
      category: required

+  "#interrupt-cells":
+      const: 1
+
 "#cells":
  - irq
--- a/dts/bindings/interrupt-controller/snps,arcv2-intc.yaml
+++ b/dts/bindings/interrupt-controller/snps,arcv2-intc.yaml
@ -16,6 +16,9 @@ properties:
    compatible:
      constraint: "snps,arcv2-intc"

+    "#interrupt-cells":
+      const: 2
+
 "#cells":
  - irq
  - priority
--- a/dts/bindings/interrupt-controller/snps,designware-intc.yaml
+++ b/dts/bindings/interrupt-controller/snps,designware-intc.yaml
@ -16,6 +16,9 @@ properties:
  interrupts:
      category: required

+  "#interrupt-cells":
+      const: 3
+
 "#cells":
  - irq
  - sense
--- a/dts/bindings/interrupt-controller/vexriscv,intc0.yaml
+++ b/dts/bindings/interrupt-controller/vexriscv,intc0.yaml
@ -24,6 +24,9 @@ properties:
      description: maximum interrupt priority
      category: required

+  "#interrupt-cells":
+      const: 2
+
 "#cells":
  - irq
  - priority
--- a/dts/bindings/interrupt-controller/xtensa,intc.yaml
+++ b/dts/bindings/interrupt-controller/xtensa,intc.yaml
@ -13,6 +13,9 @@ properties:
  reg:
      category: required

+  "#interrupt-cells":
+      const: 3
+
 "#cells":
  - irq
  - sense
--- a/dts/bindings/led/holtek,ht16k33.yaml
+++ b/dts/bindings/led/holtek,ht16k33.yaml
@ -14,11 +14,11 @@ properties:
    "#address-cells":
      type: int
      category: required
-      description: should be 1.
+      const: 1
    "#size-cells":
      type: int
      category: required
-      description: should be 0.
+      const: 0
    label:
      category: required
    irq-gpios:
--- a/dts/bindings/phy/st,stm32-usbphyc.yaml
+++ b/dts/bindings/phy/st,stm32-usbphyc.yaml
@ -20,4 +20,4 @@ properties:
      category: required

    "#phy-cells":
-      description: should be 0
+      const: 0
--- a/dts/bindings/phy/usb-nop-xceiv.yaml
+++ b/dts/bindings/phy/usb-nop-xceiv.yaml
@ -18,4 +18,4 @@ properties:
      constraint: "usb-nop-xceiv"

    "#phy-cells":
-      description: should be 0
+      const: 0
--- a/dts/bindings/pwm/atmel,sam-pwm.yaml
+++ b/dts/bindings/pwm/atmel,sam-pwm.yaml
@ -37,6 +37,9 @@ properties:
      category: required
      description: Clock divider at the input of the PWM (1 to 255)

+    "#pwm-cells":
+      const: 2
+
 "#cells":
  - channel
 # period in terms of nanoseconds
--- a/dts/bindings/pwm/fsl,imx7d-pwm.yaml
+++ b/dts/bindings/pwm/fsl,imx7d-pwm.yaml
@ -32,6 +32,9 @@ properties:
     category: required
     description: Set the RDC permission for this peripheral

+    "#pwm-cells":
+     const: 2
+
 "#cells":
  - channel
 # period in terms of nanoseconds
--- a/dts/bindings/pwm/nxp,imx-pwm.yaml
+++ b/dts/bindings/pwm/nxp,imx-pwm.yaml
@ -27,6 +27,9 @@ properties:
    label:
      category: required

+    "#pwm-cells":
+      const: 1
+
 "#cells":
  - channel
 ...
--- a/dts/bindings/pwm/nxp,kinetis-ftm.yaml
+++ b/dts/bindings/pwm/nxp,kinetis-ftm.yaml
@ -22,6 +22,9 @@ properties:
    interrupts:
      category: required

+    "#pwm-cells":
+      const: 2
+
 "#cells":
  - channel
 # period in terms of nanoseconds
--- a/dts/bindings/pwm/sifive,pwm0.yaml
+++ b/dts/bindings/pwm/sifive,pwm0.yaml
@ -32,6 +32,9 @@ properties:
      category: required
      description: Width of the PWM comparator in bits

+    "#pwm-cells":
+      const: 2
+
 "#cells":
  - channel
 # period in terms of nanoseconds
--- a/dts/bindings/pwm/st,stm32-pwm.yaml
+++ b/dts/bindings/pwm/st,stm32-pwm.yaml
@ -18,6 +18,9 @@ properties:
      category: required
      description: Clock prescaler at the input of the timer

+    "#pwm-cells":
+      const: 2
+
 "#cells":
  - channel
 # period in terms of nanoseconds
--- a/dts/bindings/riscv/openisa,rv32m1-pcc.yaml
+++ b/dts/bindings/riscv/openisa,rv32m1-pcc.yaml
@ -22,5 +22,8 @@ properties:
    label:
      category: required

+    "#clock-cells":
+      const: 1
+
 "#cells":
  - name
--- a/dts/bindings/spi/spi.yaml
+++ b/dts/bindings/spi/spi.yaml
@ -23,11 +23,11 @@ properties:
    "#address-cells":
      type: int
      category: required
-      description: should be 1.
+      const: 1
    "#size-cells":
      type: int
      category: required
-      description: should be 0.
+      const: 0
    label:
      category: required
    cs-gpios: