From patchwork Thu Sep 6 12:58:19 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vaibhav Hiremath X-Patchwork-Id: 1414891 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) by patchwork2.kernel.org (Postfix) with ESMTP id 70A35DFFCF for ; Thu, 6 Sep 2012 13:09:17 +0000 (UTC) Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1T9bkb-0004mv-64; Thu, 06 Sep 2012 13:04:05 +0000 Received: from bear.ext.ti.com ([192.94.94.41]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1T9bf8-0000EB-6C for linux-arm-kernel@lists.infradead.org; Thu, 06 Sep 2012 12:58:29 +0000 Received: from dbdp20.itg.ti.com ([172.24.170.38]) by bear.ext.ti.com (8.13.7/8.13.7) with ESMTP id q86CwNNU027111; Thu, 6 Sep 2012 07:58:23 -0500 Received: from DBDE71.ent.ti.com (localhost [127.0.0.1]) by dbdp20.itg.ti.com (8.13.8/8.13.8) with ESMTP id q86CwKpX025890; Thu, 6 Sep 2012 18:28:20 +0530 (IST) Received: from dbdp33.itg.ti.com (172.24.170.252) by DBDE71.ent.ti.com (172.24.170.149) with Microsoft SMTP Server id 14.1.323.3; Thu, 6 Sep 2012 18:28:20 +0530 Received: from [172.24.133.105] (smtpvbd.itg.ti.com [172.24.170.250]) by dbdp33.itg.ti.com (8.13.8/8.13.8) with ESMTP id q86CwJGf015727; Thu, 6 Sep 2012 18:28:20 +0530 Message-ID: <50489DEB.9010709@ti.com> Date: Thu, 6 Sep 2012 18:28:19 +0530 From: Vaibhav Hiremath User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:15.0) Gecko/20120824 Thunderbird/15.0 MIME-Version: 1.0 To: Jon Hunter Subject: Re: [PATCH 00/10] ARM: OMAP: DMTIMER fixes and clean-up References: <1346871872-24413-1-git-send-email-jon-hunter@ti.com> In-Reply-To: <1346871872-24413-1-git-send-email-jon-hunter@ti.com> X-Spam-Note: CRM114 invocation failed X-Spam-Score: -6.9 (------) X-Spam-Report: SpamAssassin version 3.3.2 on merlin.infradead.org summary: Content analysis details: (-6.9 points) pts rule name description ---- ---------------------- -------------------------------------------------- -5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/, high trust [192.94.94.41 listed in list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record 0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay domain -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Cc: Tony Lindgren , Kevin Hilman , Paul Walmsley , linux-omap , linux-arm X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org On 9/6/2012 12:34 AM, Jon Hunter wrote: > This series includes several fixes for the OMAP DMTIMER driver and a few > clean-ups to simplify some of the code. This series is based upon 3.6-rc4. > > Tested on OMAP5912 OSK, OMAP2420 H4, OMAP3430 Beagle and OMAP4430 Panda. > Testing includes ... > 1. Booting kernel on above boards > 2. Set date and ensuring time of day is correct after 24 hours > 3. Checking the timer counter is incrementing when configuring and starting > a timer > 4. Checking the timer overflow interrupt when timer expires. > 5. Using different clock sources to operate the timer with. > 6. Running a loop test overnight that continually runs test #3 and #4 for > each available timer > On the other hand I have validated this patch-series on both AM37xEVM and BeagleBone platform. I used the test code which you shared with me to validate it. AM33XX Clock tree needs minor change here, int __init am33xx_clk_init(void) Thanks, Vaibhav > Jon Hunter (10): > ARM: OMAP3+: Implement timer workaround for errata i103 and i767 > ARM: OMAP: Fix timer posted mode support > ARM: OMAP3: Correct HWMOD DMTIMER SYSC register declarations > ARM: OMAP2/3: Define HWMOD software reset status for DMTIMERs > ARM: OMAP2+: Don't use __omap_dm_timer_reset() > ARM: OMAP: Fix dmtimer reset for timer1 > ARM: OMAP: Clean-up dmtimer reset code > ARM: OMAP: Clean-up timer posted mode support > ARM: OMAP: Add dmtimer interrupt disable function > ARM: OMAP: Remove unnecessary call to clk_get() > > arch/arm/mach-omap2/omap_hwmod_2xxx_ipblock_data.c | 15 ++- > arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | 41 ++++---- > arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 4 + > arch/arm/mach-omap2/timer.c | 27 +++-- > arch/arm/plat-omap/dmtimer.c | 109 +++++++++++++------- > arch/arm/plat-omap/include/plat/dmtimer.h | 95 ++++++++++------- > 6 files changed, 184 insertions(+), 107 deletions(-) > diff --git a/arch/arm/mach-omap2/clock33xx_data.c b/arch/arm/mach-omap2/clock33xx_data.c index a4006b2..daa36a5 100644 --- a/arch/arm/mach-omap2/clock33xx_data.c +++ b/arch/arm/mach-omap2/clock33xx_data.c @@ -1070,6 +1070,8 @@ static struct omap_clk am33xx_clks[] = { CLK(NULL, "gfx_fck_div_ck", &gfx_fck_div_ck, CK_AM33XX), CLK(NULL, "sysclkout_pre_ck", &sysclkout_pre_ck, CK_AM33XX), CLK(NULL, "clkout2_ck", &clkout2_ck, CK_AM33XX), + CLK(NULL, "timer_32k_ck", &clkdiv32k_ick, CK_AM33XX), + CLK(NULL, "timer_sys_ck", &sys_clkin_ck, CK_AM33XX), };