Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Fri, 1 Apr 2022 12:53:14 -0500
From:   Rob Herring <robh@kernel.org>
To:     =?utf-8?B?cWluamlhblvopoPlgaVd?= <qinjian@cqplus1.com>
Cc:     "arnd@arndb.de" <arnd@arndb.de>,
        "linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>,
        "p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
        "maz@kernel.org" <maz@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "mturquette@baylibre.com" <mturquette@baylibre.com>,
        "sboyd@kernel.org" <sboyd@kernel.org>,
        "linux@armlinux.org.uk" <linux@armlinux.org.uk>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "krzysztof.kozlowski@canonical.com" 
        <krzysztof.kozlowski@canonical.com>,
        "tglx@linutronix.de" <tglx@linutronix.de>
Subject: Re: [PATCH v12 6/9] dt-bindings: interrupt-controller: Add bindings
 for SP7021 interrupt controller
Message-ID: <Ykc8Cv/TbYlr9GxV@robh.at.kernel.org>
References: <cover.1648714851.git.qinjian@cqplus1.com>
 <ff5bfd5611ab0defe0c98f98edbbf655e33cd16d.1648714851.git.qinjian@cqplus1.com>
 <YkYSyHVGsXkGs0uf@robh.at.kernel.org>
 <3373b11528214394baae71198df3adff@cqplus1.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <3373b11528214394baae71198df3adff@cqplus1.com>
Precedence: bulk

On Fri, Apr 01, 2022 at 02:29:58AM +0000, qinjian[覃健] wrote:
> > 
> > On Thu, 31 Mar 2022 16:29:53 +0800, Qin Jian wrote:
> > > Add documentation to describe Sunplus SP7021 interrupt controller bindings.
> > >
> > > Signed-off-by: Qin Jian <qinjian@cqplus1.com>
> > > ---
> > > Move 'reg' after 'compatible'
> > > ---
> > >  .../sunplus,sp7021-intc.yaml                  | 62 +++++++++++++++++++
> > >  MAINTAINERS                                   |  1 +
> > >  2 files changed, 63 insertions(+)
> > >  create mode 100644 Documentation/devicetree/bindings/interrupt-controller/sunplus,sp7021-intc.yaml
> > >
> > 
> > 
> > Please add Acked-by/Reviewed-by tags when posting new versions. However,
> > there's no need to repost patches *only* to add the tags. The upstream
> > maintainer will do that for acks received on the version they apply.
> > 
> > If a tag was not added on purpose, please state why and what changed.
> 
> Changes: Move 'reg' after 'compatible'
> I did a modification based on comments from krzysztof.kozlowski@canonical.com [1]
> 
> [1] https://lore.kernel.org/all/6dbeda18-a11c-609d-7a8f-bf2e6f27aea7@kernel.org/

A trivial change like that can keep a tag especially if it came after 
the comment (not sure here).

Rob