[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-i2c
Subject:    Re: [PATCH] dt-bindings: i2c: microchip,corei2c: Fix indentation of compatible items
From:       Conor Dooley <mail () conchuod ! ie>
Date:       2022-02-24 13:51:58
Message-ID: 4fa6664a-2362-b9eb-f279-12bcbf567e5e () conchuod ! ie
[Download RAW message or body]



On 24/02/2022 13:43, Rob Herring wrote:
> On Thu, Feb 24, 2022 at 02:34:18PM +0100, Geert Uytterhoeven wrote:
> > Hi Conor,
> > 
> > On Thu, Feb 24, 2022 at 2:15 PM <Conor.Dooley@microchip.com> wrote:
> > > On 24/02/2022 12:58, Geert Uytterhoeven wrote:
> > > > On Thu, Feb 24, 2022 at 1:55 PM <Conor.Dooley@microchip.com> wrote:
> > > > > On 24/02/2022 12:31, Geert Uytterhoeven wrote:
> > > > > > make dt_binding_check:
> > > > > > 
> > > > > > Documentation/devicetree/bindings/i2c/microchip,corei2c.yaml:19:9: \
> > > > > > [warning] wrong indentation: expected 10 but found 8 (indentation)
> > > > > 
> > > > > Hey Geert,
> > > > > I've run dt_binding_check locally but I dont get a warning, is there
> > > > > something I am missing?
> > > > 
> > > > Interesting. Are you using the latest dtschema?
> > > > https://github.com/devicetree-org/dt-schema.git
> > > 
> > > I was on v2021.12, but have now tried v2022.01 & master
> > > (2022.2.dev1+gf677c85).
> > > 
> > > Is there something other than running "make dt_binding_check" in a fresh
> > > checkout that I need to do?
> > 
> > You did update your installation after the fresh[1] checkout?
> > 
> > $ cd dt-schema
> > $ pip3 install -e .
> > 
> > [1] What about "git pull"? ;-)
> 
> yamllint is not installed. It's optional from the kbuild perspective.

Safe to say I won't forget yamllint next time I do an os reinstall..

Thanks.


[prev in list] [next in list] [prev in thread] [next in thread]