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@xxxxxxxxxxx> > > > --- > > > 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@xxxxxxxxxxxxx [1] > > [1] https://lore.kernel.org/all/6dbeda18-a11c-609d-7a8f-bf2e6f27aea7@xxxxxxxxxx/ A trivial change like that can keep a tag especially if it came after the comment (not sure here). Rob