From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [v2 0/7] OMAP: GPIO: Use PM runtime framework Date: Thu, 21 Apr 2011 08:15:23 -0700 Message-ID: <87r58v3ab8.fsf@ti.com> References: <1303139217-10285-1-git-send-email-charu@ti.com> <20110419062633.GA15620@atomide.com> <87r58w4gq3.fsf@ti.com> <20110421054221.GF5918@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from na3sys009aog102.obsmtp.com ([74.125.149.69]:58162 "EHLO na3sys009aog102.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754459Ab1DUPP2 (ORCPT ); Thu, 21 Apr 2011 11:15:28 -0400 Received: by mail-pz0-f47.google.com with SMTP id 35so1222367pzk.34 for ; Thu, 21 Apr 2011 08:15:27 -0700 (PDT) In-Reply-To: <20110421054221.GF5918@atomide.com> (Tony Lindgren's message of "Thu, 21 Apr 2011 08:42:21 +0300") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Tony Lindgren Cc: "Varadarajan, Charulatha" , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, paul@pwsan.com Tony Lindgren writes: > * Kevin Hilman [110421 02:56]: >> Hi Tony, >> >> Tony Lindgren writes: >> >> > * Varadarajan, Charulatha [110418 18:00]: >> >> From: Charulatha V >> >> >> >> Use PM runtime framework in OMAP GPIO driver. >> > ... >> > >> >> arch/arm/mach-omap2/gpio.c | 6 + >> >> arch/arm/mach-omap2/pm34xx.c | 22 +- >> >> arch/arm/plat-omap/gpio.c | 766 ++++++++++++++++++-------------- >> >> arch/arm/plat-omap/include/plat/gpio.h | 3 +- >> >> 4 files changed, 439 insertions(+), 358 deletions(-) >> > >> > Before this series gets merged we first need to do the following: >> > - Pass some registers in platform_data so we can have common >> > functions in gpio.c instead of having to test for the bank->method >> > in each function. This allows getting rid of the ifdefs as >> > discussed in the ARM Linux consolidation thread. >> >> I have a series that starts this process, will post it shortly. > > Saw that, looks good :) Need to post it also to lakml BTW. Yeah, I only posted to l-o so far because it's not done. I only removed a few of the #ifdef sections, there are still several more to do. >> > - Move it to drivers/gpio >> > >> > - Further consolidate with whatever common GPIO code might >> > be coming up. >> >> Is the above order required? > > Yes please. Otherwise we'll have more of the "crazy churn" in > arch/arm/*omap*/ and the runtime PM patches should apply easily > in drivers/gpio too. > >> IMO, before we move it to drivers/gpio, this code needs to be runtime PM >> converted (using this series from Charu.) The runtime PM conversion >> removes a bunch of platform-specific hacks that should be moved into >> drivers/*. >> >> Personally, I think we should go in this order >> >> - runtime PM conversion > > Let's do this after the consolidation and move to drivers/gpio. > This way any additions can be reviewed by the drivers/gpio maintainers. OK >> - #ifdef cleanup > > Looks like you got this already done :) Well, it's only partially done. I only converted the direction and data in/out functions. There are several others to do still. >> - move to drivers/gpio > > With this part there may be more comments from the drivers/gpio > maintainers so we need to get to this point ASAP. OK >> - look at consolidation with other drivers >> >> I'm willing to queue the GPIO work in my tree while it's under way if >> you like as well. > > If so, please put all the consolidation patches into a separate > branch with absolutely zero additional new code. And let's merge > those branches into linux-omap devel-cleanup branch on regular > basis for testing. > OK Kevin