From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752498Ab0IHR24 (ORCPT ); Wed, 8 Sep 2010 13:28:56 -0400 Received: from mho-01-ewr.mailhop.org ([204.13.248.71]:65288 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752180Ab0IHR2t (ORCPT ); Wed, 8 Sep 2010 13:28:49 -0400 X-Mail-Handler: MailHop Outbound by DynDNS X-Originating-IP: 69.181.193.102 X-Report-Abuse-To: abuse@dyndns.com (see http://www.dyndns.com/services/mailhop/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX19LmflqUMAzc2d0/9N6FWwa Date: Wed, 8 Sep 2010 10:28:45 -0700 From: Tony Lindgren To: Greg KH Cc: David Cross , linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org Subject: Re: [PATCH] gpmc, EXPORT_SYMBOLS, west bridge related Message-ID: <20100908172845.GO22507@atomide.com> References: <1283887607.7250.10.camel@odc-laptop> <20100907235800.GD12823@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100907235800.GD12823@kroah.com> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Greg KH [100907 17:17]: > On Tue, Sep 07, 2010 at 12:26:47PM -0700, David Cross wrote: > > This patch exports some of the gpmc driver functions in OMAP3. The purpose behind this patch > > is to allow device drivers compiled as loadable modules to be interfaced to the GPMC. I am > > hoping that Tony is the correct maintainer and willing to ACK this change. Please let me know > > if there are any issues or concerns with this patch. > > Thanks, > > David > > > > Signed-off-by: David Cross > > > > diff -uprN -X linux-next-vanilla/Documentation/dontdiff linux-next-vanilla/arch/arm/mach-omap2/gpmc.c linux-next-incl-sdk/arch/arm/mach-omap2/gpmc.c > > --- linux-next-vanilla/arch/arm/mach-omap2/gpmc.c 2010-08-31 19:32:51.000000000 -0700 > > +++ linux-next-incl-sdk/arch/arm/mach-omap2/gpmc.c 2010-09-01 16:10:21.000000000 -0700 > > @@ -133,6 +133,7 @@ void gpmc_cs_write_reg(int cs, int idx, > > reg_addr = gpmc_base + GPMC_CS0_OFFSET + (cs * GPMC_CS_SIZE) + idx; > > __raw_writel(val, reg_addr); > > } > > +EXPORT_SYMBOL(gpmc_cs_write_reg); > > EXPORT_SYMBOL_GPL() perhaps? > > What about platforms that don't have this symbol, how will the driver > build properly then? Shouldn't something like this be in a arch-neutral > place in the kernel tree? I don't think exporting these functions is a good idea, so NAK from me for that. All the drivers should be generic, and whatever needs to be done that's omap specific should be done in the platform init code. If the driver needs to reconfigure something, such as memory timings, it can be done by passing a retime function pointer to the driver in the platform_data. Then the driver can call that function if it exists. Regards, Tony