Re: [EXTERNAL] Re: [PATCH v15 2/4] dt-bindings: power: Convert battery.txt to battery.yaml

[Ricardo Rivera-Matos]

Rob

On 7/2/20 3:53 PM, Rob Herring wrote:
> On Wed, 01 Jul 2020 16:10:42 -0500, Ricardo Rivera-Matos wrote:
> > From: Dan Murphy <dmurphy@xxxxxx>
> >
> > Convert the battery.txt file to yaml and fix up the examples.
> >
> > Signed-off-by: Dan Murphy <dmurphy@xxxxxx>
> > ---
> >   .../bindings/power/supply/battery.txt         |  86 +---------
> >   .../bindings/power/supply/battery.yaml        | 157 ++++++++++++++++++
> >   2 files changed, 158 insertions(+), 85 deletions(-)
> >   create mode 100644 Documentation/devicetree/bindings/power/supply/battery.yaml
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> Unknown file referenced: [Errno 2] No such file or directory: '/usr/local/lib/python3.6/dist-packages/dtschema/schema/types.yaml'
> Documentation/devicetree/bindings/Makefile:20: recipe for target 'Documentation/devicetree/bindings/power/supply/battery.example.dts' failed
> make[1]: *** [Documentation/devicetree/bindings/power/supply/battery.example.dts] Error 255
> make[1]: *** Waiting for unfinished jobs....
> Makefile:1347: recipe for target 'dt_binding_check' failed
> make: *** [dt_binding_check] Error 2
I think your bot is looking for the types.yaml in the wrong place. 
'/usr/local/lib/python3.6/dist-packages/dtschema/schema/types.yaml' 
should be 
'/usr/local/lib/python3.6/dist-packages/dtschema/schemas/types.yaml'. I 
renamed might 'schemas' directory to 'schema' and my battery.yaml passed 
the dt_binding_check.
> See https://patchwork.ozlabs.org/patch/1320813
>
> If you already ran 'make dt_binding_check' and didn't see the above
> error(s), then make sure dt-schema is up to date:
>
> pip3 install git+https://github.com/devicetree-org/dt-schema.git@master --upgrade
Confirmed on latest dt-schema
> Please check and re-submit.

ÂÂÂ Thanks,

ÂÂÂ ÂÂÂ Ricardo

>