linux-kernel - Re: [PATCH v2 0/2] add clk controller driver for Meson-AXG SoC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1511790489.30519.11.camel@baylibre.com>
Date:   Mon, 27 Nov 2017 14:48:09 +0100
From:   Jerome Brunet <jbrunet@...libre.com>
To:     Yixun Lan <yixun.lan@...ogic.com>,
        Neil Armstrong <narmstrong@...libre.com>,
        Kevin Hilman <khilman@...libre.com>
Cc:     Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...eaurora.org>,
        Carlo Caione <carlo@...one.org>,
        Qiufang Dai <qiufang.dai@...ogic.com>,
        linux-amlogic@...ts.infradead.org, devicetree@...r.kernel.org,
        linux-clk@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/2] add clk controller driver for Meson-AXG SoC

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
>