2020-02-03 07:38:29

by Stephen Boyd

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

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 <[email protected]>
Cc: Wen He <[email protected]>
Signed-off-by: Stephen Boyd <[email protected]>
---
Documentation/devicetree/bindings/clock/fsl,plldig.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

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
--
Sent by a computer, using git, on the internet

2020-02-03 11:19:31

by Rob Herring

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

On Mon, Feb 3, 2020 at 5:25 AM Stephen Boyd <[email protected]> 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 <[email protected]>
> Cc: Wen He <[email protected]>
> Signed-off-by: Stephen Boyd <[email protected]>
> ---
> 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 <[email protected]>

Rob

2020-02-03 19:15:34

by Stephen Boyd

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

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 <[email protected]>
> Cc: Wen He <[email protected]>
> Signed-off-by: Stephen Boyd <[email protected]>
> ---

Applied to clk-next