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] [day] [month] [year] [list]

Date:   Tue, 28 Nov 2017 20:47:09 +0800
From:   Yixun Lan <yixun.lan@...ogic.com>
To:     Jerome Brunet <jbrunet@...libre.com>,
        Neil Armstrong <narmstrong@...libre.com>,
        Kevin Hilman <khilman@...libre.com>
CC:     <yixun.lan@...ogic.com>, 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

Hi jerome:


On 11/27/17 21:48, Jerome Brunet wrote:
> 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. 
> 
can drop the calculations, but there is already a comment at top here..

>>    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

>>   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
>>
> 
> .
>