From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [GIT PULL] omap2 sparse fixes Date: Wed, 29 Sep 2010 14:56:36 -0700 Message-ID: <20100929215635.GC3117@atomide.com> References: <20100923225851.GR4211@atomide.com> <20100925001914.GO4211@atomide.com> <20100928152139.GB3117@atomide.com> <20100928183157.GH3117@atomide.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]:60266 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750996Ab0I2V4t (ORCPT ); Wed, 29 Sep 2010 17:56:49 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "G, Manjunath Kondaiah" Cc: "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Samuel Ortiz , Dmitry Torokhov , "linux-mtd@lists.infradead.org" , Russell King - ARM Linux * G, Manjunath Kondaiah [100929 12:42]: > > Tony, > > > -----Original Message----- > > From: Tony Lindgren [mailto:tony@atomide.com] > > Sent: Wednesday, September 29, 2010 12:02 AM > > To: G, Manjunath Kondaiah > > Cc: linux-omap@vger.kernel.org; > > linux-arm-kernel@lists.infradead.org; Samuel Ortiz; Dmitry > > Torokhov; linux-mtd@lists.infradead.org; Russell King - ARM Linux > > Subject: Re: [GIT PULL] omap2 sparse fixes > > > > * G, Manjunath Kondaiah [100928 08:26]: > ... > > > Yes please take out all patches that are not in arch/arm/*omap*/ path. > > The others really should go to the driver maintainers. > > > > Tony > > Here is new branch which has only arch/arm/*omap*/ related patches. Thanks. Noticed one more thing though: In your patch "OMAP: mach-omap2: Fix static function warnings" looks like you're adding omap2_gp_clockevent_set_gptimer twice: --- a/arch/arm/plat-omap/include/plat/timer-gp.h +++ b/arch/arm/mach-omap2/timer-gp.h @@ -11,7 +11,6 @@ #ifndef __ARCH_ARM_PLAT_OMAP_INCLUDE_MACH_TIMER_GP_H #define __ARCH_ARM_PLAT_OMAP_INCLUDE_MACH_TIMER_GP_H -int __init omap2_gp_clockevent_set_gptimer(u8 id); +extern int __init omap2_gp_clockevent_set_gptimer(u8 id); #endif - --- a/arch/arm/plat-omap/include/plat/dmtimer.h +++ b/arch/arm/plat-omap/include/plat/dmtimer.h @@ -81,6 +81,6 @@ unsigned int omap_dm_timer_read_counter(struct omap_dm_timer *timer); void omap_dm_timer_write_counter(struct omap_dm_timer *timer, unsigned int value); int omap_dm_timers_active(void); - +int omap2_gp_clockevent_set_gptimer(u8 id); #endif /* __ASM_ARCH_DMTIMER_H */ Please also be careful that your branch contains the same patches as you've posted for review. Nobody wants to pull different stuff than what was posted! Also looks like Paul had one comment related to the same patch. Regards, Tony