On Mon, 2017-11-27 at 17:48 +0800, Yixun Lan wrote: > Add driver for the clk controller which found in Meson AXG SoC > > Note, we deliberately create a seperate source file for the Meson AXG > series, instead of sharing code with previous GXBB/GXL - the file axg.c > It would help us maintaining the code more easily. But this will be the last meson SoC added this way. We need factor the code the between meson8, gx and axg series since there is a lot of duplication there. > > Changes since v1 [1]: > - rework register definion, use '(offset << 2)' to better match As previously discussed, please drop these calculations and just write what the offset actually are. Putting one comment at the top, explaining the translation, would be nice though. > the description from data sheet > - drop "#include dt-bindings/clock/gxbb-aoclkc.h" from dts > - rebase code to v4.15-rc1 > > [1] > http://lists.infradead.org/pipermail/linux-amlogic/2017-November/005239.html > http://lists.infradead.org/pipermail/linux-amlogic/2017-November/005240.html > http://lists.infradead.org/pipermail/linux-amlogic/2017-November/005241.html > > > Qiufang Dai (2): First patch of the series should be adding the compatible documentation, which is missing here (unless it has gone through another channel and I missed it) > clk: meson-axg: add clock controller drivers > arm64: dts: meson-axg: add clock DT info for Meson AXG SoC > > arch/arm64/Kconfig.platforms | 1 + > arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 15 + > drivers/clk/meson/Kconfig | 8 + > drivers/clk/meson/Makefile | 1 + > drivers/clk/meson/axg.c | 948 > +++++++++++++++++++++++++++++ > drivers/clk/meson/axg.h | 126 ++++ > include/dt-bindings/clock/axg-clkc.h | 72 +++ > 7 files changed, 1171 insertions(+) > create mode 100644 drivers/clk/meson/axg.c > create mode 100644 drivers/clk/meson/axg.h > create mode 100644 include/dt-bindings/clock/axg-clkc.h > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html