From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH v12 6/9] dmtimer: switch-over to platform device driver Date: Fri, 11 Mar 2011 11:14:17 -0800 Message-ID: <20110311191417.GF10079@atomide.com> References: <1299627948-20040-1-git-send-email-tarun.kanti@ti.com> <1299627948-20040-7-git-send-email-tarun.kanti@ti.com> <20110309220241.GR26463@atomide.com> <5A47E75E594F054BAF48C5E4FC4B92AB037A4DD852@dbde02.ent.ti.com> <20110310175611.GV26463@atomide.com> <5A47E75E594F054BAF48C5E4FC4B92AB037A4DD9D2@dbde02.ent.ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-01-ewr.mailhop.org ([204.13.248.71]:33324 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753256Ab1CKTOS (ORCPT ); Fri, 11 Mar 2011 14:14:18 -0500 Content-Disposition: inline In-Reply-To: <5A47E75E594F054BAF48C5E4FC4B92AB037A4DD9D2@dbde02.ent.ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "DebBarma, Tarun Kanti" Cc: "linux-omap@vger.kernel.org" * DebBarma, Tarun Kanti [110310 21:33]: > [...] > > > > > > > > * Tarun Kanti DebBarma [110308 15:41]: > > > > > switch-over to platform device driver through following changes: > > > > > (a) initiate dmtimer early initialization from omap2_gp_timer_init() > > > > > in timer-gp.c. This is equivalent of timer_init()->timer->init(). > > > > > (b) modify plat-omap/dmtimer routines to use new register map and > > > > > platform data. > > > > > > > > This patch breaks booting for 2420 and makes boot hang early > > > > on 2430. Note that to test with omap-for-linus branch you need > > > > to disable the following in omap2plus_defconfig: > > > > > > > > CONFIG_ARCH_OMAP3 > > > > CONFIG_ARCH_OMAP4 > > > > CONFIG_CPU_32v6K > > > I disabled above options on omap-for-linus *WITHOUT* my patch series. > > > I am still seeing crash. Here is the log. > > > I tried on two different setups. Not sure if I am missing something. > > > > Hmm sounds like H4 needs some attention. Can you try the following > > completely untested patch? > > This works fine! OK thanks, can I add your Tested-by then? Tony