Re: [PATCH] dt-bindings: Fix paths in schema $id fields

From: Guenter Roeck
Date: Tue Feb 04 2020 - 17:52:29 EST


On Tue, Feb 04, 2020 at 10:49:09PM +0000, Rob Herring wrote:
> The $id path checks were inadequately checking the path part of the $id
> value. With the check fixed, there's a number of errors that need to be
> fixed. Most of the errors are including 'bindings/' in the path which
> should not be as that is considered the root.
>
> Cc: Andy Gross <agross@xxxxxxxxxx>
> Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>
> Cc: Michael Turquette <mturquette@xxxxxxxxxxxx>
> Cc: Stephen Boyd <sboyd@xxxxxxxxxx>
> Cc: Shawn Guo <shawnguo@xxxxxxxxxx>
> Cc: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
> Cc: Pengutronix Kernel Team <kernel@xxxxxxxxxxxxxx>
> Cc: Fabio Estevam <festevam@xxxxxxxxx>
> Cc: NXP Linux Team <linux-imx@xxxxxxx>
> Cc: Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx>
> Cc: Alexandre Torgue <alexandre.torgue@xxxxxx>
> Cc: "Nuno Sá" <nuno.sa@xxxxxxxxxx>
> Cc: Jean Delvare <jdelvare@xxxxxxxx>
> Cc: Guenter Roeck <linux@xxxxxxxxxxxx>
> Cc: Stefan Popa <stefan.popa@xxxxxxxxxx>
> Cc: Jonathan Cameron <jic23@xxxxxxxxxx>
> Cc: Hartmut Knaack <knaack.h@xxxxxx>
> Cc: Lars-Peter Clausen <lars@xxxxxxxxxx>
> Cc: Peter Meerwald-Stadler <pmeerw@xxxxxxxxxx>
> Cc: Marcus Folkesson <marcus.folkesson@xxxxxxxxx>
> Cc: Kent Gustavsson <kent@xxxxxxxxxx>
> Cc: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx>
> Cc: linux-clk@xxxxxxxxxxxxxxx
> Cc: linux-iio@xxxxxxxxxxxxxxx
> Cc: linux-input@xxxxxxxxxxxxxxx
> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/arm/fsl.yaml | 2 +-
> Documentation/devicetree/bindings/arm/qcom.yaml | 2 +-
> Documentation/devicetree/bindings/clock/bitmain,bm1880-clk.yaml | 2 +-
> Documentation/devicetree/bindings/clock/fsl,sai-clock.yaml | 2 +-
> Documentation/devicetree/bindings/clock/imx8mn-clock.yaml | 2 +-
> Documentation/devicetree/bindings/clock/imx8mp-clock.yaml | 2 +-
> Documentation/devicetree/bindings/clock/milbeaut-clock.yaml | 2 +-
> Documentation/devicetree/bindings/clock/qcom,dispcc.yaml | 2 +-
> Documentation/devicetree/bindings/clock/qcom,gcc.yaml | 2 +-
> Documentation/devicetree/bindings/clock/qcom,gpucc.yaml | 2 +-
> Documentation/devicetree/bindings/clock/qcom,mmcc.yaml | 2 +-
> Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml | 2 +-
> Documentation/devicetree/bindings/clock/qcom,videocc.yaml | 2 +-
> Documentation/devicetree/bindings/clock/st,stm32mp1-rcc.yaml | 2 +-
> Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml | 2 +-
> Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml | 2 +-

Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx>

> Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml | 2 +-
> .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml | 2 +-
> .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml | 2 +-
> Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml | 2 +-
> Documentation/devicetree/bindings/input/gpio-vibrator.yaml | 2 +-
> 22 files changed, 22 insertions(+), 22 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> index f79683a628f0..b0a7454a70b8 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/arm/fsl.yaml#
> +$id: http://devicetree.org/schemas/arm/fsl.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Freescale i.MX Platforms Device Tree Bindings
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index e39d8f02e33c..b5bef5abc281 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/arm/qcom.yaml#
> +$id: http://devicetree.org/schemas/arm/qcom.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: QCOM device tree bindings
> diff --git a/Documentation/devicetree/bindings/clock/bitmain,bm1880-clk.yaml b/Documentation/devicetree/bindings/clock/bitmain,bm1880-clk.yaml
> index e63827399c1a..8559fe8f7efd 100644
> --- a/Documentation/devicetree/bindings/clock/bitmain,bm1880-clk.yaml
> +++ b/Documentation/devicetree/bindings/clock/bitmain,bm1880-clk.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/bitmain,bm1880-clk.yaml#
> +$id: http://devicetree.org/schemas/clock/bitmain,bm1880-clk.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Bitmain BM1880 Clock Controller
> diff --git a/Documentation/devicetree/bindings/clock/fsl,sai-clock.yaml b/Documentation/devicetree/bindings/clock/fsl,sai-clock.yaml
> index 8fb2060ac47f..fc3bdfdc091a 100644
> --- a/Documentation/devicetree/bindings/clock/fsl,sai-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/fsl,sai-clock.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/fsl,sai-clock.yaml#
> +$id: http://devicetree.org/schemas/clock/fsl,sai-clock.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Freescale SAI bitclock-as-a-clock binding
> diff --git a/Documentation/devicetree/bindings/clock/imx8mn-clock.yaml b/Documentation/devicetree/bindings/clock/imx8mn-clock.yaml
> index 622f3658bd9f..cd0b8a341321 100644
> --- a/Documentation/devicetree/bindings/clock/imx8mn-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/imx8mn-clock.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/imx8mn-clock.yaml#
> +$id: http://devicetree.org/schemas/clock/imx8mn-clock.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: NXP i.MX8M Nano Clock Control Module Binding
> diff --git a/Documentation/devicetree/bindings/clock/imx8mp-clock.yaml b/Documentation/devicetree/bindings/clock/imx8mp-clock.yaml
> index 80278882cf57..89aee63c9019 100644
> --- a/Documentation/devicetree/bindings/clock/imx8mp-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/imx8mp-clock.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/imx8mp-clock.yaml#
> +$id: http://devicetree.org/schemas/clock/imx8mp-clock.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: NXP i.MX8M Plus Clock Control Module Binding
> diff --git a/Documentation/devicetree/bindings/clock/milbeaut-clock.yaml b/Documentation/devicetree/bindings/clock/milbeaut-clock.yaml
> index 5cf0b811821e..f0b804a7f096 100644
> --- a/Documentation/devicetree/bindings/clock/milbeaut-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/milbeaut-clock.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/milbeaut-clock.yaml#
> +$id: http://devicetree.org/schemas/clock/milbeaut-clock.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Milbeaut SoCs Clock Controller Binding
> diff --git a/Documentation/devicetree/bindings/clock/qcom,dispcc.yaml b/Documentation/devicetree/bindings/clock/qcom,dispcc.yaml
> index 9c58e02a1de1..795fe686f3ea 100644
> --- a/Documentation/devicetree/bindings/clock/qcom,dispcc.yaml
> +++ b/Documentation/devicetree/bindings/clock/qcom,dispcc.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0-only
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/qcom,dispcc.yaml#
> +$id: http://devicetree.org/schemas/clock/qcom,dispcc.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Qualcomm Display Clock & Reset Controller Binding
> diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc.yaml
> index cac1150c9292..e814eec1bf8d 100644
> --- a/Documentation/devicetree/bindings/clock/qcom,gcc.yaml
> +++ b/Documentation/devicetree/bindings/clock/qcom,gcc.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0-only
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/qcom,gcc.yaml#
> +$id: http://devicetree.org/schemas/clock/qcom,gcc.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Qualcomm Global Clock & Reset Controller Binding
> diff --git a/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml b/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml
> index 622845aa643f..679e7fe0fa83 100644
> --- a/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml
> +++ b/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0-only
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/qcom,gpucc.yaml#
> +$id: http://devicetree.org/schemas/clock/qcom,gpucc.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Qualcomm Graphics Clock & Reset Controller Binding
> diff --git a/Documentation/devicetree/bindings/clock/qcom,mmcc.yaml b/Documentation/devicetree/bindings/clock/qcom,mmcc.yaml
> index 91101c915904..85518494ce43 100644
> --- a/Documentation/devicetree/bindings/clock/qcom,mmcc.yaml
> +++ b/Documentation/devicetree/bindings/clock/qcom,mmcc.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0-only
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/qcom,mmcc.yaml#
> +$id: http://devicetree.org/schemas/clock/qcom,mmcc.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Qualcomm Multimedia Clock & Reset Controller Binding
> diff --git a/Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml b/Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml
> index 94e2f14eb967..2cd158f13bab 100644
> --- a/Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml
> +++ b/Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0-only
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/qcom,rpmhcc.yaml#
> +$id: http://devicetree.org/schemas/clock/qcom,rpmhcc.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Qualcomm Technologies, Inc. RPMh Clocks Bindings
> diff --git a/Documentation/devicetree/bindings/clock/qcom,videocc.yaml b/Documentation/devicetree/bindings/clock/qcom,videocc.yaml
> index 43cfc893a8d1..2946b240e161 100644
> --- a/Documentation/devicetree/bindings/clock/qcom,videocc.yaml
> +++ b/Documentation/devicetree/bindings/clock/qcom,videocc.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0-only
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/qcom,videocc.yaml#
> +$id: http://devicetree.org/schemas/clock/qcom,videocc.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Qualcomm Video Clock & Reset Controller Binding
> diff --git a/Documentation/devicetree/bindings/clock/st,stm32mp1-rcc.yaml b/Documentation/devicetree/bindings/clock/st,stm32mp1-rcc.yaml
> index b8f91e444d2f..4e385508f516 100644
> --- a/Documentation/devicetree/bindings/clock/st,stm32mp1-rcc.yaml
> +++ b/Documentation/devicetree/bindings/clock/st,stm32mp1-rcc.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/st,stm32mp1-rcc.yaml#
> +$id: http://devicetree.org/schemas/clock/st,stm32mp1-rcc.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Reset Clock Controller Binding
> diff --git a/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml b/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml
> index f1150cad34a4..229af98b1d30 100644
> --- a/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml
> +++ b/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/clock/xlnx,versal-clk.yaml#
> +$id: http://devicetree.org/schemas/clock/xlnx,versal-clk.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Xilinx Versal clock controller
> diff --git a/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml b/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> index ae04903f34bf..6a742a51e2f9 100644
> --- a/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/hwmon/adi,ltc2947.yaml#
> +$id: http://devicetree.org/schemas/hwmon/adi,ltc2947.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Analog Devices LTC2947 high precision power and energy monitor
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> index e932d5aed02f..f0934b295edc 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> @@ -2,7 +2,7 @@
> # Copyright 2019 Analog Devices Inc.
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/iio/adc/adi,ad7124.yaml#
> +$id: http://devicetree.org/schemas/iio/adc/adi,ad7124.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Analog Devices AD7124 ADC device driver
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> index 567a33a83dce..84d25bd39488 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> @@ -2,7 +2,7 @@
> # Copyright 2019 Analog Devices Inc.
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/iio/adc/adi,ad7192.yaml#
> +$id: http://devicetree.org/schemas/iio/adc/adi,ad7192.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Analog Devices AD7192 ADC device driver
> diff --git a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> index 881059b80d61..0ce290473fb0 100644
> --- a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> @@ -2,7 +2,7 @@
> # Copyright 2019 Marcus Folkesson <marcus.folkesson@xxxxxxxxx>
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/bindings/iio/adc/microchip,mcp3911.yaml#";
> +$id: "http://devicetree.org/schemas/iio/adc/microchip,mcp3911.yaml#";
> $schema: "http://devicetree.org/meta-schemas/core.yaml#";
>
> title: Microchip MCP3911 Dual channel analog front end (ADC)
> 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 c91407081aa5..acf36eef728b 100644
> --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/st,stm32-dfsdm-adc.yaml#
> +$id: http://devicetree.org/schemas/iio/adc/st,stm32-dfsdm-adc.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: STMicroelectronics STM32 DFSDM ADC device driver
> diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
> index 13d005b68931..a285eaba7125 100644
> --- a/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
> @@ -2,7 +2,7 @@
> # Copyright 2019 Marcus Folkesson <marcus.folkesson@xxxxxxxxx>
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/bindings/iio/dac/lltc,ltc1660.yaml#";
> +$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc1660.yaml#";
> $schema: "http://devicetree.org/meta-schemas/core.yaml#";
>
> title: Linear Technology Micropower octal 8-Bit and 10-Bit DACs
> diff --git a/Documentation/devicetree/bindings/input/gpio-vibrator.yaml b/Documentation/devicetree/bindings/input/gpio-vibrator.yaml
> index 903475f52dbd..b98bf9363c8f 100644
> --- a/Documentation/devicetree/bindings/input/gpio-vibrator.yaml
> +++ b/Documentation/devicetree/bindings/input/gpio-vibrator.yaml
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/bindings/input/gpio-vibrator.yaml#
> +$id: http://devicetree.org/schemas/input/gpio-vibrator.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: GPIO vibrator
> --
> 2.20.1
>