linux-kernel - Re: [PATCH v2] clk: hisilicon: add CRG driver for Hi3798CV200 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]

Date:   Fri, 23 Sep 2016 12:47:04 -0500
From:   Rob Herring <robh@...nel.org>
To:     Jiancheng Xue <xuejiancheng@...ilicon.com>
Cc:     mturquette@...libre.com, sboyd@...eaurora.org,
        mark.rutland@....com, linux-clk@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        yanhaifeng@...ilicon.com, gaofei@...ilicon.com,
        hermit.wangheming@...ilicon.com, scott.bambrough@...aro.org,
        mark.gregotski@...aro.org
Subject: Re: [PATCH v2] clk: hisilicon: add CRG driver for Hi3798CV200 SoC

On Sun, Sep 18, 2016 at 03:30:21PM +0800, Jiancheng Xue wrote:
> Add CRG driver for Hi3798CV200 SoC. CRG(Clock and Reset
> Generator) module generates clock and reset signals used
> by other module blocks on SoC.
> 
> Signed-off-by: Jiancheng Xue <xuejiancheng@...ilicon.com>
> ---
> change log
> v2:
> - Fixed compiling error when compiled as a module.
> - Fixed issues pointed by Stephen Boyd.
> - Added prefix HISTB for clock index macro definitions.
> 
>  .../clock/{hi3519-crg.txt => hisi-crg.txt}         |  11 +-
>  drivers/clk/hisilicon/Kconfig                      |   8 +
>  drivers/clk/hisilicon/Makefile                     |   1 +
>  drivers/clk/hisilicon/crg-hi3798cv200.c            | 305 +++++++++++++++++++++
>  drivers/clk/hisilicon/crg.h                        |  34 +++
>  include/dt-bindings/clock/histb-clock.h            |  64 +++++
>  6 files changed, 419 insertions(+), 4 deletions(-)
>  rename Documentation/devicetree/bindings/clock/{hi3519-crg.txt => hisi-crg.txt} (82%)
>  create mode 100644 drivers/clk/hisilicon/crg-hi3798cv200.c
>  create mode 100644 drivers/clk/hisilicon/crg.h
>  create mode 100644 include/dt-bindings/clock/histb-clock.h
> 
> diff --git a/Documentation/devicetree/bindings/clock/hi3519-crg.txt b/Documentation/devicetree/bindings/clock/hisi-crg.txt
> similarity index 82%
> rename from Documentation/devicetree/bindings/clock/hi3519-crg.txt
> rename to Documentation/devicetree/bindings/clock/hisi-crg.txt
> index acd1f23..e3919b6 100644
> --- a/Documentation/devicetree/bindings/clock/hi3519-crg.txt
> +++ b/Documentation/devicetree/bindings/clock/hisi-crg.txt
> @@ -1,7 +1,7 @@
> -* Hisilicon Hi3519 Clock and Reset Generator(CRG)
> +* HiSilicon Clock and Reset Generator(CRG)
>  
> -The Hi3519 CRG module provides clock and reset signals to various
> -controllers within the SoC.
> +The CRG module provides clock and reset signals to various
> +modules within the SoC.
>  
>  This binding uses the following bindings:
>      Documentation/devicetree/bindings/clock/clock-bindings.txt
> @@ -10,7 +10,10 @@ This binding uses the following bindings:
>  Required Properties:
>  
>  - compatible: should be one of the following.
> -  - "hisilicon,hi3519-crg" - controller compatible with Hi3519 SoC.
> +  - "hisilicon,hi3516cv300-crg"
> +  - "hisilicon,hi3519-crg"
> +  - "hisilicon,hi3798cv200-crg"
> +  - "hisilicon,hi3798cv200-sysctrl"

Why crg and sysctrl?

>  
>  - reg: physical base address of the controller and length of memory mapped
>    region.