Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753815AbaA0KVT (ORCPT ); Mon, 27 Jan 2014 05:21:19 -0500 Received: from cam-admin0.cambridge.arm.com ([217.140.96.50]:39396 "EHLO cam-admin0.cambridge.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751571AbaA0KVM (ORCPT ); Mon, 27 Jan 2014 05:21:12 -0500 Date: Mon, 27 Jan 2014 10:20:37 +0000 From: Mark Rutland To: Jonas Jensen Cc: "mturquette@linaro.org" , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "arm@kernel.org" , "tomasz.figa@gmail.com" , "adam.jaremko@gmail.com" , "sylvester.nawrocki@gmail.com" , Sudeep Holla Subject: Re: [PATCH v10] clk: add MOXA ART SoCs clock driver Message-ID: <20140127102037.GB16516@e106331-lin.cambridge.arm.com> References: <1389971035-17781-1-git-send-email-jonas.jensen@gmail.com> <1390308261-4026-1-git-send-email-jonas.jensen@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1390308261-4026-1-git-send-email-jonas.jensen@gmail.com> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 21, 2014 at 12:44:21PM +0000, Jonas Jensen wrote: > MOXA ART SoCs allow to determine PLL output and APB frequencies > by reading registers holding multiplier and divisor information. > > Add a clock driver for this SoC. > > Signed-off-by: Jonas Jensen > --- > > Notes: > Thanks for the reply Sudeep, changes are in v10. > > Changes since v9: > > 1. rebase drivers/clk/Makefile to next-20140121 > 2. remove unnecessary switch > 3. use a more elaborate commit message > > Applies to next-20140121 > > .../bindings/clock/moxa,moxart-clock.txt | 48 +++++++++++ > drivers/clk/Makefile | 1 + > drivers/clk/clk-moxart.c | 99 ++++++++++++++++++++++ > 3 files changed, 148 insertions(+) > create mode 100644 Documentation/devicetree/bindings/clock/moxa,moxart-clock.txt > create mode 100644 drivers/clk/clk-moxart.c > > diff --git a/Documentation/devicetree/bindings/clock/moxa,moxart-clock.txt b/Documentation/devicetree/bindings/clock/moxa,moxart-clock.txt > new file mode 100644 > index 0000000..242e3fc > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/moxa,moxart-clock.txt > @@ -0,0 +1,48 @@ > +Device Tree Clock bindings for arch-moxart > + > +This binding uses the common clock binding[1]. > + > +[1] Documentation/devicetree/bindings/clock/clock-bindings.txt > + > +MOXA ART SoCs allow to determine PLL output and APB frequencies > +by reading registers holding multiplier and divisor information. > + > + > +PLL: > + > +Required properties: > +- compatible : Must be "moxa,moxart-pll-clock" > +- #clock-cells : Should be 0 > +- reg : Should contain registers location and length > +- clocks : Should contain phandle to parent clock Nit: clocks are referenced with a clock-specifier (which might be zero cells), not just a phandle. How about: clocks: should contain a phandle + clock-specifier for the parent clock > + > +Optional properties: > +- clock-output-names : Should contain clock name > + > + > +APB: > + > +Required properties: > +- compatible : Must be "moxa,moxart-apb-clock" > +- #clock-cells : Should be 0 > +- reg : Should contain registers location and length > +- clocks : Should contain phandle to parent clock Likewise. > + > +Optional properties: > +- clock-output-names : Should contain clock name > + > + > +For example: > + > + clk_pll: clk_pll@98100000 { > + compatible = "moxa,moxart-pll-clock"; > + #clock-cells = <0>; > + reg = <0x98100000 0x34>; > + }; > + > + clk_apb: clk_apb@98100000 { > + compatible = "moxa,moxart-apb-clock"; > + #clock-cells = <0>; > + reg = <0x98100000 0x34>; > + clocks = <&clk_pll>; > + }; > diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile > index 58b2d72..24361bf 100644 > --- a/drivers/clk/Makefile > +++ b/drivers/clk/Makefile > @@ -12,6 +12,7 @@ obj-$(CONFIG_COMMON_CLK) += clk-composite.o > # SoCs specific > obj-$(CONFIG_ARCH_BCM2835) += clk-bcm2835.o > obj-$(CONFIG_ARCH_EFM32) += clk-efm32gg.o > +obj-$(CONFIG_ARCH_MOXART) += clk-moxart.o > obj-$(CONFIG_ARCH_NOMADIK) += clk-nomadik.o > obj-$(CONFIG_ARCH_HIGHBANK) += clk-highbank.o > obj-$(CONFIG_ARCH_HI3xxx) += hisilicon/ > diff --git a/drivers/clk/clk-moxart.c b/drivers/clk/clk-moxart.c > new file mode 100644 > index 0000000..7021748 > --- /dev/null > +++ b/drivers/clk/clk-moxart.c > @@ -0,0 +1,99 @@ > +/* > + * MOXA ART SoCs clock driver. > + * > + * Copyright (C) 2013 Jonas Jensen > + * > + * Jonas Jensen > + * > + * This file is licensed under the terms of the GNU General Public > + * License version 2. This program is licensed "as is" without any > + * warranty of any kind, whether express or implied. > + */ > + > +#include > +#include > +#include > +#include > + > +void __init moxart_of_pll_clk_init(struct device_node *node) > +{ > + static void __iomem *base; > + struct clk *clk, *ref_clk; > + unsigned long rate; > + unsigned int mul; > + const char *name = node->name; > + > + of_property_read_string(node, "clock-output-names", &name); > + > + base = of_iomap(node, 0); > + if (!base) { > + pr_err("%s: of_iomap failed\n", node->full_name); > + return; > + } > + > + mul = readl(base + 0x30) >> 3 & 0x3f; > + iounmap(base); > + > + ref_clk = of_clk_get(node, 0); > + if (IS_ERR(ref_clk)) { > + pr_err("%s: of_clk_get failed\n", node->full_name); > + return; > + } > + > + rate = mul * clk_get_rate(ref_clk); > + > + clk = clk_register_fixed_rate(NULL, name, NULL, CLK_IS_ROOT, rate); If this clock has a parent, Surely CLK_IS_ROOT is not appropriate by its definition: #define CLK_IS_ROOT BIT(4) /* root clk, has no parent */ Surely this is a fixed factor clock rather than a fixed rate clock? > + if (IS_ERR(clk)) { > + pr_err("%s: clk_register_fixed_rate failed\n", node->full_name); > + return; > + } > + > + clk_register_clkdev(clk, NULL, name); > + of_clk_add_provider(node, of_clk_src_simple_get, clk); > +} > +CLK_OF_DECLARE(moxart_pll_clock, "moxa,moxart-pll-clock", > + moxart_of_pll_clk_init); > + > +void __init moxart_of_apb_clk_init(struct device_node *node) > +{ > + static void __iomem *base; > + struct clk *clk, *pll_clk; > + unsigned long rate; > + unsigned int div, val; > + unsigned int div_idx[] = { 2, 3, 4, 6, 8}; > + const char *name = node->name; > + > + of_property_read_string(node, "clock-output-names", &name); > + > + base = of_iomap(node, 0); > + if (!base) { > + pr_err("%s: of_iomap failed\n", node->full_name); > + return; > + } > + > + val = readl(base + 0xc) >> 4 & 0x7; > + iounmap(base); > + > + if (val > 4) > + val = 0; > + div = div_idx[val]; > + > + pll_clk = of_clk_get(node, 0); > + if (IS_ERR(pll_clk)) { > + pr_err("%s: of_clk_get failed\n", node->full_name); > + return; > + } > + > + rate = clk_get_rate(pll_clk) / (div * 2); > + > + clk = clk_register_fixed_rate(NULL, name, NULL, CLK_IS_ROOT, rate); Likewise. 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/