Re: [PATCH net-next + leds v2 1/7] dt-bindings: leds: document binding for HW controlled LEDs

From: Marek Behun
Date: Wed Sep 09 2020 - 17:43:08 EST


On Wed, 9 Sep 2020 15:31:22 -0600
Rob Herring <robh@xxxxxxxxxx> wrote:

> On Wed, Sep 09, 2020 at 11:07:26PM +0200, Marek Behun wrote:
> > On Wed, 9 Sep 2020 14:59:23 -0600
> > Rob Herring <robh@xxxxxxxxxx> wrote:
> >
> > > >
> > > > I don't know :) I copied this from other drivers, I once tried setting
> > > > up environment for doing checking of device trees with YAML schemas,
> > > > and it was a little painful :)
> > >
> > > pip3 install dtschema ?
> > >
> > > Can you elaborate on the issue.
> > >
> > > Rob
> > >
> >
> > I am using Gentoo and didn't want to bloat system with non-portage
> > packages, nor try to start a virtual environment. In the end I did it
> > in a chroot Ubuntu :)
>
> A user install doesn't work?
>
> I don't really care for virtual env either.
>
> > The other thing is that the make dt_binding_check executed for
> > quite a long time, and I didn't find a way to just do the binding check
> > some of the schemas.
>
> It's a bit faster now with what's queued for 5.10. The schema
> validation is under 10sec now on my laptop. For the examples, any
> new schema could apply to any example, so we have to check them all.
> It's faster too, but still minutes to run.
>
> > But I am not criticizing anything, I think that it is a good thing to
> > have this system.
>
> Good to hear. Just want to improve any pain points if possible.
>
> Rob
>

OK I am running it now in my chroot environment :)