Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752754AbaBJPXg (ORCPT ); Mon, 10 Feb 2014 10:23:36 -0500 Received: from mail-ee0-f49.google.com ([74.125.83.49]:44954 "EHLO mail-ee0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752361AbaBJPXN (ORCPT ); Mon, 10 Feb 2014 10:23:13 -0500 From: Michal Simek To: linux-arm-kernel@lists.infradead.org, Arnd Bergmann Cc: monstr@monstr.eu, Josh Cartwright , Steffen Trumtrar , Rob Herring , Peter Crosthwaite , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Mike Turquette , Soren Brinkmann , Stephen Boyd , Stephen Warren , James Hogan , Felipe Pena , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [RFC PATCH 3/3] ARM: zynq: Use early syscon initialization Date: Mon, 10 Feb 2014 16:22:35 +0100 Message-Id: <19fbad8c4c7d655ef73e0c7dd16a045a02a63286.1392045742.git.michal.simek@xilinx.com> X-Mailer: git-send-email 1.8.2.3 In-Reply-To: References: In-Reply-To: References: Content-Type: multipart/signed; boundary="=_mimegpg-monstr-desktop-27121-1392045788-0001"; micalg=pgp-sha1; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This is a MIME GnuPG-signed message. If you see this text, it means that your E-mail or Usenet software does not support MIME signed messages. The Internet standard for MIME PGP messages, RFC 2015, was published in 1996. To open this message correctly you will need to install E-mail or Usenet software that supports modern Internet standards. --=_mimegpg-monstr-desktop-27121-1392045788-0001 Use early syscon initialization to simplify slcr code. - Remove two slcr inits (zynq_slcr_init, zynq_early_slcr_init) - Directly use regmap accesses in zynq_slcr_read/write - Remove zynq_clock_init() and use addresses from syscon (This is the most problematic part now because clock doesn't support regmap accesses that's why reading slcr base is ugly. There are some attempts to get clk regmap to work - for example: https://lkml.org/lkml/2013/10/16/112) Signed-off-by: Michal Simek --- Especially look at slcr.c which is much simpler than was before. clkc.c will be simpler when regmap support is added because then syscon_early_regmap_lookup_by_phandle() will be called without zynq_slcr_base search. --- arch/arm/boot/dts/zynq-7000.dtsi | 1 + arch/arm/mach-zynq/common.c | 6 ++--- arch/arm/mach-zynq/slcr.c | 42 ++--------------------------- drivers/clk/zynq/clkc.c | 57 ++++++++++++---------------------------- 4 files changed, 23 insertions(+), 83 deletions(-) diff --git a/arch/arm/boot/dts/zynq-7000.dtsi b/arch/arm/boot/dts/zynq-7000.dtsi index 7284499..e414489 100644 --- a/arch/arm/boot/dts/zynq-7000.dtsi +++ b/arch/arm/boot/dts/zynq-7000.dtsi @@ -135,6 +135,7 @@ compatible = "xlnx,zynq-slcr", "syscon"; reg = <0xF8000000 0x1000>; ranges; + syscon = <&slcr>; clkc: clkc@100 { #clock-cells = <1>; compatible = "xlnx,ps7-clkc"; diff --git a/arch/arm/mach-zynq/common.c b/arch/arm/mach-zynq/common.c index 9d3c88e..78589e3 100644 --- a/arch/arm/mach-zynq/common.c +++ b/arch/arm/mach-zynq/common.c @@ -28,6 +28,7 @@ #include #include #include +#include #include #include @@ -130,15 +131,14 @@ out: of_platform_populate(NULL, of_default_bus_match_table, NULL, parent); platform_device_register(&zynq_cpuidle_device); - - zynq_slcr_init(); } static void __init zynq_timer_init(void) { + early_syscon_init(); + zynq_early_slcr_init(); - zynq_clock_init(); of_clk_init(NULL); clocksource_of_init(); } diff --git a/arch/arm/mach-zynq/slcr.c b/arch/arm/mach-zynq/slcr.c index 594b280..a89b082 100644 --- a/arch/arm/mach-zynq/slcr.c +++ b/arch/arm/mach-zynq/slcr.c @@ -35,7 +35,6 @@ #define SLCR_PSS_IDCODE_DEVICE_SHIFT 12 #define SLCR_PSS_IDCODE_DEVICE_MASK 0x1F -static void __iomem *zynq_slcr_base; static struct regmap *zynq_slcr_regmap; /** @@ -48,11 +47,6 @@ static struct regmap *zynq_slcr_regmap; */ static int zynq_slcr_write(u32 val, u32 offset) { - if (!zynq_slcr_regmap) { - writel(val, zynq_slcr_base + offset); - return 0; - } - return regmap_write(zynq_slcr_regmap, offset, val); } @@ -66,12 +60,7 @@ static int zynq_slcr_write(u32 val, u32 offset) */ static int zynq_slcr_read(u32 *val, u32 offset) { - if (zynq_slcr_regmap) - return regmap_read(zynq_slcr_regmap, offset, val); - - *val = readl(zynq_slcr_base + offset); - - return 0; + return regmap_read(zynq_slcr_regmap, offset, val); } /** @@ -169,24 +158,6 @@ void zynq_slcr_cpu_stop(int cpu) } /** - * zynq_slcr_init - Regular slcr driver init - * - * Return: 0 on success, negative errno otherwise. - * - * Called early during boot from platform code to remap SLCR area. - */ -int __init zynq_slcr_init(void) -{ - zynq_slcr_regmap = syscon_regmap_lookup_by_compatible("xlnx,zynq-slcr"); - if (IS_ERR(zynq_slcr_regmap)) { - pr_err("%s: failed to find zynq-slcr\n", __func__); - return -ENODEV; - } - - return 0; -} - -/** * zynq_early_slcr_init - Early slcr init function * * Return: 0 on success, negative errno otherwise. @@ -202,20 +173,11 @@ int __init zynq_early_slcr_init(void) pr_err("%s: no slcr node found\n", __func__); BUG(); } - - zynq_slcr_base = of_iomap(np, 0); - if (!zynq_slcr_base) { - pr_err("%s: Unable to map I/O memory\n", __func__); - BUG(); - } - - np->data = (__force void *)zynq_slcr_base; + zynq_slcr_regmap = syscon_early_regmap_lookup_by_phandle(np, "syscon"); /* unlock the SLCR so that registers can be changed */ zynq_slcr_unlock(); - pr_info("%s mapped to %p\n", np->name, zynq_slcr_base); - of_node_put(np); return 0; diff --git a/drivers/clk/zynq/clkc.c b/drivers/clk/zynq/clkc.c index c812b93..b2fd160 100644 --- a/drivers/clk/zynq/clkc.c +++ b/drivers/clk/zynq/clkc.c @@ -214,6 +214,10 @@ err: clks[clk1] = ERR_PTR(-ENOMEM); } +struct syscon { + void __iomem *base; +}; + static void __init zynq_clk_setup(struct device_node *np) { int i; @@ -227,6 +231,19 @@ static void __init zynq_clk_setup(struct device_node *np) const char *periph_parents[4]; const char *swdt_ext_clk_mux_parents[2]; const char *can_mio_mux_parents[NUM_MIO_PINS]; + struct resource res; + void __iomem *zynq_slcr_base; + + struct device_node *slcr = of_get_parent(np); + struct syscon *syscon = slcr->data; + zynq_slcr_base = syscon->base; + + if (of_address_to_resource(np, 0, &res)) { + pr_err("%s: failed to get resource\n", np->name); + return; + } + + zynq_clkc_base = zynq_slcr_base + res.start; pr_info("Zynq clock init\n"); @@ -569,43 +586,3 @@ static void __init zynq_clk_setup(struct device_node *np) } CLK_OF_DECLARE(zynq_clkc, "xlnx,ps7-clkc", zynq_clk_setup); - -void __init zynq_clock_init(void) -{ - struct device_node *np; - struct device_node *slcr; - struct resource res; - - np = of_find_compatible_node(NULL, NULL, "xlnx,ps7-clkc"); - if (!np) { - pr_err("%s: clkc node not found\n", __func__); - goto np_err; - } - - if (of_address_to_resource(np, 0, &res)) { - pr_err("%s: failed to get resource\n", np->name); - goto np_err; - } - - slcr = of_get_parent(np); - - if (slcr->data) { - zynq_clkc_base = (__force void __iomem *)slcr->data + res.start; - } else { - pr_err("%s: Unable to get I/O memory\n", np->name); - of_node_put(slcr); - goto np_err; - } - - pr_info("%s: clkc starts at %p\n", __func__, zynq_clkc_base); - - of_node_put(slcr); - of_node_put(np); - - return; - -np_err: - of_node_put(np); - BUG(); - return; -} -- 1.8.2.3 --=_mimegpg-monstr-desktop-27121-1392045788-0001 Content-Type: application/pgp-signature Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iEYEABECAAYFAlL47twACgkQykllyylKDCH4lgCgib15xAjL4XjQFzI8/soUUKsi fScAn3ZC4ynpNxuX6WC4Tz9kWIj/IoqO =nQ5r -----END PGP SIGNATURE----- --=_mimegpg-monstr-desktop-27121-1392045788-0001-- -- 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/