Hi jerome:


On 11/27/17 21:48, Jerome Brunet wrote:

On Mon, 2017-11-27 at 17:48 +0800, Yixun Lan wrote:

quoted

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.

quoted

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. 

can drop the calculations, but there is already a comment at top here..

quoted

   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)

I missed the dt-binding, will fix in next version

quoted

  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

.