From patchwork Mon May 6 18:30:36 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vaibhav Hiremath X-Patchwork-Id: 2524891 Return-Path: X-Original-To: patchwork-linux-omap@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id B29D73FC5A for ; Mon, 6 May 2013 18:31:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754527Ab3EFSbI (ORCPT ); Mon, 6 May 2013 14:31:08 -0400 Received: from bear.ext.ti.com ([192.94.94.41]:42851 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754031Ab3EFSbH convert rfc822-to-8bit (ORCPT ); Mon, 6 May 2013 14:31:07 -0400 Received: from dbdlxv05.itg.ti.com ([172.24.171.60]) by bear.ext.ti.com (8.13.7/8.13.7) with ESMTP id r46IUge3024612; Mon, 6 May 2013 13:30:43 -0500 Received: from DBDE72.ent.ti.com (dbde72.ent.ti.com [172.24.171.97]) by dbdlxv05.itg.ti.com (8.14.3/8.13.8) with ESMTP id r46IUcH4006747; Mon, 6 May 2013 13:30:39 -0500 Received: from DBDE04.ent.ti.com ([fe80::21ac:d9f:f810:c8e7]) by DBDE72.ent.ti.com ([fe80::bd56:32bf:87fd:2763%27]) with mapi id 14.02.0342.003; Tue, 7 May 2013 02:30:37 +0800 From: "Hiremath, Vaibhav" To: Tony Lindgren CC: "linux-omap@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" Subject: RE: mach-omap2/timer.c: Bug introduced while merging patch - ff931c82 Thread-Topic: mach-omap2/timer.c: Bug introduced while merging patch - ff931c82 Thread-Index: Ac5KKDi66bBp82szRKKLkCu0fW3iLwAAuOcAABaw+tA= Date: Mon, 6 May 2013 18:30:36 +0000 Message-ID: <79CD15C6BA57404B839C016229A409A83EC7C93A@DBDE04.ent.ti.com> References: <79CD15C6BA57404B839C016229A409A83EC7C45D@DBDE04.ent.ti.com> <20130506152706.GD28721@atomide.com> In-Reply-To: <20130506152706.GD28721@atomide.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.24.170.142] MIME-Version: 1.0 Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org > -----Original Message----- > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap- > owner@vger.kernel.org] On Behalf Of Tony Lindgren > Sent: Monday, May 06, 2013 8:57 PM > To: Hiremath, Vaibhav > Cc: linux-omap@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > arm-kernel@lists.infradead.org > Subject: Re: mach-omap2/timer.c: Bug introduced while merging patch - > ff931c82 > > * Hiremath, Vaibhav [130506 00:12]: > > Hi, > > > > Not sure whether this got discussed and fixed already, but Google > didn't give me anything :) > > > > The below commit got merged wrongly to Mainline (Changes from timer.c > got missed), and > > due to this none of omap boards will boot. > > > > > > From ff931c821bab6713a52b768b0cd7ee7e90713b36 Mon Sep 17 00:00:00 > 2001 > > From: Rajendra Nayak > > Date: Thu, 21 Mar 2013 11:04:52 +0000 > > Subject: ARM: OMAP: clocks: Delay clk inits atleast until slab is > initialized > > > > > > It seems Linux-omap tree has the fix patch for this, and it should > get pushed ASAP. > > Below are links for Linus's mainline commit [1], the actual patch > submitted to the list [2] and > > Commit present in Linux-omap [3]. > > Can you please provide a proper patch for the regression since sounds > like you already figured out what happened? > Ohh yeah, I already have a patch for it - From: Vaibhav Hiremath Date: Mon, 6 May 2013 13:23:27 +0530 Subject: [PATCH] ARM: OMAP2+: Regression: Add missing code while merging commit ff931c82 to Mainline While merging commit ff931c82 [1] to Mainline,, it seems not all the changes from the original patch [2] have been merged; changes from mach-omap2/timer.c have been dropped, which invokes xxx_clk_init() function required to register platform clock table. And without this none of OMAP family of devices will boot from Mainline. [1] http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/arch/arm/mach-omap2/io.c?id=ff931c821bab6713a52b768b0cd7ee7e90713b36 [2] https://patchwork.kernel.org/patch/2300071/ Signed-off-by: Vaibhav Hiremath --- arch/arm/mach-omap2/timer.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) Thanks, Vaibhav -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c index f12aa6c..31ae764 100644 --- a/arch/arm/mach-omap2/timer.c +++ b/arch/arm/mach-omap2/timer.c @@ -549,6 +549,8 @@ static inline void __init realtime_counter_init(void) clksrc_nr, clksrc_src, clksrc_prop) \ void __init omap##name##_gptimer_timer_init(void) \ { \ + if (omap_clk_init) \ + omap_clk_init(); \ omap_dmtimer_init(); \ omap2_gp_clockevent_init((clkev_nr), clkev_src, clkev_prop); \ omap2_gptimer_clocksource_init((clksrc_nr), clksrc_src, \ @@ -559,6 +561,8 @@ void __init omap##name##_gptimer_timer_init(void) \ clksrc_nr, clksrc_src, clksrc_prop) \ void __init omap##name##_sync32k_timer_init(void) \ { \ + if (omap_clk_init) \ + omap_clk_init(); \ omap_dmtimer_init(); \ omap2_gp_clockevent_init((clkev_nr), clkev_src, clkev_prop); \ /* Enable the use of clocksource="gp_timer" kernel parameter */ \