From patchwork Fri Aug 23 17:19:35 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Soren Brinkmann X-Patchwork-Id: 269500 Return-Path: X-Original-To: incoming-imx@patchwork.ozlabs.org Delivered-To: patchwork-incoming-imx@bilbo.ozlabs.org Received: from casper.infradead.org (unknown [IPv6:2001:770:15f::2]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 5BD072C0084 for ; Sat, 24 Aug 2013 03:20:50 +1000 (EST) Received: from merlin.infradead.org ([2001:4978:20e::2]) by casper.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VCv2B-0006b3-0o; Fri, 23 Aug 2013 17:20:27 +0000 Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1VCv22-0000z3-O1; Fri, 23 Aug 2013 17:20:18 +0000 Received: from mail-db8lp0184.outbound.messaging.microsoft.com ([213.199.154.184] helo=db8outboundpool.messaging.microsoft.com) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VCv1v-0000ww-K1 for linux-arm-kernel@lists.infradead.org; Fri, 23 Aug 2013 17:20:13 +0000 Received: from mail13-db8-R.bigfish.com (10.174.8.229) by DB8EHSOBE026.bigfish.com (10.174.4.89) with Microsoft SMTP Server id 14.1.225.22; Fri, 23 Aug 2013 17:19:49 +0000 Received: from mail13-db8 (localhost [127.0.0.1]) by mail13-db8-R.bigfish.com (Postfix) with ESMTP id 890AE16024E; Fri, 23 Aug 2013 17:19:49 +0000 (UTC) X-Forefront-Antispam-Report: CIP:149.199.60.83; KIP:(null); UIP:(null); IPV:NLI; H:xsj-gw1; RD:unknown-60-83.xilinx.com; EFVD:NLI X-SpamScore: -4 X-BigFish: VPS-4(zzbb2dI98dIc89bh148cI1432I1418Idb82hzz1f42h208ch1ee6h1de0h1fdah2073h1202h1e76h1d1ah1d2ah1fc6hzz1de098h8275bh1de097hz2fh95h839h93fhd24hf0ah119dh1288h12a5h12a9h12bdh137ah13b6h1441h14ddh1504h1537h153bh162dh1631h1758h18e1h1946h19b5h1b0ah1d0ch1d2eh1d3fh1dfeh1dffh1e1dh1fe8h1ff5h906i1155h192ch) Received-SPF: pass (mail13-db8: domain of xilinx.com designates 149.199.60.83 as permitted sender) client-ip=149.199.60.83; envelope-from=soren.brinkmann@xilinx.com; helo=xsj-gw1 ; helo=xsj-gw1 ; Received: from mail13-db8 (localhost.localdomain [127.0.0.1]) by mail13-db8 (MessageSwitch) id 1377278386754548_10296; Fri, 23 Aug 2013 17:19:46 +0000 (UTC) Received: from DB8EHSMHS026.bigfish.com (unknown [10.174.8.251]) by mail13-db8.bigfish.com (Postfix) with ESMTP id B3B2E60041; Fri, 23 Aug 2013 17:19:46 +0000 (UTC) Received: from xsj-gw1 (149.199.60.83) by DB8EHSMHS026.bigfish.com (10.174.4.36) with Microsoft SMTP Server id 14.16.227.3; Fri, 23 Aug 2013 17:19:46 +0000 Received: from unknown-38-66.xilinx.com ([149.199.38.66] helo=xsj-smtp1.xilinx.com) by xsj-gw1 with esmtp (Exim 4.63) (envelope-from ) id 1VCv1V-0004s6-GI; Fri, 23 Aug 2013 10:19:45 -0700 Date: Fri, 23 Aug 2013 10:19:35 -0700 From: =?utf-8?B?U8O2cmVu?= Brinkmann To: Sebastian Hesselbarth , Steffen Trumtrar Subject: Re: [RFC 17/17] clk: zynq: remove call to of_clk_init References: <1376964271-22715-1-git-send-email-sebastian.hesselbarth@gmail.com> <1376964271-22715-18-git-send-email-sebastian.hesselbarth@gmail.com> <52172BAA.7020009@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <52172BAA.7020009@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-RCIS-Action: ALLOW Message-ID: X-OriginatorOrg: xilinx.com X-FOPE-CONNECTOR: Id%0$Dn%*$RO%0$TLS%0$FQDN%$TlsDn% X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20130823_132011_980107_41808ADA X-CRM114-Status: GOOD ( 34.17 ) X-Spam-Score: -1.9 (-) X-Spam-Report: SpamAssassin version 3.3.2 on merlin.infradead.org summary: Content analysis details: (-1.9 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/, no trust [213.199.154.184 listed in list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Cc: Russell King , Arnd Bergmann , Michal Simek , linux-kernel@vger.kernel.org, Soren Brinkmann , Mike Turquette , Steffen Trumtrar , linux-arm-kernel@lists.infradead.org X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.15 Precedence: list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org List-Id: linux-imx-kernel.lists.patchwork.ozlabs.org Hi Sebastian, Steffen, On Fri, Aug 23, 2013 at 11:30:18AM +0200, Sebastian Hesselbarth wrote: > On 08/23/13 02:59, Sören Brinkmann wrote: > >On Thu, Aug 22, 2013 at 05:26:47PM -0700, Sören Brinkmann wrote: > >>On Tue, Aug 20, 2013 at 04:04:31AM +0200, Sebastian Hesselbarth wrote: > >>>With arch/arm calling of_clk_init(NULL) from time_init(), we can now > >>>remove it from corresponding drivers/clk code. > >> > >>I think that would break Zynq. > >>If I see this correctly you call of_clk_init() from common code, > >>_before_ the SOC specific time init function is called. > >>The problem is, that we have code setting up a global pointer which is > >>required by zynq_clk_setup() which is triggered when of_clk_init() is > >>called. [ ... ] > thanks for looking into this. I also had a look at the files in > question. Based on Steffen's proposal, I prepared a diff that should do > the trick. It moves zynq_slcr_init() to early_init, instead of reusing > another hook that has magic cow powers (it calls irqchip_init that zynq > also wants sooner or later). > > Also, it removes zynq_clock_init() and let zynq_clk_setup() map the > register itself by finding the node and use of_iomap(). I realized that > clock registers are quite separated within slcr, so you can consider > to have your own node for the clk-provider. As Steffen is proposing > this but mentioned incompatible DT changes, I chose that intermediate > step above. > > It would be great, if you test the diff and prepare a patch out of > it, that I pick-up in the patch set. That way, we also have your > Signed-off on it. I looked into this. Looks like init_early() happens to early. I suspect slab is missing to make zynq_slcr_init() work. So, I moved it into init_irq(). Is there any init_call() type which is called at the correct time? I looked briefly into syscon and regmap, and that does actually look promising and to really fix this mess, I guess we have to wait a little until Steffen finishes his work on it. To facilitate Sebastian's series I came up with the patch below. The problem I have is, I do not really want the clkc to map the registers. They are in the SLCR and the SLCR driver is doing it, hence we should work with what that driver provides - which ideally would be based on regmap and syscon, but we're not there yet. Hence I somehow need to pass the SLCR pointer to the clkc. To avoid accessing the global pointer directly I kept the zynq_clock_init() routine which is called from zynq_slcr_init(). That is the best I could come up with quickly and w/o investing a lot of time to figure out the regmap and syscon stuff, which seems to be handled by Steffen already, anyway. It is essentially a stripped down version of Sebastian's proposal. Sören -----8<--------------------8<-------------------8<------------------- From bb7a02dad9cc578caf1e21a1b7f45ed602676bfa Mon Sep 17 00:00:00 2001 From: Soren Brinkmann Date: Fri, 23 Aug 2013 09:27:11 -0700 Subject: [PATCH RFC] arm: zynq: Don't call of_clk_init() of_clk_init() has been moved to be called from common code, therefore remove it from Zynq's clock init routine. Since the Zynq's clock setup routine relies on an initialized SLCR, zynq_slcr_init() is moved to init_irq() (note: it must be before init_time() but after slab is available, hence init_early() does not work). Signed-off-by: Soren Brinkmann --- arch/arm/mach-zynq/common.c | 9 ++++----- drivers/clk/zynq/clkc.c | 4 +++- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/arm/mach-zynq/common.c b/arch/arm/mach-zynq/common.c index 5f25256..f28046e 100644 --- a/arch/arm/mach-zynq/common.c +++ b/arch/arm/mach-zynq/common.c @@ -19,10 +19,9 @@ #include #include #include -#include -#include #include #include +#include #include #include @@ -58,10 +57,10 @@ static void __init zynq_init_machine(void) of_platform_bus_probe(NULL, zynq_of_bus_ids, NULL); } -static void __init zynq_timer_init(void) +static void __init zynq_init_irq(void) { + irqchip_init(); zynq_slcr_init(); - clocksource_of_init(); } static struct map_desc zynq_cortex_a9_scu_map __initdata = { @@ -104,8 +103,8 @@ static const char * const zynq_dt_match[] = { DT_MACHINE_START(XILINX_EP107, "Xilinx Zynq Platform") .smp = smp_ops(zynq_smp_ops), .map_io = zynq_map_io, + .init_irq = zynq_init_irq, .init_machine = zynq_init_machine, - .init_time = zynq_timer_init, .dt_compat = zynq_dt_match, .restart = zynq_system_reset, MACHINE_END diff --git a/drivers/clk/zynq/clkc.c b/drivers/clk/zynq/clkc.c index 089d3e3..53b851e 100644 --- a/drivers/clk/zynq/clkc.c +++ b/drivers/clk/zynq/clkc.c @@ -206,6 +206,9 @@ static void __init zynq_clk_setup(struct device_node *np) pr_info("Zynq clock init\n"); + if (WARN_ON(!zynq_slcr_base_priv)) + return; + /* get clock output names from DT */ for (i = 0; i < clk_max; i++) { if (of_property_read_string_index(np, "clock-output-names", @@ -532,5 +535,4 @@ CLK_OF_DECLARE(zynq_clkc, "xlnx,ps7-clkc", zynq_clk_setup); void __init zynq_clock_init(void __iomem *slcr_base) { zynq_slcr_base_priv = slcr_base; - of_clk_init(NULL); }