Re: [PATCH v2] clk: hisilicon: add CRG driver for Hi3798CV200 SoC

[Rob Herring <robh@kernel.org>]

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