Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 16 Oct 2020 19:24:19 +0300
From:   Serge Semin <fancer.lancer@gmail.com>
To:     Andy Shevchenko <andy.shevchenko@gmail.com>
Cc:     Jisheng Zhang <Jisheng.Zhang@synaptics.com>,
        Hoan Tran <hoan@os.amperecomputing.com>,
        Linus Walleij <linus.walleij@linaro.org>,
        Bartosz Golaszewski <bgolaszewski@baylibre.com>,
        Rob Herring <robh+dt@kernel.org>,
        Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
        Dinh Nguyen <dinguyen@kernel.org>,
        Khuong Dinh <khuong@os.amperecomputing.com>,
        Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>,
        Wei Xu <xuwei5@hisilicon.com>,
        "open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>,
        devicetree <devicetree@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        linux-arm Mailing List <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 8/8] dt-bindings: gpio: dw-apb: remove never-used
 "snps,dw-apb-gpio-port"
Message-ID: <20201016162419.qupop7qf5edjagxc@mobilestation>
References: <20201015115524.24e3c36b@xhacker.debian>
 <20201015115903.2a782f78@xhacker.debian>
 <20201015120457.qaooft5y5dzl4z7s@mobilestation>
 <CAHp75Vf2GZ_kQT3cdr2hRtGqe8vA8as4EP7qH0_9uCkrk5PP2A@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAHp75Vf2GZ_kQT3cdr2hRtGqe8vA8as4EP7qH0_9uCkrk5PP2A@mail.gmail.com>
Precedence: bulk

On Fri, Oct 16, 2020 at 05:43:08PM +0300, Andy Shevchenko wrote:
> On Thu, Oct 15, 2020 at 3:33 PM Serge Semin <fancer.lancer@gmail.com> wrote:
> > On Thu, Oct 15, 2020 at 11:59:03AM +0800, Jisheng Zhang wrote:
> > > The compatible string is never used.
> >
> > Before sending v2 could you run "make dt_binding_check" for the DT schema
> > modified in this patch? Like this:
> >
> > make -j8 ARCH=mips CROSS_COMPILE=mipsel-baikal-linux- dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> 

> Do you really need to install all cross compilation tools for that?
> I think it should simple work as
> 
> % make dt_binding_check
> DT_SCHEMA_FILES=Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml

Of course you don't. I just cited my normal command. Your version should be
enough to perform the bindings file validation. Note that there are host
tools/libs requirements, which need to be satisfied to make that command
working. See the next file for details:
Documentation/devicetree/writing-schema.rst

-Sergey

> 
> -- 
> With Best Regards,
> Andy Shevchenko