On 2020/3/3 16:43, Kevin Hilman wrote: > Rob Herring <robh@xxxxxxxxxx> writes: > >> On Mon, 2 Mar 2020 23:54:08 +0800, Jianxin Pan wrote: >>> Missing ';' in the end of secure-monitor example node. >>> >>> Fixes: 165b5fb294e8 ("dt-bindings: power: add Amlogic secure power domains bindings") >>> Reported-by: Rob Herring <robh+dt@xxxxxxxxxx> >>> Signed-off-by: Jianxin Pan <jianxin.pan@xxxxxxxxxxx> >>> --- >>> Documentation/devicetree/bindings/power/amlogic,meson-sec-pwrc.yaml | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >> >> 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. > > I've (re)added these tags: > > Reviewed-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx> > Acked-by: Rob Herring <robh@xxxxxxxxxx> > > when applying this time. > > Jianxin, please collect the tags in the future and add when you send > follow-up versions. OK, I will do it next time, thanks for your time. > > Thanks, > > Kevin > > . >