From patchwork Thu Nov 29 10:53:38 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Paul Walmsley X-Patchwork-Id: 1820981 Return-Path: X-Original-To: patchwork-linux-omap@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id 4E10FDF23A for ; Thu, 29 Nov 2012 10:53:40 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752277Ab2K2Kxj (ORCPT ); Thu, 29 Nov 2012 05:53:39 -0500 Received: from utopia.booyaka.com ([74.50.51.50]:43078 "EHLO utopia.booyaka.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751692Ab2K2Kxj (ORCPT ); Thu, 29 Nov 2012 05:53:39 -0500 Received: (qmail 28052 invoked by uid 1019); 29 Nov 2012 10:53:38 -0000 Date: Thu, 29 Nov 2012 10:53:38 +0000 (UTC) From: Paul Walmsley To: Jon Hunter cc: linux-omap , linux-arm Subject: Re: [PATCH 1/3] ARM: OMAP4: Update timer clock aliases In-Reply-To: <1352313782-21602-2-git-send-email-jon-hunter@ti.com> Message-ID: References: <1352313782-21602-1-git-send-email-jon-hunter@ti.com> <1352313782-21602-2-git-send-email-jon-hunter@ti.com> User-Agent: Alpine 2.00 (DEB 1167 2008-08-23) MIME-Version: 1.0 Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org Hi Jon, On Wed, 7 Nov 2012, Jon Hunter wrote: > Commit "ARM: dts: OMAP4: Update timer addresses" updated the device-tree > names of the OMAP4 timers 5-7 because the default address for the timers > was changed from the L3 address to the MPU private address. When booting > with device-tree, this introduces a regression when attempting to set > the parent clock of timers 5-7 to the sys_clk_div_ck. Therefore, update > the clock aliases for timer 5-7 to reflect the updated device-tree name > for the timers. Reviewing my E-mail inbox, just saw that this one is marked as being needed for v3.8. Is that still the case? If so, we should ask Tony to take an updated version of this patch, given the recent CCF conversion. Following is the updated patch. - Paul From: Jon Hunter Date: Thu, 29 Nov 2012 03:47:46 -0700 Subject: [PATCH] ARM: OMAP4: Update timer clock aliases Commit "ARM: dts: OMAP4: Update timer addresses" updated the device-tree names of the OMAP4 timers 5-7 because the default address for the timers was changed from the L3 address to the MPU private address. When booting with device-tree, this introduces a regression when attempting to set the parent clock of timers 5-7 to the sys_clk_div_ck. Therefore, update the clock aliases for timer 5-7 to reflect the updated device-tree name for the timers. Signed-off-by: Jon Hunter [paul@pwsan.com: updated to apply after the CCF conversion] Signed-off-by: Paul Walmsley --- arch/arm/mach-omap2/cclock44xx_data.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-omap2/cclock44xx_data.c b/arch/arm/mach-omap2/cclock44xx_data.c index aa56c3e..a1f0b55 100644 --- a/arch/arm/mach-omap2/cclock44xx_data.c +++ b/arch/arm/mach-omap2/cclock44xx_data.c @@ -1935,10 +1935,10 @@ static struct omap_clk omap44xx_clks[] = { CLK("4803e000.timer", "timer_sys_ck", &sys_clkin_ck, CK_443X), CLK("48086000.timer", "timer_sys_ck", &sys_clkin_ck, CK_443X), CLK("48088000.timer", "timer_sys_ck", &sys_clkin_ck, CK_443X), - CLK("49038000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), - CLK("4903a000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), - CLK("4903c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), - CLK("4903e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), + CLK("40138000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), + CLK("4013a000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), + CLK("4013c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), + CLK("4013e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), CLK(NULL, "cpufreq_ck", &dpll_mpu_ck, CK_443X), };