Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753972AbbBET0I (ORCPT ); Thu, 5 Feb 2015 14:26:08 -0500 Received: from foss-mx-na.foss.arm.com ([217.140.108.86]:42154 "EHLO foss-mx-na.foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753913AbbBET0F (ORCPT ); Thu, 5 Feb 2015 14:26:05 -0500 Date: Thu, 5 Feb 2015 19:25:33 +0000 From: Mark Rutland To: Bintian Wang Cc: "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Catalin Marinas , Will Deacon , "devicetree@vger.kernel.org" , "robh+dt@kernel.org" , Pawel Moll , "ijc+devicetree@hellion.org.uk" , "galak@codeaurora.org" , "khilman@linaro.org" , "mturquette@linaro.org" , "rob.herring@linaro.org" , "zhangfei.gao@linaro.org" , "haojian.zhuang@linaro.org" , "xuwei5@hisilicon.com" , "jh80.chung@samsung.com" , "olof@lixom.net" , "yanhaifeng@gmail.com" , "sboyd@codeaurora.org" , "xuejiancheng@huawei.com" , "sledge.yanwei@huawei.com" , "tomeu.vizoso@collabora.com" , "linux@arm.linux.org.uk" , "guodong.xu@linaro.org" , "xuyiping@hisilicon.com" , "wangbinghui@hisilicon.com" , "zhenwei.wang@hisilicon.com" , "victor.lixin@hisilicon.com" , "puck.chen@hisilicon.com" , "dan.zhao@hisilicon.com" , "huxinwei@huawei.com" , "z.liuxinliang@huawei.com" , "heyunlei@huawei.com" , "kong.kongxinwei@hisilicon.com" , "btw@mail.itp.ac.cn" , "w.f@huawei.com" , "liguozhu@hisilicon.com" Subject: Re: [PATCH 2/3] clk: hi6220: Clock driver support for Hisilicon hi6220 SoC Message-ID: <20150205192533.GE20735@leverpostej> References: <1423128277-10297-1-git-send-email-bintian.wang@huawei.com> <1423128277-10297-3-git-send-email-bintian.wang@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1423128277-10297-3-git-send-email-bintian.wang@huawei.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3626 Lines: 77 On Thu, Feb 05, 2015 at 09:24:36AM +0000, Bintian Wang wrote: > Add clock drivers for hi6220 SoC, this driver controls the SoC > registers to supply different clocks to different IPs in the SoC. > > We add one divider clock for hi6220 because the divider in hi6220 > also has a mask bit but it doesnot obey the rule defined by flag > "CLK_DIVIDER_HIWORD_MASK", we can not get index of the mask bit by > left shift fixed bits (e.g. 16 bits), so we add this divider clock > to handle it. > > This patch also enables this clock driver for ARCH_HISI and document > devicetree bindings. > > Signed-off-by: Bintian Wang > Reviewed-by: Haojian Zhuang > Reviewed-by: Zhangfei Gao > --- > .../devicetree/bindings/clock/hi6220-clock.txt | 30 +++ > arch/arm64/Kconfig | 1 + > drivers/clk/Kconfig | 2 + > drivers/clk/Makefile | 4 +- > drivers/clk/hisilicon/Kconfig | 5 + > drivers/clk/hisilicon/Makefile | 1 + > drivers/clk/hisilicon/clk-hi6220.c | 284 ++++++++++++++++++++ > drivers/clk/hisilicon/clk.c | 29 ++ > drivers/clk/hisilicon/clk.h | 17 ++ > drivers/clk/hisilicon/clkdivider-hi6220.c | 273 +++++++++++++++++++ > include/dt-bindings/clock/hi6220-clock.h | 172 ++++++++++++ > 11 files changed, 815 insertions(+), 3 deletions(-) > create mode 100644 Documentation/devicetree/bindings/clock/hi6220-clock.txt > create mode 100644 drivers/clk/hisilicon/Kconfig > create mode 100644 drivers/clk/hisilicon/clk-hi6220.c > create mode 100644 drivers/clk/hisilicon/clkdivider-hi6220.c > create mode 100644 include/dt-bindings/clock/hi6220-clock.h > > diff --git a/Documentation/devicetree/bindings/clock/hi6220-clock.txt b/Documentation/devicetree/bindings/clock/hi6220-clock.txt > new file mode 100644 > index 0000000..a3ddda1 > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/hi6220-clock.txt > @@ -0,0 +1,30 @@ > +* Hisilicon Hi6220 Clock Controller > + > +The hi6220 clock controller generates and supplies clock to various > +controllers within the hi6220 SoC. > + > +Required Properties: > + > +- compatible: should be one of the following: > + - "hisilicon,hi6220-clock-ao" - controller for those clocks under SoC > + power always on(AO) domain, it is the sub node of SoC power AO > + controller in dts file. > + - "hisilicon,hi6220-clock-sys" - controller for those clocks under SoC > + system control domain, it is the sub node of SoC system controller > + in dts file. > + - "hisilicon,hi6220-clock-media" - controller for those clocks under > + SoC media control domain, it is the sub node of SoC media controller > + in dts file. > + - "hisilicon,hi6220-clock-power" - controller for those clocks under > + SoC power control domain, it is the sub node of SoC power controller > + in dts file. These all refer to things which aren't documented (yet). Please sort out your patches so that any documentation you depend on comes earlier. Please also separate documentation from code. Note that dt includes are _bindings_ and should be added with the relevant documentation. Thanks, Mark. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/