From mboxrd@z Thu Jan 1 00:00:00 1970 From: "DebBarma, Tarun Kanti" Subject: Re: [PATCH v2 0/4] ARM: OMAP2+: dmtimer: cleanup related to devm API and clk usage Date: Thu, 5 Jul 2012 09:32:58 +0530 Message-ID: References: <1334925563-20768-1-git-send-email-tarun.kanti@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: Received: from na3sys009aog125.obsmtp.com ([74.125.149.153]:39075 "EHLO na3sys009aog125.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750706Ab2GEEDA (ORCPT ); Thu, 5 Jul 2012 00:03:00 -0400 Received: by qadz32 with SMTP id z32so3424831qad.7 for ; Wed, 04 Jul 2012 21:02:59 -0700 (PDT) In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Tony Cc: "Shilimkar, Santosh" , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, "Cousson, Benoit" , Paul Walmsley , Kevin Hilman , Rajendra Nayak Hi Tony, On Thu, Jun 14, 2012 at 4:28 PM, Shilimkar, Santosh wrote: > Tony, > > On Thu, Jun 14, 2012 at 4:24 PM, DebBarma, Tarun Kanti > wrote: >> On Fri, Apr 20, 2012 at 6:09 PM, Tarun Kanti DebBarma >> wrote: >>> The devm API usage in probe() simplifies error handling operation. >>> Since iclk is not used in the driver it is removed from wherever >>> not needed. >>> Corrected the timer fck name mis-match between clock44xx_data.c and >>> omap_hwmod_44xx_data.c. >>> Added omap_hwmod_get_main_clk() API. There is no more need to construct >>> clock names using sprintf() to be used in clk_get() during initialization. >>> >>> Reference: >>> git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git >>> Commit: e816b57a337ea3b755de72bec38c10c864f23015 (Linux 3.4-rc3) >>> >>> Series is available here for reference: >>> git://gitorious.org/~tarunkanti/omap-sw-develoment/tarunkantis-linux-omap-dev dmtimer_cleanup_for_3.5 >>> >>> Tested on following platforms: >>> OMAP5, OMAP4430SDP, OMAP3430SDP, OMAP2430SDP. >>> Could not test on OMAP2420 due to unavailability of board. >>> >>> v2: >>> - Use devm_request_and_ioremap() instead of request_mem_region() and ioremap() >>> - Add omap_hwmod_get_main_clk() API >>> - Reverted changes of clock names from OMAP2 and OMAP3 platforms >>> >>> Cc: Cousson, Benoit >>> Cc: Paul Walmsley >>> Cc: Tony Lindgren >>> Cc: Kevin Hilman >>> Cc: Rajendra Nayak >>> Cc: Santosh Shilimkar >>> >>> Tarun Kanti DebBarma (4): >>> ARM: OMAP: dmtimer: use devm_ API and do some cleanup in probe() >>> ARM: OMAP2+: hwmod: add omap_hwmod_get_main_clk() API >>> ARM: OMAP2+: dmtimer: cleanup iclk usage >>> ARM: OMAP2+: dmtimer: cleanup fclk usage >> [ping] >> Only one of the patches in the series (ARM: OMAP2+: dmtimer: cleanup iclk usage) >> has been taken. I just applied the remaining patches on top of Linux 3.5-rc2 and >> tested on OMAP3 and OMAP4. Can the remaining patches be taken as well? > > Just to let you know that this cleanup is essential for Benoit's OMAP4 > hwmod cleanup > and OMAP5 minimal support series. This is a gentle reminder in case you have missed somehow!! Let me know if anything else needs to be done. Thanks. -- Tarun From mboxrd@z Thu Jan 1 00:00:00 1970 From: tarun.kanti@ti.com (DebBarma, Tarun Kanti) Date: Thu, 5 Jul 2012 09:32:58 +0530 Subject: [PATCH v2 0/4] ARM: OMAP2+: dmtimer: cleanup related to devm API and clk usage In-Reply-To: References: <1334925563-20768-1-git-send-email-tarun.kanti@ti.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Tony, On Thu, Jun 14, 2012 at 4:28 PM, Shilimkar, Santosh wrote: > Tony, > > On Thu, Jun 14, 2012 at 4:24 PM, DebBarma, Tarun Kanti > wrote: >> On Fri, Apr 20, 2012 at 6:09 PM, Tarun Kanti DebBarma >> wrote: >>> The devm API usage in probe() simplifies error handling operation. >>> Since iclk is not used in the driver it is removed from wherever >>> not needed. >>> Corrected the timer fck name mis-match between clock44xx_data.c and >>> omap_hwmod_44xx_data.c. >>> Added omap_hwmod_get_main_clk() API. There is no more need to construct >>> clock names using sprintf() to be used in clk_get() during initialization. >>> >>> Reference: >>> git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git >>> Commit: e816b57a337ea3b755de72bec38c10c864f23015 (Linux 3.4-rc3) >>> >>> Series is available here for reference: >>> git://gitorious.org/~tarunkanti/omap-sw-develoment/tarunkantis-linux-omap-dev dmtimer_cleanup_for_3.5 >>> >>> Tested on following platforms: >>> OMAP5, OMAP4430SDP, OMAP3430SDP, OMAP2430SDP. >>> Could not test on OMAP2420 due to unavailability of board. >>> >>> v2: >>> - Use devm_request_and_ioremap() instead of request_mem_region() and ioremap() >>> - Add omap_hwmod_get_main_clk() API >>> - Reverted changes of clock names from OMAP2 and OMAP3 platforms >>> >>> Cc: Cousson, Benoit >>> Cc: Paul Walmsley >>> Cc: Tony Lindgren >>> Cc: Kevin Hilman >>> Cc: Rajendra Nayak >>> Cc: Santosh Shilimkar >>> >>> Tarun Kanti DebBarma (4): >>> ARM: OMAP: dmtimer: use devm_ API and do some cleanup in probe() >>> ARM: OMAP2+: hwmod: add omap_hwmod_get_main_clk() API >>> ARM: OMAP2+: dmtimer: cleanup iclk usage >>> ARM: OMAP2+: dmtimer: cleanup fclk usage >> [ping] >> Only one of the patches in the series (ARM: OMAP2+: dmtimer: cleanup iclk usage) >> has been taken. I just applied the remaining patches on top of Linux 3.5-rc2 and >> tested on OMAP3 and OMAP4. Can the remaining patches be taken as well? > > Just to let you know that this cleanup is essential for Benoit's OMAP4 > hwmod cleanup > and OMAP5 minimal support series. This is a gentle reminder in case you have missed somehow!! Let me know if anything else needs to be done. Thanks. -- Tarun