LinuxLists.cc - [PATCH] dt-bindings: Drop type references on common properties

2021-03-16 21:24:20

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

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.

Cc: Nicolas Saenz Julienne <[email protected]>
Cc: Maxime Ripard <[email protected]>
Cc: Linus Walleij <[email protected]>
Cc: Bartosz Golaszewski <[email protected]>
Cc: Bjorn Andersson <[email protected]>
Cc: Krzysztof Kozlowski <[email protected]>
Cc: Marc Kleine-Budde <[email protected]>
Cc: "David S. Miller" <[email protected]>
Cc: Jakub Kicinski <[email protected]>
Cc: Srinivas Kandagatla <[email protected]>
Cc: Ohad Ben-Cohen <[email protected]>
Cc: Mark Brown <[email protected]>
Cc: Cheng-Yi Chiang <[email protected]>
Cc: Benson Leung <[email protected]>
Cc: Zhang Rui <[email protected]>
Cc: Daniel Lezcano <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
Cc: Stefan Wahren <[email protected]>
Cc: Masahiro Yamada <[email protected]>
Cc: Odelu Kukatla <[email protected]>
Cc: Alex Elder <[email protected]>
Cc: Suman Anna <[email protected]>
Cc: Kuninori Morimoto <[email protected]>
Cc: Dmitry Baryshkov <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Signed-off-by: Rob Herring <[email protected]>
---
.../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

--
2.27.0

2021-03-16 21:25:30

by Alex Elder

[permalink] [raw]

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

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 <[email protected]>

>
> Cc: Nicolas Saenz Julienne <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: Linus Walleij <[email protected]>
> Cc: Bartosz Golaszewski <[email protected]>
> Cc: Bjorn Andersson <[email protected]>
> Cc: Krzysztof Kozlowski <[email protected]>
> Cc: Marc Kleine-Budde <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Jakub Kicinski <[email protected]>
> Cc: Srinivas Kandagatla <[email protected]>
> Cc: Ohad Ben-Cohen <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Cheng-Yi Chiang <[email protected]>
> Cc: Benson Leung <[email protected]>
> Cc: Zhang Rui <[email protected]>
> Cc: Daniel Lezcano <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: Stefan Wahren <[email protected]>
> Cc: Masahiro Yamada <[email protected]>
> Cc: Odelu Kukatla <[email protected]>
> Cc: Alex Elder <[email protected]>
> Cc: Suman Anna <[email protected]>
> Cc: Kuninori Morimoto <[email protected]>
> Cc: Dmitry Baryshkov <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../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
>
>

2021-03-16 21:26:17

by Dmitry Baryshkov

[permalink] [raw]

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

On Tue, 16 Mar 2021 at 22:49, Rob Herring <[email protected]> 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.
>
> Cc: Nicolas Saenz Julienne <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: Linus Walleij <[email protected]>
> Cc: Bartosz Golaszewski <[email protected]>
> Cc: Bjorn Andersson <[email protected]>
> Cc: Krzysztof Kozlowski <[email protected]>
> Cc: Marc Kleine-Budde <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Jakub Kicinski <[email protected]>
> Cc: Srinivas Kandagatla <[email protected]>
> Cc: Ohad Ben-Cohen <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Cheng-Yi Chiang <[email protected]>
> Cc: Benson Leung <[email protected]>
> Cc: Zhang Rui <[email protected]>
> Cc: Daniel Lezcano <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: Stefan Wahren <[email protected]>
> Cc: Masahiro Yamada <[email protected]>
> Cc: Odelu Kukatla <[email protected]>
> Cc: Alex Elder <[email protected]>
> Cc: Suman Anna <[email protected]>
> Cc: Kuninori Morimoto <[email protected]>
> Cc: Dmitry Baryshkov <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../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(-)

For the qcom-spmi-adc-tm5.yaml:
Acked-by: Dmity Baryshkov <[email protected]>

>
> 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
>
> --
> 2.27.0
>

--
With best wishes
Dmitry

2021-03-16 21:27:10

by Mark Brown

[permalink] [raw]

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

Attachments:

(No filename) (270.00 B)
signature.asc (499.00 B)
Download all attachments

2021-03-16 22:55:42

by Suman Anna

[permalink] [raw]

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

2021-03-17 05:56:06

by Greg KH

[permalink] [raw]

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

On Tue, Mar 16, 2021 at 01:48:58PM -0600, 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.
>
> Cc: Nicolas Saenz Julienne <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: Linus Walleij <[email protected]>
> Cc: Bartosz Golaszewski <[email protected]>
> Cc: Bjorn Andersson <[email protected]>
> Cc: Krzysztof Kozlowski <[email protected]>
> Cc: Marc Kleine-Budde <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Jakub Kicinski <[email protected]>
> Cc: Srinivas Kandagatla <[email protected]>
> Cc: Ohad Ben-Cohen <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Cheng-Yi Chiang <[email protected]>
> Cc: Benson Leung <[email protected]>
> Cc: Zhang Rui <[email protected]>
> Cc: Daniel Lezcano <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: Stefan Wahren <[email protected]>
> Cc: Masahiro Yamada <[email protected]>
> Cc: Odelu Kukatla <[email protected]>
> Cc: Alex Elder <[email protected]>
> Cc: Suman Anna <[email protected]>
> Cc: Kuninori Morimoto <[email protected]>
> Cc: Dmitry Baryshkov <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../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(-)

Acked-by: Greg Kroah-Hartman <[email protected]>

2021-03-17 08:34:57

by Marc Kleine-Budde

[permalink] [raw]

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

On 16.03.2021 13:48:58, 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.
>
> Cc: Nicolas Saenz Julienne <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: Linus Walleij <[email protected]>
> Cc: Bartosz Golaszewski <[email protected]>
> Cc: Bjorn Andersson <[email protected]>
> Cc: Krzysztof Kozlowski <[email protected]>
> Cc: Marc Kleine-Budde <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Jakub Kicinski <[email protected]>
> Cc: Srinivas Kandagatla <[email protected]>
> Cc: Ohad Ben-Cohen <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Cheng-Yi Chiang <[email protected]>
> Cc: Benson Leung <[email protected]>
> Cc: Zhang Rui <[email protected]>
> Cc: Daniel Lezcano <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: Stefan Wahren <[email protected]>
> Cc: Masahiro Yamada <[email protected]>
> Cc: Odelu Kukatla <[email protected]>
> Cc: Alex Elder <[email protected]>
> Cc: Suman Anna <[email protected]>
> Cc: Kuninori Morimoto <[email protected]>
> Cc: Dmitry Baryshkov <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../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 -

For the flexcan:

Acked-by: Marc Kleine-Budde <[email protected]>

regards,
Marc

--
Pengutronix e.K. | Marc Kleine-Budde |
Embedded Linux | https://www.pengutronix.de |
Vertretung West/Dortmund | Phone: +49-231-2826-924 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |

Attachments:

(No filename) (2.56 kB)
signature.asc (499.00 B)
Download all attachments

2021-03-17 09:02:00

by Srinivas Kandagatla

[permalink] [raw]

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

On 16/03/2021 19:48, 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.
>
> Cc: Nicolas Saenz Julienne<[email protected]>
> Cc: Maxime Ripard<[email protected]>
> Cc: Linus Walleij<[email protected]>
> Cc: Bartosz Golaszewski<[email protected]>
> Cc: Bjorn Andersson<[email protected]>
> Cc: Krzysztof Kozlowski<[email protected]>
> Cc: Marc Kleine-Budde<[email protected]>
> Cc: "David S. Miller"<[email protected]>
> Cc: Jakub Kicinski<[email protected]>
> Cc: Srinivas Kandagatla<[email protected]>
> Cc: Ohad Ben-Cohen<[email protected]>
> Cc: Mark Brown<[email protected]>
> Cc: Cheng-Yi Chiang<[email protected]>
> Cc: Benson Leung<[email protected]>
> Cc: Zhang Rui<[email protected]>
> Cc: Daniel Lezcano<[email protected]>
> Cc: Greg Kroah-Hartman<[email protected]>
> Cc: Stefan Wahren<[email protected]>
> Cc: Masahiro Yamada<[email protected]>
> Cc: Odelu Kukatla<[email protected]>
> Cc: Alex Elder<[email protected]>
> Cc: Suman Anna<[email protected]>
> Cc: Kuninori Morimoto<[email protected]>
> Cc: Dmitry Baryshkov<[email protected]>
> Cc:[email protected]
> Cc:[email protected]
> Cc:[email protected]
> Cc:[email protected]
> Cc:[email protected]
> Cc:[email protected]
> Cc:[email protected]
> Signed-off-by: Rob Herring<[email protected]>
> ---
> .../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 --

For nvmem parts,

Reviewed-by: Srinivas Kandagatla <[email protected]>

--srini

2021-03-17 09:48:11

by Maxime Ripard

[permalink] [raw]

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

Attachments:

(No filename) (1.72 kB)
signature.asc (235.00 B)
Download all attachments

2021-03-17 22:47:08

by Kuninori Morimoto

[permalink] [raw]

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

Hi Rob

> 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.
>
> Cc: Nicolas Saenz Julienne <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: Linus Walleij <[email protected]>
> Cc: Bartosz Golaszewski <[email protected]>
> Cc: Bjorn Andersson <[email protected]>
> Cc: Krzysztof Kozlowski <[email protected]>
> Cc: Marc Kleine-Budde <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Jakub Kicinski <[email protected]>
> Cc: Srinivas Kandagatla <[email protected]>
> Cc: Ohad Ben-Cohen <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Cheng-Yi Chiang <[email protected]>
> Cc: Benson Leung <[email protected]>
> Cc: Zhang Rui <[email protected]>
> Cc: Daniel Lezcano <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: Stefan Wahren <[email protected]>
> Cc: Masahiro Yamada <[email protected]>
> Cc: Odelu Kukatla <[email protected]>
> Cc: Alex Elder <[email protected]>
> Cc: Suman Anna <[email protected]>
> Cc: Kuninori Morimoto <[email protected]>
> Cc: Dmitry Baryshkov <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>
> ---

For below

Acked-by: Kuninori Morimoto <[email protected]>

> Documentation/devicetree/bindings/sound/ak4642.yaml | 2 --
> Documentation/devicetree/bindings/sound/renesas,rsnd.yaml | 1 -

Thank you for your help !!

Best regards
---
Kuninori Morimoto

2021-03-18 09:18:07

by Nicolas Saenz Julienne

[permalink] [raw]

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

On Tue, 2021-03-16 at 13:48 -0600, 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.
>
> Cc: Nicolas Saenz Julienne <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: Linus Walleij <[email protected]>
> Cc: Bartosz Golaszewski <[email protected]>
> Cc: Bjorn Andersson <[email protected]>
> Cc: Krzysztof Kozlowski <[email protected]>
> Cc: Marc Kleine-Budde <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Jakub Kicinski <[email protected]>
> Cc: Srinivas Kandagatla <[email protected]>
> Cc: Ohad Ben-Cohen <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Cheng-Yi Chiang <[email protected]>
> Cc: Benson Leung <[email protected]>
> Cc: Zhang Rui <[email protected]>
> Cc: Daniel Lezcano <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: Stefan Wahren <[email protected]>
> Cc: Masahiro Yamada <[email protected]>
> Cc: Odelu Kukatla <[email protected]>
> Cc: Alex Elder <[email protected]>
> Cc: Suman Anna <[email protected]>
> Cc: Kuninori Morimoto <[email protected]>
> Cc: Dmitry Baryshkov <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml | 5 +----

^
|

Reviewed-by: Nicolas Saenz Julienne <[email protected]>

Regards,
Nicolas

Attachments:

signature.asc (499.00 B)
This is a digitally signed message part