diff mbox series

dt-bindings: Fix generated example files getting added to schemas

Message ID 20190730145935.26248-1-robh@kernel.org
State Accepted
Commit 740ce365a4dccc1df4d05219cd5bf68f01359196
Headers show
Series dt-bindings: Fix generated example files getting added to schemas | expand

Commit Message

Rob Herring July 30, 2019, 2:59 p.m. UTC
Commit 837158b847a4 ("dt-bindings: Check the examples against the
schemas") started generating YAML encoded DT files to validate the
examples against the schema. When running 'make dt_binding_check' in
tree after the 1st time, the generated example .dt.yaml files are
mistakenly added to the list of schema files. Exclude *.example.dt.yaml
files from the search for schema files.

Fixes: 837158b847a4 ("dt-bindings: Check the examples against the schemas")
Reported-by: Guido Günther <agx@sigxcpu.org>
Signed-off-by: Rob Herring <robh@kernel.org>

---
 Documentation/devicetree/bindings/Makefile | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

-- 
2.20.1

Comments

Guido Günther July 30, 2019, 3:42 p.m. UTC | #1
Hi,
On Tue, Jul 30, 2019 at 08:59:35AM -0600, Rob Herring wrote:
> Commit 837158b847a4 ("dt-bindings: Check the examples against the

> schemas") started generating YAML encoded DT files to validate the

> examples against the schema. When running 'make dt_binding_check' in

> tree after the 1st time, the generated example .dt.yaml files are

> mistakenly added to the list of schema files. Exclude *.example.dt.yaml

> files from the search for schema files.

> 

> Fixes: 837158b847a4 ("dt-bindings: Check the examples against the schemas")

> Reported-by: Guido Günther <agx@sigxcpu.org>

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

> ---

>  Documentation/devicetree/bindings/Makefile | 4 +++-

>  1 file changed, 3 insertions(+), 1 deletion(-)

> 

> diff --git a/Documentation/devicetree/bindings/Makefile b/Documentation/devicetree/bindings/Makefile

> index 6b0dfd5c17ba..5138a2f6232a 100644

> --- a/Documentation/devicetree/bindings/Makefile

> +++ b/Documentation/devicetree/bindings/Makefile

> @@ -19,7 +19,9 @@ quiet_cmd_mk_schema = SCHEMA  $@

>  

>  DT_DOCS = $(shell \

>  	cd $(srctree)/$(src) && \

> -	find * \( -name '*.yaml' ! -name $(DT_TMP_SCHEMA) \) \

> +	find * \( -name '*.yaml' ! \

> +		-name $(DT_TMP_SCHEMA) ! \

> +		-name '*.example.dt.yaml' \) \

>  	)

>  

>  DT_SCHEMA_FILES ?= $(addprefix $(src)/,$(DT_DOCS))



this fixes checking twice in a row for me. Thanks!

Tested-by: Guido Günther <agx@sigxcpu.org>


 -- Guido
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/Makefile b/Documentation/devicetree/bindings/Makefile
index 6b0dfd5c17ba..5138a2f6232a 100644
--- a/Documentation/devicetree/bindings/Makefile
+++ b/Documentation/devicetree/bindings/Makefile
@@ -19,7 +19,9 @@  quiet_cmd_mk_schema = SCHEMA  $@
 
 DT_DOCS = $(shell \
 	cd $(srctree)/$(src) && \
-	find * \( -name '*.yaml' ! -name $(DT_TMP_SCHEMA) \) \
+	find * \( -name '*.yaml' ! \
+		-name $(DT_TMP_SCHEMA) ! \
+		-name '*.example.dt.yaml' \) \
 	)
 
 DT_SCHEMA_FILES ?= $(addprefix $(src)/,$(DT_DOCS))