Quoting Rob Herring (2019-11-11 16:55:44) > On Sun, 10 Nov 2019 17:28:21 +0800, Zhou Yanjie wrote: > > Add the clock bindings for the X1000 Soc from Ingenic. > > > > Signed-off-by: Zhou Yanjie <zhouyanjie@xxxxxxxx> > > --- > > .../devicetree/bindings/clock/ingenic,cgu.txt | 1 + > > include/dt-bindings/clock/x1000-cgu.h | 44 ++++++++++++++++++++++ > > 2 files changed, 45 insertions(+) > > create mode 100644 include/dt-bindings/clock/x1000-cgu.h > > > > 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. It looks like some extra defines were added. I carried forward your review tag.