Re: [PATCH] dt-bindings: Drop type references on common properties

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 3/16/21 2:48 PM, Rob Herring wrote:
Users of common properties shouldn't have a type definition as the
common schemas already have one. Drop all the unnecessary type
references in the tree.

A meta-schema update to catch these is pending.

For net/qcom,ipa.yaml:

Acked-by: Alex Elder <elder@xxxxxxxxxx>


Cc: Nicolas Saenz Julienne <nsaenzjulienne@xxxxxxx>
Cc: Maxime Ripard <mripard@xxxxxxxxxx>
Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
Cc: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx>
Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
Cc: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
Cc: Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx>
Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
Cc: Jakub Kicinski <kuba@xxxxxxxxxx>
Cc: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>
Cc: Ohad Ben-Cohen <ohad@xxxxxxxxxx>
Cc: Mark Brown <broonie@xxxxxxxxxx>
Cc: Cheng-Yi Chiang <cychiang@xxxxxxxxxxxx>
Cc: Benson Leung <bleung@xxxxxxxxxxxx>
Cc: Zhang Rui <rui.zhang@xxxxxxxxx>
Cc: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
Cc: Stefan Wahren <wahrenst@xxxxxxx>
Cc: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
Cc: Odelu Kukatla <okukatla@xxxxxxxxxxxxxx>
Cc: Alex Elder <elder@xxxxxxxxxx>
Cc: Suman Anna <s-anna@xxxxxx>
Cc: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
Cc: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
Cc: linux-gpio@xxxxxxxxxxxxxxx
Cc: linux-pm@xxxxxxxxxxxxxxx
Cc: linux-can@xxxxxxxxxxxxxxx
Cc: netdev@xxxxxxxxxxxxxxx
Cc: linux-remoteproc@xxxxxxxxxxxxxxx
Cc: alsa-devel@xxxxxxxxxxxxxxxx
Cc: linux-usb@xxxxxxxxxxxxxxx
Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
---
  .../bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml       | 5 +----
  Documentation/devicetree/bindings/arm/cpus.yaml              | 2 --
  .../bindings/display/allwinner,sun4i-a10-tcon.yaml           | 1 -
  .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml    | 3 +--
  .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml      | 1 -
  .../devicetree/bindings/interconnect/qcom,rpmh.yaml          | 1 -
  .../bindings/memory-controllers/nvidia,tegra210-emc.yaml     | 2 +-
  Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml   | 1 -
  Documentation/devicetree/bindings/net/qcom,ipa.yaml          | 1 -
  Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml  | 2 --
  .../devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml   | 2 +-
  Documentation/devicetree/bindings/sound/ak4642.yaml          | 2 --
  .../devicetree/bindings/sound/google,cros-ec-codec.yaml      | 2 +-
  Documentation/devicetree/bindings/sound/renesas,rsnd.yaml    | 1 -
  .../devicetree/bindings/thermal/qcom-spmi-adc-tm5.yaml       | 1 -
  Documentation/devicetree/bindings/usb/usb.yaml               | 1 -
  16 files changed, 5 insertions(+), 23 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml
index a2c63c8b1d10..c6144c8421fa 100644
--- a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml
+++ b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml
@@ -26,10 +26,7 @@ properties:
        - const: simple-mfd
mboxes:
-    $ref: '/schemas/types.yaml#/definitions/phandle'
-    description: |
-      Phandle to the firmware device's Mailbox.
-      (See: ../mailbox/mailbox.txt for more information)
+    maxItems: 1
clocks:
      type: object
diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
index 26b886b20b27..6be4a8852ee5 100644
--- a/Documentation/devicetree/bindings/arm/cpus.yaml
+++ b/Documentation/devicetree/bindings/arm/cpus.yaml
@@ -256,13 +256,11 @@ properties:
        where voltage is in V, frequency is in MHz.
power-domains:
-    $ref: '/schemas/types.yaml#/definitions/phandle-array'
      description:
        List of phandles and PM domain specifiers, as defined by bindings of the
        PM domain provider (see also ../power_domain.txt).
power-domain-names:
-    $ref: '/schemas/types.yaml#/definitions/string-array'
      description:
        A list of power domain name strings sorted in the same order as the
        power-domains property.
diff --git a/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-tcon.yaml b/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-tcon.yaml
index c13faf3e6581..3a7d5d731712 100644
--- a/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-tcon.yaml
+++ b/Documentation/devicetree/bindings/display/allwinner,sun4i-a10-tcon.yaml
@@ -73,7 +73,6 @@ properties:
    clock-output-names:
      description:
        Name of the LCD pixel clock created.
-    $ref: /schemas/types.yaml#/definitions/string-array
      maxItems: 1
dmas:
diff --git a/Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml b/Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml
index 1a54db04f29d..bcafa494ed7a 100644
--- a/Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml
@@ -43,8 +43,7 @@ properties:
gpio-ranges: true - gpio-ranges-group-names:
-    $ref: /schemas/types.yaml#/definitions/string-array
+  gpio-ranges-group-names: true
socionext,interrupt-ranges:
      description: |
diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml
index 6f2398cdc82d..1e7894e524f9 100644
--- a/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml
@@ -102,7 +102,6 @@ patternProperties:
st,adc-channel-names:
          description: List of single-ended channel names.
-        $ref: /schemas/types.yaml#/definitions/string-array
st,filter-order:
          description: |
diff --git a/Documentation/devicetree/bindings/interconnect/qcom,rpmh.yaml b/Documentation/devicetree/bindings/interconnect/qcom,rpmh.yaml
index 799e73cdb90b..13da7b29c707 100644
--- a/Documentation/devicetree/bindings/interconnect/qcom,rpmh.yaml
+++ b/Documentation/devicetree/bindings/interconnect/qcom,rpmh.yaml
@@ -82,7 +82,6 @@ properties:
        this interconnect to send RPMh commands.
qcom,bcm-voter-names:
-    $ref: /schemas/types.yaml#/definitions/string-array
      description: |
        Names for each of the qcom,bcm-voters specified.
diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra210-emc.yaml b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra210-emc.yaml
index 49ab09252e52..bc8477e7ab19 100644
--- a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra210-emc.yaml
+++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra210-emc.yaml
@@ -34,7 +34,7 @@ properties:
        - description: EMC general interrupt
memory-region:
-    $ref: /schemas/types.yaml#/definitions/phandle
+    maxItems: 1
      description:
        phandle to a reserved memory region describing the table of EMC
        frequencies trained by the firmware
diff --git a/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml b/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml
index fe6a949a2eab..55bff1586b6f 100644
--- a/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml
+++ b/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml
@@ -57,7 +57,6 @@ properties:
        - const: per
clock-frequency:
-    $ref: /schemas/types.yaml#/definitions/uint32
      description: |
        The oscillator frequency driving the flexcan device, filled in by the
        boot loader. This property should only be used the used operating system
diff --git a/Documentation/devicetree/bindings/net/qcom,ipa.yaml b/Documentation/devicetree/bindings/net/qcom,ipa.yaml
index 8f86084bf12e..4e8dee4aa90d 100644
--- a/Documentation/devicetree/bindings/net/qcom,ipa.yaml
+++ b/Documentation/devicetree/bindings/net/qcom,ipa.yaml
@@ -100,7 +100,6 @@ properties:
        - description: Whether the IPA clock is enabled (if valid)
qcom,smem-state-names:
-    $ref: /schemas/types.yaml#/definitions/string-array
      description: The names of the state bits used for SMP2P output
      items:
        - const: ipa-clock-enabled-valid
diff --git a/Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml b/Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml
index d5d7f113bade..828e4a1ece41 100644
--- a/Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml
+++ b/Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml
@@ -23,12 +23,10 @@ properties:
        List of phandle to the nvmem data cells.
nvmem-names:
-    $ref: /schemas/types.yaml#/definitions/string-array
      description:
        Names for the each nvmem provider.
nvmem-cell-names:
-    $ref: /schemas/types.yaml#/definitions/string-array
      description:
        Names for each nvmem-cells specified.
diff --git a/Documentation/devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml
index 1a1159097a2a..73400bc6e91d 100644
--- a/Documentation/devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml
@@ -93,7 +93,7 @@ properties:
  # The following are the optional properties:
memory-region:
-    $ref: /schemas/types.yaml#/definitions/phandle
+    maxItems: 1
      description: |
        phandle to the reserved memory node to be associated
        with the remoteproc device. The reserved memory node
diff --git a/Documentation/devicetree/bindings/sound/ak4642.yaml b/Documentation/devicetree/bindings/sound/ak4642.yaml
index 6cd213be2266..1e2caa29790e 100644
--- a/Documentation/devicetree/bindings/sound/ak4642.yaml
+++ b/Documentation/devicetree/bindings/sound/ak4642.yaml
@@ -29,11 +29,9 @@ properties:
clock-frequency:
      description: common clock binding; frequency of MCKO
-    $ref: /schemas/types.yaml#/definitions/uint32
clock-output-names:
      description: common clock name
-    $ref: /schemas/types.yaml#/definitions/string
required:
    - compatible
diff --git a/Documentation/devicetree/bindings/sound/google,cros-ec-codec.yaml b/Documentation/devicetree/bindings/sound/google,cros-ec-codec.yaml
index acfb9db021dc..77adbebed824 100644
--- a/Documentation/devicetree/bindings/sound/google,cros-ec-codec.yaml
+++ b/Documentation/devicetree/bindings/sound/google,cros-ec-codec.yaml
@@ -32,7 +32,7 @@ properties:
            The last one integer is the length of the shared memory.
memory-region:
-    $ref: '/schemas/types.yaml#/definitions/phandle'
+    maxItems: 1
      description: |
        Shared memory region to EC.  A "shared-dma-pool".
        See ../reserved-memory/reserved-memory.txt for details.
diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
index 2e1046513603..e494a0416748 100644
--- a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
+++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
@@ -78,7 +78,6 @@ properties:
clock-frequency:
      description: for audio_clkout0/1/2/3
-    $ref: /schemas/types.yaml#/definitions/uint32-array
clkout-lr-asynchronous:
      description: audio_clkoutn is asynchronizes with lr-clock.
diff --git a/Documentation/devicetree/bindings/thermal/qcom-spmi-adc-tm5.yaml b/Documentation/devicetree/bindings/thermal/qcom-spmi-adc-tm5.yaml
index 95a728f4d333..3ea8c0c1f45f 100644
--- a/Documentation/devicetree/bindings/thermal/qcom-spmi-adc-tm5.yaml
+++ b/Documentation/devicetree/bindings/thermal/qcom-spmi-adc-tm5.yaml
@@ -59,7 +59,6 @@ patternProperties:
properties:
        reg:
-        $ref: /schemas/types.yaml#/definitions/uint32
          description: Specify the sensor channel. There are 8 channels in PMIC5's ADC TM
          minimum: 0
          maximum: 7
diff --git a/Documentation/devicetree/bindings/usb/usb.yaml b/Documentation/devicetree/bindings/usb/usb.yaml
index 78491e66ed24..939f217b8c7b 100644
--- a/Documentation/devicetree/bindings/usb/usb.yaml
+++ b/Documentation/devicetree/bindings/usb/usb.yaml
@@ -16,7 +16,6 @@ properties:
      pattern: "^usb(@.*)?"
phys:
-    $ref: /schemas/types.yaml#/definitions/phandle-array
      description:
        List of all the USB PHYs on this HCD




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux