Messages in this thread | | | From | Rob Herring <> | Date | Tue, 22 Oct 2019 11:03:43 -0500 | Subject | Re: [PATCH v2 2/2] dt-bindings: iio: adc: Add DT docs for AD7292 |
| |
On Tue, Oct 22, 2019 at 9:06 AM Marcelo Schmitt <marcelo.schmitt1@gmail.com> wrote: > > Hi, > > I ran the DTC and CHECK for AD7292 schema however, the target '__build' > did not run due to errors found in regulator/fixed-regulator.yaml and > arm/allwinner,sun4i-a10-csi.yaml.
Fixes for those are still pending in -next. Use 'make -k' and ignore those.
> > I recall seeing something about the maxItems requirement over regulator > supplies being changed on the iio mailing list, so I updated my repo > locally, cloned and reinstalled the dt-schema toolset. However, I still > can't make it go through the '__build' target. > > Python 3.7.5rc1 is my default python and I got the following pip3 > packages installed: > > ruamel.yaml 0.16.5 > ruamel.yaml.clib 0.2.0 > rfc3987 1.3.8 > jsonschema 3.0.1 > dtschema 0.0.1 at $HOME/<iio repo dir>/dt-schema > > Debian Bullseye packages installed: > python3-yaml/testing,now 5.1.2-1 > libyaml-dev/testing,now 0.2.2-1 > > I was only able to run DTC after installing the libyaml-dev package, so > I think it might be worth to add it to the project dependencies at > https://github.com/robherring/dt-schema.
Strictly speaking, it's not a dependency for dt-schema. It's documented in Documentation/devicetree/writing-schema.rst. I've added a pointer to that in bindings/submitting-patches.txt. I'm not sure how else to make it more obvious.
BTW, You will get a useful error message if libyaml is missing when building 'make dtbs_check'. I need to make that work for dt_binding_check.
> apt-get install libyaml-dev
You need the lib too, but that tends to already be installed. IIRC, installing the headers doesn't install the lib automatically.
In any case, I wanted to avoid putting in distro specific instructions in the kernel.
Rob
| |