diff mbox series

[2/2] dt/bindings: clk: fsl,plldig: Drop 'bindings' from schema id

Message ID 20200203052507.93215-2-sboyd@kernel.org
State Not Applicable, archived
Headers show
Series None | expand

Commit Message

Stephen Boyd Feb. 3, 2020, 5:25 a.m. UTC
Having 'bindings' in here causes a warning when checking the schema.

 Documentation/devicetree/bindings/clock/fsl,plldig.yaml:
 $id: relative path/filename doesn't match actual path or filename
         expected: http://devicetree.org/schemas/clock/fsl,plldig.yaml#

Remove it.

Cc: Rob Herring <robh+dt@kernel.org>
Cc: Wen He <wen.he_1@nxp.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
---
 Documentation/devicetree/bindings/clock/fsl,plldig.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Rob Herring Feb. 3, 2020, 9:39 a.m. UTC | #1
On Mon, Feb 3, 2020 at 5:25 AM Stephen Boyd <sboyd@kernel.org> wrote:
>
> Having 'bindings' in here causes a warning when checking the schema.
>
>  Documentation/devicetree/bindings/clock/fsl,plldig.yaml:
>  $id: relative path/filename doesn't match actual path or filename
>          expected: http://devicetree.org/schemas/clock/fsl,plldig.yaml#
>
> Remove it.
>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Wen He <wen.he_1@nxp.com>
> Signed-off-by: Stephen Boyd <sboyd@kernel.org>
> ---
>  Documentation/devicetree/bindings/clock/fsl,plldig.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

There's actually a more few of these in clock bindings. I am going to
do a tree wide fix on this. I was waiting until the clock tree is
merged.

And I didn't really mean to commit this check to dtschema until all
these were fixed, so I've reverted it for now.

So either go ahead and apply this or I'll get it.

Acked-by: Rob Herring <robh@kernel.org>

Rob
Stephen Boyd Feb. 3, 2020, 6:33 p.m. UTC | #2
Quoting Stephen Boyd (2020-02-02 21:25:07)
> Having 'bindings' in here causes a warning when checking the schema.
> 
>  Documentation/devicetree/bindings/clock/fsl,plldig.yaml:
>  $id: relative path/filename doesn't match actual path or filename
>          expected: http://devicetree.org/schemas/clock/fsl,plldig.yaml#
> 
> Remove it.
> 
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Wen He <wen.he_1@nxp.com>
> Signed-off-by: Stephen Boyd <sboyd@kernel.org>
> ---

Applied to clk-next
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/clock/fsl,plldig.yaml b/Documentation/devicetree/bindings/clock/fsl,plldig.yaml
index ad37d3273229..c8350030b374 100644
--- a/Documentation/devicetree/bindings/clock/fsl,plldig.yaml
+++ b/Documentation/devicetree/bindings/clock/fsl,plldig.yaml
@@ -1,7 +1,7 @@ 
 # SPDX-License-Identifier: GPL-2.0
 %YAML 1.2
 ---
-$id: http://devicetree.org/schemas/bindings/clock/fsl,plldig.yaml#
+$id: http://devicetree.org/schemas/clock/fsl,plldig.yaml#
 $schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: NXP QorIQ Layerscape LS1028A Display PIXEL Clock Binding