Re: [PATCH v3] dt-bindings: pmem: Convert binding to YAML

[Ira Weiny <ira.weiny@intel.com>]

Dan Williams wrote:
> [ add Ira ]
> 
> Drew Fustini wrote:
> > Convert the PMEM device tree binding from text to YAML. This will allow
> > device trees with pmem-region nodes to pass dtbs_check.
> > 
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> > Acked-by: Oliver O'Halloran <oohall@gmail.com>
> > Signed-off-by: Drew Fustini <drew@pdp7.com>
> > ---
> > Dan/Dave/Vishal: does it make sense for this pmem binding patch to go
> > through the nvdimm tree?
> 
> Ira has been handling nvdimm pull requests as of late. Oliver's ack is
> sufficient for me.
> 
> Acked-by: Dan Williams <dan.j.williams@intel.com>
> 
> @Ira do you have anything else pending?
> 

I don't.  I've never built the device tree make targets to test.

The docs[1] say to run make dtbs_check but it is failing:

$ make dtbs_check
make[1]: *** No rule to make target 'dtbs_check'.  Stop.
make: *** [Makefile:248: __sub-make] Error 2


dt_binding_check fails too.

$ make dt_binding_check
  SCHEMA  Documentation/devicetree/bindings/processed-schema.json
Traceback (most recent call last):
  File "/usr/bin/dt-mk-schema", line 8, in <module>
    sys.exit(main())
             ~~~~^^
  File "/usr/lib/python3.13/site-packages/dtschema/mk_schema.py", line 28, in main
    schemas = dtschema.DTValidator(args.schemas).schemas
              ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^
  File "/usr/lib/python3.13/site-packages/dtschema/validator.py", line 373, in __init__
    self.make_property_type_cache()
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^
  File "/usr/lib/python3.13/site-packages/dtschema/validator.py", line 460, in make_property_type_cache
    self.props, self.pat_props = get_prop_types(self.schemas)
                                 ~~~~~~~~~~~~~~^^^^^^^^^^^^^^
  File "/usr/lib/python3.13/site-packages/dtschema/validator.py", line 194, in get_prop_types
    del props[r'^[a-z][a-z0-9\-]*$']
        ~~~~~^^^^^^^^^^^^^^^^^^^^^^^
KeyError: '^[a-z][a-z0-9\\-]*$'
make[2]: *** [Documentation/devicetree/bindings/Makefile:63: Documentation/devicetree/bindings/processed-schema.json] Error 1
make[2]: *** Deleting file 'Documentation/devicetree/bindings/processed-schema.json'
make[1]: *** [/home/iweiny/dev/linux-nvdimm/Makefile:1522: dt_binding_schemas] Error 2
make: *** [Makefile:248: __sub-make] Error 2

How do I test this?

Ira

[1] https://docs.kernel.org/devicetree/bindings/writing-schema.html